Bug 1402170 - Drop invalid test for offsets; r?hiro draft
authorBrian Birtles <birtles@gmail.com>
Fri, 22 Sep 2017 11:43:11 +0900
changeset 673222 24ecd5e2ce63f339b74db6bc22e62c909970fb21
parent 673221 b43a3bd7cf2e1ca41023a485d6c5ed8b9f329826
child 673223 10f1ab8ce6292fe785e9427c52150b9e151840a0
push id82504
push userbmo:bbirtles@mozilla.com
push dateMon, 02 Oct 2017 05:09:17 +0000
reviewershiro
bugs1402170
milestone58.0a1
Bug 1402170 - Drop invalid test for offsets; r?hiro There is a test that assumes that an offset specified on a property-indexed keyframe is applied to all generated keyframes but that behavior is not (yet) specified. This behavior will be specified in [1] but until that happens it seems invalid to test for it. Furthermore, when that is specified we will need much more thorough tests than this one. [1] https://github.com/w3c/web-animations/issues/148 MozReview-Commit-ID: HUUw88dg2P7
testing/web-platform/meta/web-animations/interfaces/KeyframeEffect/processing-a-keyframes-argument.html.ini
testing/web-platform/tests/web-animations/interfaces/KeyframeEffect/processing-a-keyframes-argument.html
deleted file mode 100644
--- a/testing/web-platform/meta/web-animations/interfaces/KeyframeEffect/processing-a-keyframes-argument.html.ini
+++ /dev/null
@@ -1,5 +0,0 @@
-[processing-a-keyframes-argument.html]
-  type: testharness
-  [Equivalent property indexed and sequenced keyframes: same offset applied to all keyframes]
-    expected: FAIL
-
--- a/testing/web-platform/tests/web-animations/interfaces/KeyframeEffect/processing-a-keyframes-argument.html
+++ b/testing/web-platform/tests/web-animations/interfaces/KeyframeEffect/processing-a-keyframes-argument.html
@@ -120,27 +120,16 @@ const gEquivalentSyntaxTests = [
     },
     sequencedKeyframes: [
       {left: '0px', opacity: '0'},
       {left: '100px'},
       {left: '200px', opacity: '1'},
     ],
   },
   {
-    description: 'same offset applied to all keyframes',
-    indexedKeyframes: {
-      left: ['0px', '100px'],
-      offset: 0.5,
-    },
-    sequencedKeyframes: [
-      {left: '0px', offset: 0.5},
-      {left: '100px', offset: 0.5},
-    ],
-  },
-  {
     description: 'same easing applied to all keyframes',
     indexedKeyframes: {
       left: ['10px', '100px', '150px'],
       opacity: ['1', '0', '1'],
       easing: 'ease',
     },
     sequencedKeyframes: [
       {left: '10px', opacity: '1', easing: 'ease'},