Summary: | Animated GIFs in page cache get updated | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jon Lee <jonlee> | ||||||
Component: | Page Loading | Assignee: | Jon Lee <jonlee> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | beidson, eric, simon.fraser, webkit-bug-importer, webkit.review.bot | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Jon Lee
2012-05-16 13:22:32 PDT
Created attachment 142339 [details]
Patch
Comment on attachment 142339 [details]
Patch
This change looks great. I've asked Jon on IRC to explore a Layout test which *might* be possible, but might also be impossible or unreasonably difficult.
But I'm fine with the change as-is
Created attachment 142370 [details]
Patch
Added test. Comment on attachment 142370 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=142370&action=review Awesome! > LayoutTests/fast/loader/image-in-page-cache-expected.txt:17 > + > + > +PASS successfullyParsed is true > + > +TEST COMPLETE > +Opening animated-image.html in new window > +PASS Opened animated-image.html > +Opening animated-image2.html in otherWindow > +PASS Opened animated-image2.html > +Opening animated-image3.html in otherWindow > +PASS Opened animated-image3.html > +Closing otherWindow > +PASS Closed otherWindow > + It's weird to me that TEST COMPLETE shows up so early in the output, but I won't stress about it. Comment on attachment 142370 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=142370&action=review > LayoutTests/fast/loader/image-in-page-cache.html:14 > +function runTest() { > + if (window.layoutTestController) { > + layoutTestController.dumpAsText(); > + layoutTestController.waitUntilDone(); > + layoutTestController.setCanOpenWindows(); > + layoutTestController.overridePreference("WebKitUsesPageCachePreferenceKey", 1); > + } Actually, please fix this indenting Comment on attachment 142370 [details]
Patch
r+, after you give the indentation a once-over
(In reply to comment #5) > (From update of attachment 142370 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=142370&action=review > > Awesome! > > > LayoutTests/fast/loader/image-in-page-cache-expected.txt:17 > > + > > + > > +PASS successfullyParsed is true > > + > > +TEST COMPLETE > > +Opening animated-image.html in new window > > +PASS Opened animated-image.html > > +Opening animated-image2.html in otherWindow > > +PASS Opened animated-image2.html > > +Opening animated-image3.html in otherWindow > > +PASS Opened animated-image3.html > > +Closing otherWindow > > +PASS Closed otherWindow > > + > > It's weird to me that TEST COMPLETE shows up so early in the output, but I won't stress about it. It's a timing issue that I realize can be fixed with a window.jsTestIsAsync = true. I will update the test and results with this. Committed r117366: <http://trac.webkit.org/changeset/117366> |