152756 - Add & implement imgIContainer::ResetAnimation

Status: RESOLVED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[ArronM (:paper)]

ResetAnimation, or starting from frame 0 again, is needed for various functions
in Mozilla (reloading page/image and setting image.src are two that come to mind).

StopAnimation and StartAnimation do not reset the animation to the beginning.
One could reset the animation on StopAnimation, but this would break backwards
compatibility and remove the "feature" of pausing an animation.

Comment 1

[ArronM (:paper)]

Adding Dependencies/Blockers.

While Bug 77497 is not a true blocker, I'm not going to put a patch up until
it's fixed. (because it would cause bit rot)

Comment 2

[ArronM (:paper)]

retargetting, adding more bugs that this one theoritically blocks, and some
general evil glaring at the blocker.

Comment 3

[ArronM (:paper)]

Blocker has moved to future.. thus fix will sit on my computer until the blocker
gets fixed.

Comment 4

[ArronM (:paper)]

Attachment: ResetAnimation

diff with a -5 in hopes I don't need to unbitrot (multiple imgContainerGIF are
awaiting in bugs)

Side Notes: imgContainerART in the Netscape code is going to also need a
ResetAnimation

Comment 5

[ArronM (:paper)]

Attachment: ResetAnimation

1) Disallow ResetAnimation after user has stopped animation (kDontAnimMode)
2) moved ResetAnimation call to better spot

This patch should fix all dependants except Bug 69405.

Comment 6

[Stuart Parmenter]

Comment on attachment 102118 [details] [diff] [review]
ResetAnimation

r=pavlov

Comment 7

[tor]

Comment on attachment 102118 [details] [diff] [review]
ResetAnimation

sr=tor

Comment 8

[Asa Dotzler [:asa]]

Comment on attachment 102118 [details] [diff] [review]
ResetAnimation

a=asa for checkin to 1.2beta (on behalf of drivers)

Comment 9

[ArronM (:paper)]

Checked In 10/09/2002 21:43

Comment 10

[ArronM (:paper)]

Build 2002101010 -> Fixed