Closed Bug 905678 Opened 6 years ago Closed 6 years ago

Use FrameAnimator class in RasterImage

Categories

(Core :: ImageLib, defect)

25 Branch
x86_64
All
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla26
Tracking Status
firefox24 --- unaffected
firefox25 - affected

People

(Reporter: milan, Assigned: milan)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

+++ This bug was initially created as a clone of Bug #899861 +++

The fix in bug 899861 was a (slightly minimized) back out of a part 7 patch from bug 717872.  This bug is to track the real fix, that uses FrameAnimator, but does not have the problem reported in bug 899861.
Keywords: regression
OS: Windows 7 → All
Once bug 905793 is fixed, we may just want to back out the fix to bug 899861 and go back to the code as originally written.
This gets us back to the part 7 of bug 717872. It needs bug 905793 landed to not regress bug 899861.
Attachment #792952 - Flags: review?(seth)
Comment on attachment 792952 [details] [diff] [review]
Backout the fix for bug 899861, which was itself a backout of a part of 717872.

Review of attachment 792952 [details] [diff] [review]:
-----------------------------------------------------------------

This looks good. Please update the commit message to say "r=seth" rather than "r=bgirard".
Attachment #792952 - Flags: review?(seth) → review+
This gets us back to the part 7 of bug 717872. It needs bug 905793 landed to not regress bug 899861.  Carry r+ from seth, just had the r= part in the comment wrong.
Attachment #792952 - Attachment is obsolete: true
Attachment #793067 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/c4fb173290bb
Assignee: nobody → milan
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.