Summary: | Delete the animated image catchup code | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Said Abou-Hallawa <sabouhallawa> | ||||||||||
Component: | Images | Assignee: | Said Abou-Hallawa <sabouhallawa> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | bfulgham, commit-queue, simon.fraser, thorton | ||||||||||
Priority: | P2 | ||||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 155322, 161566 | ||||||||||||
Attachments: |
|
Description
Said Abou-Hallawa
2016-10-13 15:30:34 PDT
Created attachment 291527 [details]
Patch
Created attachment 291529 [details]
Patch
Comment on attachment 291529 [details]
Patch
I am working on a test.
Created attachment 291700 [details]
Patch
Comment on attachment 291700 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=291700&action=review > Source/WebCore/ChangeLog:8 > + Test: fast/images/ordered-animated-image-frames.html Move this line down below the description of the changes (to line 32-ish). > Source/WebCore/ChangeLog:18 > + passed the next_frame_desired_time. In this case we should fire the timer passed -> past > Source/WebCore/ChangeLog:31 > + (k - 1) decoded. You can also justify this removal by the fact that it was never used on iOS. Created attachment 291724 [details]
Patch
Comment on attachment 291724 [details] Patch Clearing flags on attachment: 291724 Committed r207386: <http://trac.webkit.org/changeset/207386> All reviewed patches have been landed. Closing bug. |