Open
Bug 1237453
Opened 9 years ago
Updated 2 years ago
Remove Animation::mInEffectOnLastAnimationTimingUpdate and just use mProgressOnLastCompose
Categories
(Core :: DOM: Animation, defect)
Core
DOM: Animation
Tracking
()
NEW
Tracking | Status | |
---|---|---|
firefox46 | --- | affected |
People
(Reporter: birtles, Unassigned)
References
Details
Bug 1232563 introduced an mProgressOnLastCompose member to Animation. We can use this to do roughly the same thing as mInEffectOnLastAnimationTimingUpdate does. Specifically, mProgressOnLastCompose will be null if we were not in-effect on the last *ComposeStyle*. That's a bit different to mInEffectOnLastAnimationTimingUpdate which is only updated on each call to NotifyAnimationTimingUpdated. I think that's only problematic if it is possible to transition in or out of the "in effect" state and call ComposeStyle *without* triggering a call to NotifyAnimationTimingUpdated. i.e. if we had such a transition, called ComposeStyle, updated mProgressOnLastCompose, and then later called NotifyAnimationTimingUpdated we'd fail to notice the change in "in effect" state and potentially fail to update the animation cascade. So I guess we just need to be careful to ensure that is always the case.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•