wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1249212 part 5 - Remove max() clamping from endTime calculation; r=boris

Browse Source 
Based on discussion at: https://github.com/w3c/web-animations/issues/86
This commit is contained in:
Brian Birtles 2016-02-19 08:37:32 +09:00
parent b78e012eb4
commit ab4f42928a
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
dom/animation/KeyframeEffect.cpp
View File

@ -42,7 +42,7 @@ GetComputedTimingDictionary(const ComputedTiming& aComputedTiming,
// ComputedTimingProperties
aRetVal.mActiveDuration = aComputedTiming.mActiveDuration.ToMilliseconds();
aRetVal.mEndTime
= std::max(aRetVal.mDelay + aRetVal.mActiveDuration + aRetVal.mEndDelay, 0.0);
= aRetVal.mDelay + aRetVal.mActiveDuration + aRetVal.mEndDelay;
aRetVal.mLocalTime = AnimationUtils::TimeDurationToDouble(aLocalTime);
aRetVal.mProgress = aComputedTiming.mProgress;
if (!aRetVal.mProgress.IsNull()) {

4
dom/animation/test/css-animations/file_animation-computed-timing.html
View File

@ -188,11 +188,11 @@ test(function(t) {
}, 'endTime of an infinitely repeating zero-duration animation');
test(function(t) {
// Fill forwards so div.getAnimations()[0] wouldn't return
// Fill forwards so div.getAnimations()[0] won't return an
// undefined value.
var div = addDiv(t, {style: 'animation: moveAnimation 10s -100s forwards'});
var effect = div.getAnimations()[0].effect;
assert_equals(effect.getComputedTiming().endTime, 0,
assert_equals(effect.getComputedTiming().endTime, -90000,
'Initial value of endTime');
}, 'endTime of an animation that finishes before its startTime');