811904 - Identify animated images again

Status: RESOLVED FIXED

(SeaMonkey :: Page Info, defect)

Description

[[e-mail]]

Bug 504041 removed scriptable access to the frame count for animated images. However it may still be possible to differentiate between an animated and a static image.

Comment 1

[[e-mail]]

Attachment: Proposed patch

Unfortunately the best name for the string had already gone...

Comment 2

[Philip Chee]

Comment on attachment 681705 [details] [diff] [review]
Proposed patch

> Bug 504041 removed scriptable access to the frame count for animated images.
> However it may still be possible to differentiate between an animated and a
> static image.
I think you meant:
Bug 584841 - Create private mozilla::imagelib::Image API for imgIContainer implementations to inherit from.

Also see Firefox:
Bug 550686 - Page Info -> Media does not display the "animated" status & frame count in the "Type" field, for background images.
Bug 713889 - In Page Info -> Media, the "(animated, N frames)" text is missing from the "Type" line, for animated <img>.

Tested on: <http://people.mozilla.com/~dolske/apng/demo.html>
And <https://bug550686.bugzilla.mozilla.org/attachment.cgi?id=584381>

r=me.

Comment 3

[[e-mail]]

(In reply to Philip Chee from comment #2)
> > Bug 504041 removed scriptable access to the frame count for animated images.
> > However it may still be possible to differentiate between an animated and a
> > static image.
> I think you meant:
> Bug 584841 - Create private mozilla::imagelib::Image API for imgIContainer
> implementations to inherit from.
I think the strikeout must have confused me, although right now I can't actually reproduce the problem on any of my browsers...

Comment 4

[[e-mail]]

Pushed comm-central changeset 68047c3ae87f.

Comment 5

[Frank Wein [:mcsmurf]]

Fixing Target Milestone