|Summary:||Modify page cache test to verify disabled cache too and to be usable as a template for page cache tests.|
|Product:||WebKit||Reporter:||Dmitry Titov <dimich>|
|Component:||Tools / Tests||Assignee:||Dmitry Titov <dimich>|
|Version:||528+ (Nightly build)|
|OS:||OS X 10.5|
Description Dmitry Titov 2009-08-19 18:25:22 PDT
Previously introduced (Bug 28324) LayoutTests/fast/harness/use-page-cache.html verified that using overridePreference indeed enables page cache for a test. As a follow up for the review comments in that bug, I've added a 'mirror' test that verifies that in the absence of override the page cache is disabled ('load' handler is called again when the window is navigated back to the same page). Also, re-wrote test by moving all the JS into helper script so the use-page-cache.html can now be used as a simple template for page cache tests.
Comment 2 Dmitry Titov 2009-08-21 18:17:47 PDT
Comment on attachment 35173 [details] Proposed patch Removing r? because of Bugzilla memory loss - it was reviewed and discussed on IRC and agreed that it is not an optimal approach. I'll revisit this latter to see if it makes sense to have a 'template' for page cache tests.