KeyframeAnimation::animate() and AnimationBase::progress() both compute elapsedTime differently. This has not caused any visible problems so far. But with the new pauseAnimationAtTimeOnElementWithId() interface could expose problems since the differences are mostly in how they deal with pause. So I need to unify this into a common function
Created attachment 26125 [details] Patch
Comment on attachment 26125 [details] Patch It appears the behavior or progress() for elapsed times < 0 changes with this patch. I don't have enough context to tell if that's a good thing or not.
It does change the elapsed time < 0 behavior. When reviewing this I noticed that difference. The current formula is a composite of the two old formulae and I believe the current one does the right thing in all cases.
Fixed by Changeset 39849