Identify animated images again

RESOLVED FIXED in seamonkey2.16

Status

SeaMonkey
Page Info
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: neil@parkwaycc.co.uk)

Tracking

unspecified
seamonkey2.16

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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.
(Assignee)

Comment 1

5 years ago
Created attachment 681705 [details] [diff] [review]
Proposed patch

Unfortunately the best name for the string had already gone...
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #681705 - Flags: review?(philip.chee)

Comment 2

5 years ago
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.
Attachment #681705 - Flags: review?(philip.chee) → review+
(Assignee)

Comment 3

5 years ago
(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...
(Assignee)

Comment 4

5 years ago
Pushed comm-central changeset 68047c3ae87f.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Fixing Target Milestone
Target Milestone: --- → seamonkey2.16
(Assignee)

Updated

4 years ago
Depends on: 850072
You need to log in before you can comment on or make changes to this bug.