Closed
Bug 152756
Opened 22 years ago
Closed 22 years ago
Add & implement imgIContainer::ResetAnimation
Categories
(Core :: Graphics: ImageLib, defect)
Core
Graphics: ImageLib
Tracking
()
RESOLVED
FIXED
mozilla1.3alpha
People
(Reporter: paper, Assigned: paper)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
pavlov
:
review+
tor
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•22 years ago
|
||
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)
Assignee | ||
Comment 2•22 years ago
|
||
retargetting, adding more bugs that this one theoritically blocks, and some
general evil glaring at the blocker.
Assignee | ||
Comment 3•22 years ago
|
||
Blocker has moved to future.. thus fix will sit on my computer until the blocker
gets fixed.
Target Milestone: mozilla1.2alpha → Future
Assignee | ||
Comment 4•22 years ago
|
||
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
Assignee | ||
Updated•22 years ago
|
Assignee | ||
Comment 5•22 years ago
|
||
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.
Attachment #102116 -
Attachment is obsolete: true
Comment 6•22 years ago
|
||
Comment on attachment 102118 [details] [diff] [review]
ResetAnimation
r=pavlov
Attachment #102118 -
Flags: review+
Comment on attachment 102118 [details] [diff] [review]
ResetAnimation
sr=tor
Attachment #102118 -
Flags: superreview+
Comment 8•22 years ago
|
||
Comment on attachment 102118 [details] [diff] [review]
ResetAnimation
a=asa for checkin to 1.2beta (on behalf of drivers)
Attachment #102118 -
Flags: approval+
Assignee | ||
Comment 9•22 years ago
|
||
Checked In 10/09/2002 21:43
Assignee | ||
Comment 10•22 years ago
|
||
Build 2002101010 -> Fixed
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•