Title: [93109] trunk/Source/WebCore
Revision
93109
Author
[email protected]
Date
2011-08-16 09:18:20 -0700 (Tue, 16 Aug 2011)

Log Message

Use appropriate macro to handle CSSPropertyWebkitContentOrder in CSSStyleSelector::applyProperty()
https://bugs.webkit.org/show_bug.cgi?id=66279

Reviewed by Simon Fraser.

No new tests - no functionality changed - simple refactoring only.

* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::applyProperty):
Use HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE macro to handle CSSPropertyWebkitContentOrder.
This works because PrimitiveValue now supports cast to int.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (93108 => 93109)


--- trunk/Source/WebCore/ChangeLog	2011-08-16 16:13:09 UTC (rev 93108)
+++ trunk/Source/WebCore/ChangeLog	2011-08-16 16:18:20 UTC (rev 93109)
@@ -1,3 +1,17 @@
+2011-08-16  Luke Macpherson   <[email protected]>
+
+        Use appropriate macro to handle CSSPropertyWebkitContentOrder in CSSStyleSelector::applyProperty()
+        https://bugs.webkit.org/show_bug.cgi?id=66279
+
+        Reviewed by Simon Fraser.
+
+        No new tests - no functionality changed - simple refactoring only.
+
+        * css/CSSStyleSelector.cpp:
+        (WebCore::CSSStyleSelector::applyProperty):
+        Use HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE macro to handle CSSPropertyWebkitContentOrder.
+        This works because PrimitiveValue now supports cast to int.
+
 2011-08-16  Eric Carlson  <[email protected]>
 
         [REGRESSION] Media controls should always be enabled when scripting is disabled

Modified: trunk/Source/WebCore/css/CSSStyleSelector.cpp (93108 => 93109)


--- trunk/Source/WebCore/css/CSSStyleSelector.cpp	2011-08-16 16:13:09 UTC (rev 93108)
+++ trunk/Source/WebCore/css/CSSStyleSelector.cpp	2011-08-16 16:18:20 UTC (rev 93109)
@@ -4700,13 +4700,11 @@
             m_style->setFlowThread(primitiveValue->getStringValue());
         return;
     case CSSPropertyWebkitContentOrder:
-        HANDLE_INHERIT_AND_INITIAL(regionIndex, RegionIndex);
-        m_style->setRegionIndex(primitiveValue->getIntValue());
+        HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE(regionIndex, RegionIndex);
         return;
     case CSSPropertyWebkitRegionOverflow:
         HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE(regionOverflow, RegionOverflow);
         return;
-
     case CSSPropertyWebkitMarqueeDirection:
         HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE(marqueeDirection, MarqueeDirection)
         return;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to