From 1b419007d17ef8865b18f685f6a9e2ae3643c6fd Mon Sep 17 00:00:00 2001
From: Simon Sapin <simon.sapin@exyr.org>
Date: Wed, 26 Apr 2017 07:37:10 +0200
Subject: [PATCH] =?UTF-8?q?CSSKeyframesRule::name=20setter=20doesn?=
 =?UTF-8?q?=E2=80=99t=20throw=20anymore?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

https://github.com/w3c/csswg-drafts/commit/063dc05e478096e90d377909414b6a1b051714ae
https://github.com/w3c/web-platform-tests/pull/5695
---
 tests/wpt/metadata/MANIFEST.json              |  2 +-
 .../cssom/CSSKeyframesRule.html               | 19 +++++++++++--------
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/tests/wpt/metadata/MANIFEST.json b/tests/wpt/metadata/MANIFEST.json
index 8c0c34ff0a8..fc43eb22216 100644
--- a/tests/wpt/metadata/MANIFEST.json
+++ b/tests/wpt/metadata/MANIFEST.json
@@ -553680,7 +553680,7 @@
    "testharness"
   ],
   "cssom/CSSKeyframesRule.html": [
-   "3efb8e5cef257a0b433192742d526709357b24c7",
+   "bca997a63c1389ef6d14aac2f32ab770fbd15ec4",
    "testharness"
   ],
   "cssom/CSSNamespaceRule.html": [
diff --git a/tests/wpt/web-platform-tests/cssom/CSSKeyframesRule.html b/tests/wpt/web-platform-tests/cssom/CSSKeyframesRule.html
index 9cf387def86..68453498767 100644
--- a/tests/wpt/web-platform-tests/cssom/CSSKeyframesRule.html
+++ b/tests/wpt/web-platform-tests/cssom/CSSKeyframesRule.html
@@ -10,6 +10,7 @@
             0% { top: 0px; }
             100% { top: 200px; }
         }
+        @keyframes empty {}
     </style>
 
     <script>
@@ -54,16 +55,18 @@
         assert_equals(keyframe.cssRules[2].cssText, "0% { top: 50px; }", "CSSKeyframesRule cssRule cssText attribute after deleteRule function");
         assert_equals(keyframe.cssRules[3], undefined, "CSSKeyframesRule cssRule cssText attribute after deleteRule function");
 
-        keyframe.name = "bar";
-        assert_equals(keyframe.name, "bar", "CSSKeyframesRule name setter");
+        var empty = document.styleSheets[0].cssRules[1];
+        empty.name = "bar";
+        assert_equals(empty.name, "bar", "CSSKeyframesRule name setter");
+        assert_equals(empty.cssText.replace(/\s/g, ""), "@keyframesbar{}", "CSSKeyframesRule cssText attribute");
 
-        assert_throws("SyntaxError",
-                      function () { keyframe.name = "initial"; },
-                      "CSSKeyframesRule name setter on invalid keyword.");
+        empty.name = "initial";
+        assert_equals(empty.name, "initial", "CSSKeyframesRule name setter, CSS-wide keyword");
+        assert_equals(empty.cssText.replace(/\s/g, ""), "@keyframes\"initial\"{}", "CSSKeyframesRule cssText attribute with CSS-wide keyword name");
 
-        assert_throws("SyntaxError",
-                      function () { keyframe.name = "none"; },
-                      "CSSKeyframesRule name setter on invalid keyword.");
+        empty.name = "none";
+        assert_equals(empty.name, "none", "CSSKeyframesRule name setter, 'none'");
+        assert_equals(empty.cssText.replace(/\s/g, ""), "@keyframes\"none\"{}", "CSSKeyframesRule cssText attribute with 'none' name");
     });
     </script>
 </head>