Closed Bug 811904 Opened 7 years ago Closed 7 years ago

Identify animated images again

Categories

(SeaMonkey :: Page Info, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.16

People

(Reporter: neil, Assigned: neil)

References

Details

Attachments

(1 file)

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.
Attached patch Proposed patchSplinter Review
Unfortunately the best name for the string had already gone...
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #681705 - Flags: review?(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.
Attachment #681705 - Flags: review?(philip.chee) → review+
(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...
Pushed comm-central changeset 68047c3ae87f.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Fixing Target Milestone
Target Milestone: --- → seamonkey2.16
Depends on: 850072
You need to log in before you can comment on or make changes to this bug.