CSSKeyframesRule::appendRule is deprecated, but is actually the spec
authordino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Oct 2015 19:57:43 +0000 (19:57 +0000)
committerdino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Oct 2015 19:57:43 +0000 (19:57 +0000)
https://bugs.webkit.org/show_bug.cgi?id=150113

Reviewed by Simon Fraser.

Source/WebCore:

I stupidly deprecated the wrong function in
http://trac.webkit.org/changeset/174469

* css/CSSKeyframesRule.cpp:
(WebCore::CSSKeyframesRule::insertRule): Swap the code between these two.
(WebCore::CSSKeyframesRule::appendRule):

LayoutTests:

Update tests to deprecate the correct function.

* animations/change-keyframes-expected.txt:
* animations/keyframes-rule.html:
* animations/unprefixed-keyframes-rule.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@191132 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/animations/change-keyframes-expected.txt
LayoutTests/animations/keyframes-rule.html
LayoutTests/animations/unprefixed-keyframes-rule.html
Source/WebCore/ChangeLog
Source/WebCore/css/CSSKeyframesRule.cpp

index b87f1cc..acc511d 100644 (file)
@@ -1,3 +1,16 @@
+2015-10-15  Dean Jackson  <dino@apple.com>
+
+        CSSKeyframesRule::appendRule is deprecated, but is actually the spec
+        https://bugs.webkit.org/show_bug.cgi?id=150113
+
+        Reviewed by Simon Fraser.
+
+        Update tests to deprecate the correct function.
+
+        * animations/change-keyframes-expected.txt:
+        * animations/keyframes-rule.html:
+        * animations/unprefixed-keyframes-rule.html:
+
 2015-10-15  Zalan Bujtas  <zalan@apple.com>
 
         [Win] Update anonymous table results for Win port.
index 314daec..9254940 100644 (file)
@@ -1,5 +1,5 @@
-CONSOLE MESSAGE: line 58: CSSKeyframesRule 'appendRule' function is deprecated.  Use 'insertRule' instead.
-CONSOLE MESSAGE: line 60: CSSKeyframesRule 'appendRule' function is deprecated.  Use 'insertRule' instead.
+CONSOLE MESSAGE: line 57: CSSKeyframesRule 'insertRule' function is deprecated.  Use 'appendRule' instead.
+CONSOLE MESSAGE: line 59: CSSKeyframesRule 'insertRule' function is deprecated.  Use 'appendRule' instead.
 This test performs an animation of the left property and makes sure it is animating. Then it stops the animation, changes the keyframes to an animation of the top property, restarts the animation and makes sure top is animating.
 PASS - "left" property for "box" element at 0.5s saw something close to: 200
 PASS - "top" property for "box" element at 1s saw something close to: 100
index 6299dbb..1ff6992 100644 (file)
@@ -88,9 +88,9 @@ shouldBe("rules1.item(1).cssText", "'100% { left: 20px; }'");
 
 debug("");
 debug("Insert three new rules");
-keyframes1.insertRule("30% { left: 30px; }");
-keyframes1.insertRule("60% { left: 60px; }");
-keyframes1.insertRule("20% { left: 50px; }");
+keyframes1.appendRule("30% { left: 30px; }");
+keyframes1.appendRule("60% { left: 60px; }");
+keyframes1.appendRule("20% { left: 50px; }");
 shouldBe("rules1.length", "5");
 
 shouldBe("rules1.item(0).cssText", "'0% { left: 10px; }'");
index 1686eb8..6267934 100644 (file)
@@ -88,9 +88,9 @@ shouldBe("rules1.item(1).cssText", "'100% { left: 20px; }'");
 
 debug("");
 debug("Insert three new rules");
-keyframes1.insertRule("30% { left: 30px; }");
-keyframes1.insertRule("60% { left: 60px; }");
-keyframes1.insertRule("20% { left: 50px; }");
+keyframes1.appendRule("30% { left: 30px; }");
+keyframes1.appendRule("60% { left: 60px; }");
+keyframes1.appendRule("20% { left: 50px; }");
 shouldBe("rules1.length", "5");
 
 shouldBe("rules1.item(0).cssText", "'0% { left: 10px; }'");
index 5f05d1c..82f02fe 100644 (file)
@@ -1,3 +1,17 @@
+2015-10-15  Dean Jackson  <dino@apple.com>
+
+        CSSKeyframesRule::appendRule is deprecated, but is actually the spec
+        https://bugs.webkit.org/show_bug.cgi?id=150113
+
+        Reviewed by Simon Fraser.
+
+        I stupidly deprecated the wrong function in
+        http://trac.webkit.org/changeset/174469
+
+        * css/CSSKeyframesRule.cpp:
+        (WebCore::CSSKeyframesRule::insertRule): Swap the code between these two.
+        (WebCore::CSSKeyframesRule::appendRule):
+
 2015-10-14  David Hyatt  <hyatt@apple.com>
 
         Implement CSS Variables.
index 0952326..45691fa 100644 (file)
@@ -108,15 +108,6 @@ void CSSKeyframesRule::setName(const String& name)
 
 void CSSKeyframesRule::appendRule(const String& ruleText)
 {
-    if (CSSStyleSheet* parent = parentStyleSheet()) {
-        if (Document* ownerDocument = parent->ownerDocument())
-            ownerDocument->addConsoleMessage(MessageSource::JS, MessageLevel::Warning, ASCIILiteral("CSSKeyframesRule 'appendRule' function is deprecated.  Use 'insertRule' instead."));
-    }
-    insertRule(ruleText);
-}
-
-void CSSKeyframesRule::insertRule(const String& ruleText)
-{
     ASSERT(m_childRuleCSSOMWrappers.size() == m_keyframesRule->keyframes().size());
 
     CSSParser parser(parserContext());
@@ -132,6 +123,15 @@ void CSSKeyframesRule::insertRule(const String& ruleText)
     m_childRuleCSSOMWrappers.grow(length());
 }
 
+void CSSKeyframesRule::insertRule(const String& ruleText)
+{
+    if (CSSStyleSheet* parent = parentStyleSheet()) {
+        if (Document* ownerDocument = parent->ownerDocument())
+            ownerDocument->addConsoleMessage(MessageSource::JS, MessageLevel::Warning, ASCIILiteral("CSSKeyframesRule 'insertRule' function is deprecated.  Use 'appendRule' instead."));
+    }
+    appendRule(ruleText);
+}
+
 void CSSKeyframesRule::deleteRule(const String& s)
 {
     ASSERT(m_childRuleCSSOMWrappers.size() == m_keyframesRule->keyframes().size());