Change values for WEBKIT_KEYFRAMES_RULE, WEBKIT_KEYFRAME_RULE
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 11 Feb 2012 03:10:31 +0000 (03:10 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 11 Feb 2012 03:10:31 +0000 (03:10 +0000)
https://bugs.webkit.org/show_bug.cgi?id=71293

Patch by Edward O'Connor <eoconnor@apple.com> on 2012-02-10
Reviewed by Chris Marrin.

Source/WebCore:

Tests: animations/animation-css-rule-types.html

* css/CSSRule.h: Change WEBKIT_KEYFRAMES_RULE to 7 and
WEBKIT_KEYFRAME_RULE to 8.
* css/CSSRule.idl: Ditto.

LayoutTests:

* animations/animation-css-rule-types-expected.txt:
* animations/script-tests/animation-css-rule-types.js: Update
expected WEBKIT_KEYFRAMES_RULE and WEBKIT_KEYFRAME_RULE values.

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

LayoutTests/ChangeLog
LayoutTests/animations/animation-css-rule-types-expected.txt
LayoutTests/animations/script-tests/animation-css-rule-types.js
Source/WebCore/ChangeLog
Source/WebCore/css/CSSRule.h
Source/WebCore/css/CSSRule.idl

index 40268e6..768460f 100644 (file)
@@ -1,3 +1,14 @@
+2012-02-10  Edward O'Connor  <eoconnor@apple.com>
+
+        Change values for WEBKIT_KEYFRAMES_RULE, WEBKIT_KEYFRAME_RULE
+        https://bugs.webkit.org/show_bug.cgi?id=71293
+
+        Reviewed by Chris Marrin.
+
+        * animations/animation-css-rule-types-expected.txt:
+        * animations/script-tests/animation-css-rule-types.js: Update
+        expected WEBKIT_KEYFRAMES_RULE and WEBKIT_KEYFRAME_RULE values.
+
 2012-02-10  Tony Chang  <tony@chromium.org>
 
         [chromium] Unreviewed gardening.
index 309cb50..4cffd7a 100644 (file)
@@ -3,8 +3,8 @@ This test checks that the CSSRule RuleTypes for keyframe-related rules are what
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
+PASS ruleType is 7
 PASS ruleType is 8
-PASS ruleType is 9
 If we got to this point then we did not crash and the test has passed.
 PASS successfullyParsed is true
 
index f32cc44..5d47a8f 100644 (file)
@@ -3,9 +3,9 @@ description(
 );
 
 var ruleType = window.CSSRule.WEBKIT_KEYFRAMES_RULE;
-shouldBe("ruleType", "8");
+shouldBe("ruleType", "7");
 ruleType = window.CSSRule.WEBKIT_KEYFRAME_RULE;
-shouldBe("ruleType", "9");
+shouldBe("ruleType", "8");
 
 debug('If we got to this point then we did not crash and the test has passed.');
 var successfullyParsed = true;
index 3ffa1a0..0d6afb9 100644 (file)
@@ -1,3 +1,16 @@
+2012-02-10  Edward O'Connor  <eoconnor@apple.com>
+
+        Change values for WEBKIT_KEYFRAMES_RULE, WEBKIT_KEYFRAME_RULE
+        https://bugs.webkit.org/show_bug.cgi?id=71293
+
+        Reviewed by Chris Marrin.
+
+        Tests: animations/animation-css-rule-types.html
+
+        * css/CSSRule.h: Change WEBKIT_KEYFRAMES_RULE to 7 and
+        WEBKIT_KEYFRAME_RULE to 8.
+        * css/CSSRule.idl: Ditto.
+
 2012-02-10  Eric Seidel  <eric@webkit.org>
 
         AtomicMarkupTokenBase::initializeAttributes should not create a StringImpl if it doesn't need to
index 4f3ca3b..5f56274 100644 (file)
@@ -49,10 +49,12 @@ public:
         MEDIA_RULE,
         FONT_FACE_RULE,
         PAGE_RULE,
-        // 7 used to be VARIABLES_RULE
-        WEBKIT_KEYFRAMES_RULE = 8,
+        // 7 was VARIABLES_RULE; we now match other browsers with 7 as
+        // KEYFRAMES_RULE:
+        // <https://bugs.webkit.org/show_bug.cgi?id=71293>.
+        WEBKIT_KEYFRAMES_RULE,
         WEBKIT_KEYFRAME_RULE,
-        WEBKIT_REGION_RULE
+        WEBKIT_REGION_RULE = 10
     };
 
     Type type() const { return static_cast<Type>(m_type); }
index ab0b8ef..7907f54 100644 (file)
@@ -37,8 +37,8 @@ module css {
         const unsigned short MEDIA_RULE = 4;
         const unsigned short FONT_FACE_RULE = 5;
         const unsigned short PAGE_RULE = 6;
-        const unsigned short WEBKIT_KEYFRAMES_RULE = 8;
-        const unsigned short WEBKIT_KEYFRAME_RULE = 9;
+        const unsigned short WEBKIT_KEYFRAMES_RULE = 7;
+        const unsigned short WEBKIT_KEYFRAME_RULE = 8;
         const unsigned short WEBKIT_REGION_RULE = 10;
 
         readonly attribute unsigned short   type;