Bug 1305325 - Part 7: The expected value of offset is 1.0 for input value whose offset is 1.0. r=birtles
authorHiroyuki Ikezoe <hiikezoe@mozilla-japan.org>
Sun, 04 Dec 2016 08:07:40 +0900
changeset 325229 126968a8a0100a3371b83df57a6df4ef1d9054af
parent 325228 76b0afbf9a3a704683a1b635670e2b9dbcf18b9b
child 325230 30f9abc97cc3f750f68907376889ee933d66a7df
push id24
push usermaklebus@msu.edu
push dateTue, 20 Dec 2016 03:11:33 +0000
reviewersbirtles
bugs1305325
milestone53.0a1
Bug 1305325 - Part 7: The expected value of offset is 1.0 for input value whose offset is 1.0. r=birtles MozReview-Commit-ID: JnsWmDBfW3p
testing/web-platform/meta/web-animations/interfaces/Animatable/animate.html.ini
testing/web-platform/meta/web-animations/interfaces/KeyframeEffect/constructor.html.ini
testing/web-platform/meta/web-animations/interfaces/KeyframeEffect/setKeyframes.html.ini
testing/web-platform/tests/web-animations/resources/keyframe-utils.js
--- a/testing/web-platform/meta/web-animations/interfaces/Animatable/animate.html.ini
+++ b/testing/web-platform/meta/web-animations/interfaces/Animatable/animate.html.ini
@@ -1,13 +1,9 @@
 [animate.html]
   type: testharness
-  [Element.animate() accepts a one property one keyframe sequence]
-    expected: FAIL
-    bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1216844 # This will be passed in a subsequent patch.
-
   [Element.animate() accepts a keyframe sequence with different composite values, but the same composite value for a given offset]
     expected: FAIL
     bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1291468
 
   [Element.animate() accepts a single keyframe sequence with string offset]
     expected: FAIL
 
--- a/testing/web-platform/meta/web-animations/interfaces/KeyframeEffect/constructor.html.ini
+++ b/testing/web-platform/meta/web-animations/interfaces/KeyframeEffect/constructor.html.ini
@@ -7,19 +7,15 @@
   [composite values are parsed correctly when passed to the KeyframeEffectReadOnly constructor in regular keyframes]
     expected: FAIL
     bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1291468
 
   [composite values are parsed correctly when passed to the KeyframeEffectReadOnly constructor in KeyframeTimingOptions]
     expected: FAIL
     bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1311620 # This needs additive animation
 
-  [a KeyframeEffectReadOnly can be constructed with a one property one keyframe sequence]
-    expected: FAIL
-    bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1216844 # This will be passed in a subsequent patch
-
   [a KeyframeEffectReadOnly can be constructed with a keyframe sequence with different composite values, but the same composite value for a given offset]
     expected: FAIL
     bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1291468
 
   [a KeyframeEffectReadOnly can be constructed with a single keyframe sequence with string offset]
     expected: FAIL
 
--- a/testing/web-platform/meta/web-animations/interfaces/KeyframeEffect/setKeyframes.html.ini
+++ b/testing/web-platform/meta/web-animations/interfaces/KeyframeEffect/setKeyframes.html.ini
@@ -1,12 +1,9 @@
 [setKeyframes.html]
   type: testharness
-  [Keyframes can be replaced with a one property one keyframe sequence]
-    expected: FAIL # This will be passed in a subsequent patch.
-
   [Keyframes can be replaced with a keyframe sequence with different composite values, but the same composite value for a given offset]
     expected: FAIL
     bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1291468
 
   [Keyframes can be replaced with a single keyframe sequence with string offset]
     expected: FAIL
 
--- a/testing/web-platform/tests/web-animations/resources/keyframe-utils.js
+++ b/testing/web-platform/tests/web-animations/resources/keyframe-utils.js
@@ -177,17 +177,17 @@ var gPropertyIndexedKeyframesTests = [
                left: "10px" },
              { offset: null, computedOffset: 1, easing: "linear",
                left: "invalid" }] },
 ];
 
 var gKeyframeSequenceTests = [
   { desc:   "a one property one keyframe sequence",
     input:  [{ offset: 1, left: "10px" }],
-    output: [{ offset: null, computedOffset: 1, easing: "linear",
+    output: [{ offset: 1, computedOffset: 1, easing: "linear",
                left: "10px" }] },
   { desc:   "a one property two keyframe sequence",
     input:  [{ offset: 0, left: "10px" },
              { offset: 1, left: "20px" }],
     output: [{ offset: 0, computedOffset: 0, easing: "linear", left: "10px" },
              { offset: 1, computedOffset: 1, easing: "linear", left: "20px" }]
   },
   { desc:   "a two property two keyframe sequence",