Bug 1332206 - Drop some unnecessary calls to pause(); r=hiro
authorBrian Birtles <birtles@gmail.com>
Wed, 08 Feb 2017 09:25:30 +0900
changeset 341322 d18d24e941e13201d5bf0ff565ffd3908d700c0e
parent 341321 be72058340feb9b1087ea2dafa818e36426f366e
child 341323 f85b73bd0790056fa17c5fdc5b148ee98d2ba896
push id86684
push usercbook@mozilla.com
push dateWed, 08 Feb 2017 10:31:03 +0000
treeherdermozilla-inbound@c5b88e4e70f4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewershiro
bugs1332206
milestone54.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1332206 - Drop some unnecessary calls to pause(); r=hiro MozReview-Commit-ID: KCvFxh0DbQm
testing/web-platform/meta/MANIFEST.json
testing/web-platform/tests/web-animations/animation-model/keyframe-effects/effect-value-transformed-distance.html
--- a/testing/web-platform/meta/MANIFEST.json
+++ b/testing/web-platform/meta/MANIFEST.json
@@ -203737,17 +203737,17 @@
    "10d9ee521240475a1729c2facfcea8b50342614e",
    "testharness"
   ],
   "web-animations/animation-model/keyframe-effects/effect-value-overlapping-keyframes.html": [
    "a79db70a385ad767263f285c9401b66611087e42",
    "testharness"
   ],
   "web-animations/animation-model/keyframe-effects/effect-value-transformed-distance.html": [
-   "6c4b483ec111146c19c1f9ba436087b12805bb52",
+   "4f2879bda5e724ba00b1645195c7c0d1df15fa1c",
    "testharness"
   ],
   "web-animations/animation-model/keyframe-effects/effect-value-visibility.html": [
    "b01c7c5145c183fdca80dec4ca1966b0f72a7003",
    "testharness"
   ],
   "web-animations/animation-model/keyframe-effects/spacing-keyframes.html": [
    "318bc877791852b0829a3e10cb19e2a20a15adab",
--- a/testing/web-platform/tests/web-animations/animation-model/keyframe-effects/effect-value-transformed-distance.html
+++ b/testing/web-platform/tests/web-animations/animation-model/keyframe-effects/effect-value-transformed-distance.html
@@ -98,17 +98,16 @@ function assert_style_left_at(animation,
 test(function(t) {
   var target = createDiv(t);
   target.style.position = 'absolute';
   var anim = target.animate([ { left: '0px', easing: 'step-start' },
                               { left: '100px' } ],
                             { duration: 1000,
                               fill: 'forwards',
                               easing: 'cubic-bezier(0, 1.5, 1, 1.5)' });
-  anim.pause();
 
   // The bezier function produces values greater than 1 in (0.23368794, 1)
   anim.currentTime = 0;
   assert_equals(getComputedStyle(target).left, '100px');
   anim.currentTime = 230;
   assert_equals(getComputedStyle(target).left, '100px');
   anim.currentTime = 250;
   assert_equals(getComputedStyle(target).left, '100px');
@@ -119,17 +118,16 @@ test(function(t) {
 test(function(t) {
   var target = createDiv(t);
   target.style.position = 'absolute';
   var anim = target.animate([ { left: '0px', easing: 'step-end' },
                               { left: '100px' } ],
                             { duration: 1000,
                               fill: 'forwards',
                               easing: 'cubic-bezier(0, 1.5, 1, 1.5)' });
-  anim.pause();
 
   // The bezier function produces values greater than 1 in (0.23368794, 1)
   anim.currentTime = 0;
   assert_equals(getComputedStyle(target).left, '0px');
   anim.currentTime = 230;
   assert_equals(getComputedStyle(target).left, '0px');
   anim.currentTime = 250;
   assert_equals(getComputedStyle(target).left, '100px');
@@ -140,17 +138,16 @@ test(function(t) {
 test(function(t) {
   var target = createDiv(t);
   target.style.position = 'absolute';
   var anim = target.animate([ { left: '0px', easing: 'step-start' },
                               { left: '100px' } ],
                             { duration: 1000,
                               fill: 'forwards',
                               easing: 'cubic-bezier(0, -0.5, 1, -0.5)' });
-  anim.pause();
 
   // The bezier function produces negative values in (0, 0.766312060)
   anim.currentTime = 0;
   assert_equals(getComputedStyle(target).left, '100px');
   anim.currentTime = 750;
   assert_equals(getComputedStyle(target).left, '0px');
   anim.currentTime = 800;
   assert_equals(getComputedStyle(target).left, '100px');
@@ -161,17 +158,16 @@ test(function(t) {
 test(function(t) {
   var target = createDiv(t);
   target.style.position = 'absolute';
   var anim = target.animate([ { left: '0px', easing: 'step-end' },
                               { left: '100px' } ],
                             { duration: 1000,
                               fill: 'forwards',
                               easing: 'cubic-bezier(0, -0.5, 1, -0.5)' });
-  anim.pause();
 
   // The bezier function produces negative values in (0, 0.766312060)
   anim.currentTime = 0;
   assert_equals(getComputedStyle(target).left, '0px');
   anim.currentTime = 750;
   assert_equals(getComputedStyle(target).left, '0px');
   anim.currentTime = 800;
   assert_equals(getComputedStyle(target).left, '0px');
@@ -201,18 +197,16 @@ test(function(t) {
       'This function should be called in (1.0, infinity) range');
     // p3x + (p2y - p3y) / (p2x - p3x) * (x - p3x)
     return 1.0 + (0 - 1) / (0.5 - 1) * (x - 1.0);
   }
   var effectEasing = function(x) {
     return cubicBezier(0, 1.5, 1, 1.5)(x);
   }
 
-  anim.pause();
-
   // The effect-easing produces values greater than 1 in (0.23368794, 1)
   assert_style_left_at(anim, 0, function(x) {
     return keyframeEasing(effectEasing(x));
   });
   assert_style_left_at(anim, 230, function(x) {
     return keyframeEasing(effectEasing(x));
   });
   assert_style_left_at(anim, 240, function(x) {
@@ -250,18 +244,16 @@ test(function(t) {
   var easingExtrapolated = function(x) {
     assert_greater_than(x, 1.0,
       'This function should be called in negative range');
     // For cubic-bezier(0, 1.5, 1, 1.5), the tangent at the
     // endpoint (x = 1.0) is infinity so we should just return 1.0.
     return 1.0;
   }
 
-  anim.pause();
-
   // The effect-easing produces values greater than 1 in (0.23368794, 1)
   assert_style_left_at(anim, 0, function(x) {
     return easing(easing(x))
   });
   assert_style_left_at(anim, 230, function(x) {
     return easing(easing(x))
   });
   assert_style_left_at(anim, 240, function(x) {
@@ -302,18 +294,16 @@ test(function(t) {
       'This function should be called in negative range');
     // p0x + (p1y - p0y) / (p1x - p0x) * (x - p0x)
     return (1 / 0.5) * x;
   }
   var effectEasing = function(x) {
     return cubicBezier(0, -0.5, 1, -0.5)(x);
   }
 
-  anim.pause();
-
   // The effect-easing produces negative values in (0, 0.766312060)
   assert_style_left_at(anim, 0, function(x) {
     return keyframeEasing(effectEasing(x));
   });
   assert_style_left_at(anim, 10, function(x) {
     return keyframeEasingExtrapolated(effectEasing(x));
   });
   // Near the extreme point of the effect-easing function
@@ -351,18 +341,16 @@ test(function(t) {
   var easingExtrapolated = function(x) {
     assert_less_than(x, 0.0,
       'This function should be called in negative range');
     // For cubic-bezier(0, -0.5, 1, -0.5), the tangent at the
     // endpoint (x = 0.0) is infinity so we should just return 0.0.
     return 0.0;
   }
 
-  anim.pause();
-
   // The effect-easing produces negative values in (0, 0.766312060)
   assert_style_left_at(anim, 0, function(x) {
     return easing(easing(x))
   });
   assert_style_left_at(anim, 10, function(x) {
     return easingExtrapolated(easing(x));
   });
   // Near the extreme point of the effect-easing function