47 bytes, text/x-phabricator-request
|Details | Review|
I was going to land this as part of recycling, but let's just get it now on its own. It will be good to have a nightly produced with just this setting.
Priority: -- → P3
Also, fix a minor bug where when we discard an animated image that is a full frame animated image, we need to reset AnimationState::mCompositedFrameInvalid to false, just like we do for animated images blended by FrameAnimator. This is because it is used as part of our state checking in FrameAnimator::GetCompositedFrame before we are willing to yield frame data.
Okay, that was harder than I thought.... try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b472593dab9565f1f6ab40923c5a519aedda440b
Apparently it is also hard if you somehow lost most of the patch in the push...for real this time: try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=919d7a8006a516c8f9549bb6278a76641bf71170
Assignee: nobody → aosmond
Status: NEW → ASSIGNED
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/3c7a3fc19c85 Enable animated images producing full frames by default. r=tnikkel
Status: ASSIGNED → RESOLVED
Last Resolved: 4 months ago
Resolution: --- → FIXED
status-firefox65: affected → fixed
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.