[Webkit-unassigned] [Bug 50430] [GTK] The webkit cache model needs to be set when WebFrameLoaderClient::didPerformFirstNavigation() is called
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Dec 6 01:30:52 PST 2010
https://bugs.webkit.org/show_bug.cgi?id=50430
Martin Robinson <mrobinson at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #75656|review? |review-
Flag| |
--- Comment #4 from Martin Robinson <mrobinson at webkit.org> 2010-12-06 01:30:52 PST ---
(From update of attachment 75656)
View in context: https://bugs.webkit.org/attachment.cgi?id=75656&action=review
I think some of the comments should be shuffled around and expanded.
> WebKit/gtk/WebCoreSupport/FrameLoaderClientGtk.cpp:698
> + // Page cache capacity (in pages). Comment from Mac port:
> + // (Research indicates that value / page drops substantially after 3 pages.)
This can should probably move to the relevant line in webkit_set_cache_model (the one that sets 3 cache size to 3 pages for WEBKIT_CACHE_MODEL_WEB_BROWSER).
> WebKit/gtk/WebCoreSupport/FrameLoaderClientGtk.cpp:699
> + // FIXME: Expose this with an API and/or calculate based on available resources
I think you're right that we can just remove this line now.
> WebKit/gtk/webkit/webkitwebview.h:54
> + WEBKIT_CACHE_MODEL_DEFAULT,
We should include some documentation with webkit_web_view_set_cache_model explaining that the default is WEB_BROWSER.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list