1901076 - Add new metadata decoding variant to calculate frame count

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, task)

Comment 1

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

Attachment: Bug 1901076 - Part 1. Include animated images loop count in metadata decoding results.

Comment 2

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

Attachment: Bug 1901076 - Part 2. Expose frame count variant of metadata decoders.

It is useful/necessary for WebCodecs image decoding support to be able
to calculate a frame count for the encoded image without actually
decoding every frame. It needs to be able to provide results without the
complete buffer as well.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/79fece25d231
Part 1. Include animated images loop count in metadata decoding results. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/de52938146cd
Part 2. Expose frame count variant of metadata decoders. r=tnikkel

Comment 4

[Sandor Molnar[:smolnar]]

Backed out for causing assertion failures @ image/RasterImage.cpp

Backout link: https://hg.mozilla.org/integration/autoland/rev/63d6d7df08f3e23e4911f0489b7292b2e684ba41

Push with failures

Failure log -> [3112] Assertion failure: mAnimationState->LoopCount() == aMetadata.GetLoopCount(), at /builds/worker/checkouts/gecko/image/RasterImage.cpp

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9bf9bf28bdeb
Part 1. Include animated images loop count in metadata decoding results. r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/762be8554a41
Part 2. Expose frame count variant of metadata decoders. r=tnikkel

Comment 6

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/9bf9bf28bdeb
https://hg.mozilla.org/mozilla-central/rev/762be8554a41