1508393 - Enable producing full frame animated images by default

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, enhancement, P3)

Description

[Andrew Osmond [:aosmond] (he/him)]

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.

Comment 1

[Andrew Osmond [:aosmond] (he/him)]

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=bc569457aeeadabc94746d042b5e69bea4a5c8e2

Comment 2

[Andrew Osmond [:aosmond] (he/him)]

Attachment: Bug 1508393 - Enable animated images producing full frames by default.

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.

Comment 3

[Andrew Osmond [:aosmond] (he/him)]

Okay, that was harder than I thought....

try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b472593dab9565f1f6ab40923c5a519aedda440b

Comment 4

[Andrew Osmond [:aosmond] (he/him)]

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

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3c7a3fc19c85
Enable animated images producing full frames by default. r=tnikkel

Comment 6

[Andrew Osmond [:aosmond] (he/him)]

https://hg.mozilla.org/mozilla-central/rev/3c7a3fc19c85ebca0be908143148c388f9d36f04