https://github.com/mozilla/gecko-dev
Revision cb967789c50c1f13b9679743eaae5ea78c089b9b authored by Brian Birtles on 28 May 2014, 07:51:49 UTC, committed by Brian Birtles on 28 May 2014, 07:51:49 UTC
This patch shuffles the code in ElementAnimations::GetEventsAt to make it easier
to follow.

It also removes a check for whether or not the animation is paused.
Previously we would not dispatch events if the animation was paused and in its
active phase (but we would if the animation had finished). There doesn't seem to
be any reason for this. If the animation was paused between the last sample and
the current sample and the boundary of an iteration also occurred in that time
then I expect we should dispatch that event. Removing this check for the pause
state does not cause any tests fail.

Separating out the event logic here makes it clear that we do not dispatch start
events in the situation where one sample falls before the active interval and
one sample falls after it (filed as bug 1004361). This patch adds a comment to
this effect.
1 parent 3ce242e
History
Tip revision: cb967789c50c1f13b9679743eaae5ea78c089b9b authored by Brian Birtles on 28 May 2014, 07:51:49 UTC
Bug 1004871 part 8 - Simplify ElementAnimations::GetEventsAt; r=dholbert
Tip revision: cb96778

README.txt

back to top