Bug 28324 – Add a test to verify that page cache can be enabled for a test

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Dmitry Titov]

The test verifies that override setting that enables page cache works correctly. It navigates forward and then back and checks that onload is not fired again  but the timer continues firing.

[Dmitry Titov]

Created attachment 34877 [details]
Proposed patch

[Eric Seidel (no email)]

Comment on attachment 34877 [details]
Proposed patch

Yay for trailblazing!

The test makes sense to me.  I assume you've tested to make sure it fails with:
 30         layoutTestController.overridePreference("WebKitUsesPageCachePreferenceKey", 1);
commented out.

I'm not sure we have a test to verify that onload would be sent twice w/ the page cache off.  That seems useful to test.

Eventually we'll want to make this sort of thing into a framework, so that it's easy to test page-cache bugs.  For example, we want to test what sort of things prevents a page from entering the cache.  Ideally that will eventually be as easy as making a single function call in a test with a little HTML passed to it. :)  But I dream...

LGTM assuming you've tested with the page cache off to make sure it fails.

[Dmitry Titov]

Landed http://trac.webkit.org/changeset/47306

Eric, I think your points are very good. I've landed the test so it is out there and tests things, however I will think if I can create some simple 'helper script' which can make it easy to write a page cache test!