Bug 1305325 - Part 7: The expected value of offset is 1.0 for input value whose offset is 1.0. r?birtles draft
authorHiroyuki Ikezoe <hiikezoe@mozilla-japan.org>
Sat, 26 Nov 2016 07:18:21 +0900
changeset 444078 7948076d058f57a5f7a585eeeff0fdcb7274bc2c
parent 444077 ccfb3fb7113004cc064b9c667ebce1ac8b371863
child 444079 8fc727b295bd882edf22ec9c7e66a084f4163679
push id37193
push userhiikezoe@mozilla-japan.org
push dateSat, 26 Nov 2016 03:37:22 +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",