Brian Birtles (:birtles) Reporter
	Description • 9 years ago

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.