Update CSS Exclusion wrap-flow values left & right to start & end
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Mar 2012 22:21:01 +0000 (22:21 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 29 Mar 2012 22:21:01 +0000 (22:21 +0000)
https://bugs.webkit.org/show_bug.cgi?id=82366

Source/WebCore:

http://dev.w3.org/csswg/css3-exclusions/
-webkit-wrap-flow now takes the values start and end rather than
left and right. Updating the code to reflect this. Functionality
is covered by existing tests.

Patch by Bear Travis <betravis@adobe.com> on 2012-03-29
Reviewed by Andreas Kling.

* css/CSSParser.cpp:
(WebCore::isValidKeywordPropertyAndValue):
* css/CSSPrimitiveValueMappings.h:
(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
(WebCore::CSSPrimitiveValue::operator WrapFlow):
* rendering/style/RenderStyleConstants.h:

LayoutTests:

Updating test values for the wrap-flow and wrap shorthand property

Patch by Bear Travis <betravis@adobe.com> on 2012-03-29
Reviewed by Andreas Kling.

* fast/exclusions/script-tests/wrap-flow-parsing.js:
* fast/exclusions/script-tests/wrap-parsing.js:
* fast/exclusions/wrap-flow-parsing-expected.txt:
* fast/exclusions/wrap-parsing-expected.txt:

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

LayoutTests/ChangeLog
LayoutTests/fast/exclusions/script-tests/wrap-flow-parsing.js
LayoutTests/fast/exclusions/script-tests/wrap-parsing.js
LayoutTests/fast/exclusions/wrap-flow-parsing-expected.txt
LayoutTests/fast/exclusions/wrap-parsing-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/css/CSSParser.cpp
Source/WebCore/css/CSSPrimitiveValueMappings.h
Source/WebCore/rendering/style/RenderStyleConstants.h

index c1e39b8..7f95d1e 100644 (file)
@@ -1,3 +1,17 @@
+2012-03-29  Bear Travis  <betravis@adobe.com>
+
+        Update CSS Exclusion wrap-flow values left & right to start & end
+        https://bugs.webkit.org/show_bug.cgi?id=82366
+
+        Updating test values for the wrap-flow and wrap shorthand property
+        
+        Reviewed by Andreas Kling.
+
+        * fast/exclusions/script-tests/wrap-flow-parsing.js:
+        * fast/exclusions/script-tests/wrap-parsing.js:
+        * fast/exclusions/wrap-flow-parsing-expected.txt:
+        * fast/exclusions/wrap-parsing-expected.txt:
+
 2012-03-29  Enrica Casucci  <enrica@apple.com>
 
         Skipping plugin mouse events tests failing on Mac platforms.
index 2031908..eb4192d 100644 (file)
@@ -34,8 +34,8 @@ function testNotInherited(parentValue, childValue) {
 
 shouldBeEqualToString('test("-webkit-wrap-flow: auto")', "auto");
 shouldBeEqualToString('test("-webkit-wrap-flow: both")', "both");
-shouldBeEqualToString('test("-webkit-wrap-flow: left")', "left");
-shouldBeEqualToString('test("-webkit-wrap-flow: right")', "right");
+shouldBeEqualToString('test("-webkit-wrap-flow: start")', "start");
+shouldBeEqualToString('test("-webkit-wrap-flow: end")', "end");
 shouldBeEqualToString('test("-webkit-wrap-flow: maximum")', "maximum");
 shouldBeEqualToString('test("-webkit-wrap-flow: clear")', "clear");
 
@@ -48,6 +48,6 @@ shouldBeEqualToString('testComputedStyle("auto")', "auto");
 shouldBeEqualToString('testComputedStyle("5")', "auto");
 shouldBeEqualToString('testComputedStyle("\'string\'")', "auto");
 
-shouldBeEqualToString('testNotInherited("auto", "left")', "left");
-shouldBeEqualToString('testNotInherited("right", "auto")', "auto");
+shouldBeEqualToString('testNotInherited("auto", "start")', "start");
+shouldBeEqualToString('testNotInherited("end", "auto")', "auto");
 shouldBeEqualToString('testNotInherited("both", "clear")', "clear");
index 16b7a3a..98e13e9 100644 (file)
@@ -18,8 +18,8 @@ function testComputedStyle(value) {
 }
 
 shouldBeEqualToString('test("-webkit-wrap: auto")', "auto");
-shouldBeEqualToString('test("-webkit-wrap: left 1px")', "left 1px");
-shouldBeEqualToString('test("-webkit-wrap: right 1px 2px")', "right 1px 2px");
+shouldBeEqualToString('test("-webkit-wrap: start 1px")', "start 1px");
+shouldBeEqualToString('test("-webkit-wrap: end 1px 2px")', "end 1px 2px");
 shouldBeEqualToString('test("-webkit-wrap: 5px both 10pt;")', "both 5px 10pt");
 shouldBeEqualToString('test("-webkit-wrap: 2px 3px clear;")', "clear 2px 3px");
 shouldBeEqualToString('test("-webkit-wrap: 5px maximum;")', "maximum 5px");
@@ -30,12 +30,12 @@ shouldBeEqualToString('test("-webkit-wrap: none 10px 10em;")', "");
 shouldBeEqualToString('test("-webkit-wrap: auto -5px;")', "");
 shouldBeEqualToString('test("-webkit-wrap: auto both;")', "");
 shouldBeEqualToString('test("-webkit-wrap: auto 1px -10px;")', "");
-shouldBeEqualToString('test("-webkit-wrap: -5px left;")', "");
-shouldBeEqualToString('test("-webkit-wrap: 5px right \'string\';")', "");
+shouldBeEqualToString('test("-webkit-wrap: -5px start;")', "");
+shouldBeEqualToString('test("-webkit-wrap: 5px end \'string\';")', "");
 
 shouldBeEqualToString('testComputedStyle("auto")', "auto 0px 0px");
-shouldBeEqualToString('testComputedStyle("left 1px")', "left 1px 0px");
-shouldBeEqualToString('testComputedStyle("right 1px 2px")', "right 1px 2px");
+shouldBeEqualToString('testComputedStyle("start 1px")', "start 1px 0px");
+shouldBeEqualToString('testComputedStyle("end 1px 2px")', "end 1px 2px");
 shouldBeEqualToString('testComputedStyle("5px maximum")', "maximum 5px 0px");
 shouldBeEqualToString('testComputedStyle("5px")', "auto 5px 0px");
 shouldBeEqualToString('testComputedStyle("5px 10px")', "auto 5px 10px");
index 93630b8..8a220f6 100644 (file)
@@ -5,8 +5,8 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 PASS test("-webkit-wrap-flow: auto") is "auto"
 PASS test("-webkit-wrap-flow: both") is "both"
-PASS test("-webkit-wrap-flow: left") is "left"
-PASS test("-webkit-wrap-flow: right") is "right"
+PASS test("-webkit-wrap-flow: start") is "start"
+PASS test("-webkit-wrap-flow: end") is "end"
 PASS test("-webkit-wrap-flow: maximum") is "maximum"
 PASS test("-webkit-wrap-flow: clear") is "clear"
 PASS test("-webkit-wrap-flow: ;") is ""
@@ -16,8 +16,8 @@ PASS test("-webkit-wrap-flow: 'string'") is ""
 PASS testComputedStyle("auto") is "auto"
 PASS testComputedStyle("5") is "auto"
 PASS testComputedStyle("'string'") is "auto"
-PASS testNotInherited("auto", "left") is "left"
-PASS testNotInherited("right", "auto") is "auto"
+PASS testNotInherited("auto", "start") is "start"
+PASS testNotInherited("end", "auto") is "auto"
 PASS testNotInherited("both", "clear") is "clear"
 PASS successfullyParsed is true
 
index 1c52812..37be1f8 100644 (file)
@@ -4,8 +4,8 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 PASS test("-webkit-wrap: auto") is "auto"
-PASS test("-webkit-wrap: left 1px") is "left 1px"
-PASS test("-webkit-wrap: right 1px 2px") is "right 1px 2px"
+PASS test("-webkit-wrap: start 1px") is "start 1px"
+PASS test("-webkit-wrap: end 1px 2px") is "end 1px 2px"
 PASS test("-webkit-wrap: 5px both 10pt;") is "both 5px 10pt"
 PASS test("-webkit-wrap: 2px 3px clear;") is "clear 2px 3px"
 PASS test("-webkit-wrap: 5px maximum;") is "maximum 5px"
@@ -15,11 +15,11 @@ PASS test("-webkit-wrap: none 10px 10em;") is ""
 PASS test("-webkit-wrap: auto -5px;") is ""
 PASS test("-webkit-wrap: auto both;") is ""
 PASS test("-webkit-wrap: auto 1px -10px;") is ""
-PASS test("-webkit-wrap: -5px left;") is ""
-PASS test("-webkit-wrap: 5px right 'string';") is ""
+PASS test("-webkit-wrap: -5px start;") is ""
+PASS test("-webkit-wrap: 5px end 'string';") is ""
 PASS testComputedStyle("auto") is "auto 0px 0px"
-PASS testComputedStyle("left 1px") is "left 1px 0px"
-PASS testComputedStyle("right 1px 2px") is "right 1px 2px"
+PASS testComputedStyle("start 1px") is "start 1px 0px"
+PASS testComputedStyle("end 1px 2px") is "end 1px 2px"
 PASS testComputedStyle("5px maximum") is "maximum 5px 0px"
 PASS testComputedStyle("5px") is "auto 5px 0px"
 PASS testComputedStyle("5px 10px") is "auto 5px 10px"
index a4e80e8..db123d3 100644 (file)
@@ -1,3 +1,22 @@
+2012-03-29  Bear Travis  <betravis@adobe.com>
+
+        Update CSS Exclusion wrap-flow values left & right to start & end
+        https://bugs.webkit.org/show_bug.cgi?id=82366
+
+        http://dev.w3.org/csswg/css3-exclusions/
+        -webkit-wrap-flow now takes the values start and end rather than
+        left and right. Updating the code to reflect this. Functionality
+        is covered by existing tests.
+        
+        Reviewed by Andreas Kling.
+
+        * css/CSSParser.cpp:
+        (WebCore::isValidKeywordPropertyAndValue):
+        * css/CSSPrimitiveValueMappings.h:
+        (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
+        (WebCore::CSSPrimitiveValue::operator WrapFlow):
+        * rendering/style/RenderStyleConstants.h:
+
 2012-03-29  Dirk Pranke  <dpranke@chromium.org>
 
         rollout r112484, r112545, r112574
index cfb6592..c67aae7 100644 (file)
@@ -775,7 +775,7 @@ static inline bool isValidKeywordPropertyAndValue(int propertyId, int valueID)
             return true;
         break;
     case CSSPropertyWebkitWrapFlow:
-        if (valueID == CSSValueAuto || valueID == CSSValueBoth || valueID == CSSValueLeft || valueID == CSSValueRight || valueID == CSSValueMaximum || valueID == CSSValueClear)
+        if (valueID == CSSValueAuto || valueID == CSSValueBoth || valueID == CSSValueStart || valueID == CSSValueEnd || valueID == CSSValueMaximum || valueID == CSSValueClear)
             return true;
         break;
     case CSSPropertyWebkitWrapThrough:
index 8c822d9..71ceae2 100644 (file)
@@ -3704,11 +3704,11 @@ template<> inline CSSPrimitiveValue::CSSPrimitiveValue(WrapFlow wrapFlow)
     case WrapFlowBoth:
         m_value.ident = CSSValueBoth;
         break;
-    case WrapFlowLeft:
-        m_value.ident = CSSValueLeft;
+    case WrapFlowStart:
+        m_value.ident = CSSValueStart;
         break;
-    case WrapFlowRight:
-        m_value.ident = CSSValueRight;
+    case WrapFlowEnd:
+        m_value.ident = CSSValueEnd;
         break;
     case WrapFlowMaximum:
         m_value.ident = CSSValueMaximum;
@@ -3726,10 +3726,10 @@ template<> inline CSSPrimitiveValue::operator WrapFlow() const
         return WrapFlowAuto;
     case CSSValueBoth:
         return WrapFlowBoth;
-    case CSSValueLeft:
-        return WrapFlowLeft;
-    case CSSValueRight:
-        return WrapFlowRight;
+    case CSSValueStart:
+        return WrapFlowStart;
+    case CSSValueEnd:
+        return WrapFlowEnd;
     case CSSValueMaximum:
         return WrapFlowMaximum;
     case CSSValueClear:
index 477b532..840688c 100644 (file)
@@ -459,7 +459,7 @@ enum LineSnap { LineSnapNone, LineSnapBaseline, LineSnapContain };
 
 enum LineAlign { LineAlignNone, LineAlignEdges };
 
-enum WrapFlow { WrapFlowAuto, WrapFlowBoth, WrapFlowLeft, WrapFlowRight, WrapFlowMaximum, WrapFlowClear };
+enum WrapFlow { WrapFlowAuto, WrapFlowBoth, WrapFlowStart, WrapFlowEnd, WrapFlowMaximum, WrapFlowClear };
 
 enum WrapThrough { WrapThroughWrap, WrapThroughNone };