1901077 - Make GIF decoder yield frame at frame complete rather than next frame start

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

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

WebCodecs tests verify that the decoder is able to produce frames as data comes in. They are structured such that only the bare minimum of data is provided, so we don't have the start of the next frame when it expects us to produce the current frame. We actually wait until the next frame starts before notifying the decoder listener about the previous frame, and there isn't much reason to do this. Let's fix that.

Comment 1

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

Attachment: Bug 1901077 - Switch GIF decoder to yield new frames at terminating block.

Originally we would yield a new frame when the next frame was available,
or the decoder finished. This is problematic for some WPTs used by
WebCodecs image decoding. There should be no functional difference
besides yielding a new frame slightly earlier.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7aa82d756333
Switch GIF decoder to yield new frames at terminating block. r=tnikkel

Comment 3

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/7aa82d756333