getPropertyValue for -webkit-column-rule returns null, should compute the shorthand...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Feb 2013 06:16:37 +0000 (06:16 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Feb 2013 06:16:37 +0000 (06:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=110698

Patch by Uday Kiran <udaykiran@motorola.com> on 2013-02-26
Reviewed by Alexander Pavlov.

Source/WebCore:

Test: fast/css/getPropertyValue-column-rule.html

* css/StylePropertySet.cpp:
(WebCore::StylePropertySet::getPropertyValue):

LayoutTests:

* fast/css/getPropertyValue-column-rule-expected.txt: Added.
* fast/css/getPropertyValue-column-rule.html: Added.

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

LayoutTests/ChangeLog
LayoutTests/fast/css/getPropertyValue-column-rule-expected.txt [new file with mode: 0644]
LayoutTests/fast/css/getPropertyValue-column-rule.html [new file with mode: 0644]
Source/WebCore/ChangeLog
Source/WebCore/css/StylePropertySet.cpp

index c340e5e417760a48ba6cfa65135975aa2c75f68b..c0eb094dea85db14d2812156f7c4f09e4ceb127b 100644 (file)
@@ -1,3 +1,13 @@
+2013-02-26  Uday Kiran  <udaykiran@motorola.com>
+
+        getPropertyValue for -webkit-column-rule returns null, should compute the shorthand value
+        https://bugs.webkit.org/show_bug.cgi?id=110698
+
+        Reviewed by Alexander Pavlov.
+
+        * fast/css/getPropertyValue-column-rule-expected.txt: Added.
+        * fast/css/getPropertyValue-column-rule.html: Added.
+
 2013-02-26  Uday Kiran  <udaykiran@motorola.com>
 
         getPropertyValue for -webkit-margin-collapse returns null, should compute the shorthand value
 2013-02-26  Uday Kiran  <udaykiran@motorola.com>
 
         getPropertyValue for -webkit-margin-collapse returns null, should compute the shorthand value
diff --git a/LayoutTests/fast/css/getPropertyValue-column-rule-expected.txt b/LayoutTests/fast/css/getPropertyValue-column-rule-expected.txt
new file mode 100644 (file)
index 0000000..0937e38
--- /dev/null
@@ -0,0 +1,24 @@
+          
+Bug 110698: getPropertyValue for -webkit-column-rule returns null, should compute the shorthand value
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+PASS div1.style.getPropertyValue("-webkit-column-rule") is '10px solid black'
+PASS div2.style.getPropertyValue("-webkit-column-rule") is '10px solid'
+PASS div3.style.getPropertyValue("-webkit-column-rule") is '20px red'
+PASS div4.style.getPropertyValue("-webkit-column-rule") is '10px'
+PASS div5.style.getPropertyValue("-webkit-column-rule") is 'solid black'
+PASS div6.style.getPropertyValue("-webkit-column-rule") is 'dashed'
+PASS div7.style.getPropertyValue("-webkit-column-rule") is 'green'
+PASS div8.style.getPropertyValue("-webkit-column-rule") is null
+NOTE: 'foo' is an illegal CSS value for '-webkit-column-rule'.
+PASS div9.style.getPropertyValue("-webkit-column-rule") is '10px dotted black'
+NOTE: If only few longhand properties are specified, getPropertyValue for shorthand property returns null.
+PASS div10.style.getPropertyValue("-webkit-column-rule") is null
+PASS div11.style.getPropertyValue("-webkit-column-rule") is null
+PASS div12.style.getPropertyValue("-webkit-column-rule") is null
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
diff --git a/LayoutTests/fast/css/getPropertyValue-column-rule.html b/LayoutTests/fast/css/getPropertyValue-column-rule.html
new file mode 100644 (file)
index 0000000..338040c
--- /dev/null
@@ -0,0 +1,74 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <style>
+      .test {
+        width: 100px;
+        height: 100px;
+        display: inline-block;
+      }
+    </style>
+    <script src="../js/resources/js-test-pre.js"></script>
+    <script>
+      if (window.testRunner)
+        testRunner.dumpAsText();
+    </script>
+  </head>
+  <body>
+    <div id="-webkit-column-rule1" class="test" style="-webkit-column-rule: 10px solid black;"></div>
+    <div id="-webkit-column-rule2" class="test" style="-webkit-column-rule: 10px solid;"></div>
+    <div id="-webkit-column-rule3" class="test" style="-webkit-column-rule: 20px red;"></div>
+    <div id="-webkit-column-rule4" class="test" style="-webkit-column-rule: 10px;"></div>
+    <div id="-webkit-column-rule5" class="test" style="-webkit-column-rule: solid black;"></div>
+    <div id="-webkit-column-rule6" class="test" style="-webkit-column-rule: dashed;"></div>
+    <div id="-webkit-column-rule7" class="test" style="-webkit-column-rule: green;"></div>
+    <div id="-webkit-column-rule8" class="test" style="-webkit-column-rule: foo;"></div>
+    <div id="-webkit-column-rule9" class="test" style="-webkit-column-rule-style: dotted; -webkit-column-rule-width: 10px; -webkit-column-rule-color: black;"></div>
+    <div id="-webkit-column-rule10" class="test" style="-webkit-column-rule-style: dashed; -webkit-column-rule-width: 10px;"></div>
+    <div id="-webkit-column-rule11" class="test" style="-webkit-column-rule-style: dotted; -webkit-column-rule-color: black;"></div>
+    <div id="-webkit-column-rule12" class="test" style="-webkit-column-rule-width: 10px;"></div>
+    <p id="description"></p>
+    <div id="console"></div>
+    <script>
+      description("<a href=\"https://bugs.webkit.org/show_bug.cgi?id=110698\">Bug 110698: getPropertyValue for -webkit-column-rule returns null, should compute the shorthand value</a>");
+      var div1 = document.getElementById("-webkit-column-rule1");
+      shouldBe('div1.style.getPropertyValue("-webkit-column-rule")', "'10px solid black'");
+
+      var div2 = document.getElementById("-webkit-column-rule2");
+      shouldBe('div2.style.getPropertyValue("-webkit-column-rule")', "'10px solid'");
+
+      var div3 = document.getElementById("-webkit-column-rule3");
+      shouldBe('div3.style.getPropertyValue("-webkit-column-rule")', "'20px red'");
+
+      var div4 = document.getElementById("-webkit-column-rule4");
+      shouldBe('div4.style.getPropertyValue("-webkit-column-rule")', "'10px'");
+
+      var div5 = document.getElementById("-webkit-column-rule5");
+      shouldBe('div5.style.getPropertyValue("-webkit-column-rule")', "'solid black'");
+
+      var div6 = document.getElementById("-webkit-column-rule6");
+      shouldBe('div6.style.getPropertyValue("-webkit-column-rule")', "'dashed'");
+
+      var div7 = document.getElementById("-webkit-column-rule7");
+      shouldBe('div7.style.getPropertyValue("-webkit-column-rule")', "'green'");
+
+      var div8 = document.getElementById("-webkit-column-rule8");
+      shouldBe('div8.style.getPropertyValue("-webkit-column-rule")', "null");
+      debug("NOTE: 'foo' is an illegal CSS value for '-webkit-column-rule'.");
+
+      var div9 = document.getElementById("-webkit-column-rule9");
+      shouldBe('div9.style.getPropertyValue("-webkit-column-rule")', "'10px dotted black'");
+
+      debug("NOTE: If only few longhand properties are specified, getPropertyValue for shorthand property returns null.");
+      var div10 = document.getElementById("-webkit-column-rule10");
+      shouldBe('div10.style.getPropertyValue("-webkit-column-rule")', "null");
+
+      var div11 = document.getElementById("-webkit-column-rule11");
+      shouldBe('div11.style.getPropertyValue("-webkit-column-rule")', "null");
+
+      var div12 = document.getElementById("-webkit-column-rule12");
+      shouldBe('div12.style.getPropertyValue("-webkit-column-rule")', "null");
+    </script>
+    <script src="../js/resources/js-test-post.js"></script>
+  </body>
+</html>
index 5771c4e9ae276a76dbd66946d2b7d08a301e6626..934ff4b65fa380f52a7b43b446baece8b8ce081e 100644 (file)
@@ -1,3 +1,15 @@
+2013-02-26  Uday Kiran  <udaykiran@motorola.com>
+
+        getPropertyValue for -webkit-column-rule returns null, should compute the shorthand value
+        https://bugs.webkit.org/show_bug.cgi?id=110698
+
+        Reviewed by Alexander Pavlov.
+
+        Test: fast/css/getPropertyValue-column-rule.html
+
+        * css/StylePropertySet.cpp:
+        (WebCore::StylePropertySet::getPropertyValue):
+
 2013-02-26  Uday Kiran  <udaykiran@motorola.com>
 
         getPropertyValue for -webkit-margin-collapse returns null, should compute the shorthand value
 2013-02-26  Uday Kiran  <udaykiran@motorola.com>
 
         getPropertyValue for -webkit-margin-collapse returns null, should compute the shorthand value
index ea92053e046dda1f1fc2b5dfc7d27817bab082a4..fa7e595d109bf3a88c2b8bd277c20dcc75f2aeec 100644 (file)
@@ -160,6 +160,8 @@ String StylePropertySet::getPropertyValue(CSSPropertyID propertyID) const
         return get4Values(borderWidthShorthand());
     case CSSPropertyBorderStyle:
         return get4Values(borderStyleShorthand());
         return get4Values(borderWidthShorthand());
     case CSSPropertyBorderStyle:
         return get4Values(borderStyleShorthand());
+    case CSSPropertyWebkitColumnRule:
+        return getShorthandValue(webkitColumnRuleShorthand());
     case CSSPropertyWebkitFlex:
         return getShorthandValue(webkitFlexShorthand());
     case CSSPropertyWebkitFlexFlow:
     case CSSPropertyWebkitFlex:
         return getShorthandValue(webkitFlexShorthand());
     case CSSPropertyWebkitFlexFlow: