Title: [89969] trunk/Source/WebCore
Revision
89969
Author
[email protected]
Date
2011-06-28 15:33:06 -0700 (Tue, 28 Jun 2011)

Log Message

2011-06-28  Luke Macpherson   <[email protected]>

        Reviewed by Eric Seidel.

        Use appropriate macro for CSSPropertyWebkitAppearance in CSSStyleSelector::applyProperty()
        https://bugs.webkit.org/show_bug.cgi?id=63197

        No new tests / no functionality changed.

        * css/CSSStyleSelector.cpp:
        (WebCore::CSSStyleSelector::applyProperty):
        Use HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE macro and delete duplicate code.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (89968 => 89969)


--- trunk/Source/WebCore/ChangeLog	2011-06-28 22:27:15 UTC (rev 89968)
+++ trunk/Source/WebCore/ChangeLog	2011-06-28 22:33:06 UTC (rev 89969)
@@ -1,3 +1,16 @@
+2011-06-28  Luke Macpherson   <[email protected]>
+
+        Reviewed by Eric Seidel.
+
+        Use appropriate macro for CSSPropertyWebkitAppearance in CSSStyleSelector::applyProperty()
+        https://bugs.webkit.org/show_bug.cgi?id=63197
+
+        No new tests / no functionality changed.
+
+        * css/CSSStyleSelector.cpp:
+        (WebCore::CSSStyleSelector::applyProperty):
+        Use HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE macro and delete duplicate code.
+
 2011-06-28  Cary Clark  <[email protected]>
 
         Reviewed by James Robinson.

Modified: trunk/Source/WebCore/css/CSSStyleSelector.cpp (89968 => 89969)


--- trunk/Source/WebCore/css/CSSStyleSelector.cpp	2011-06-28 22:27:15 UTC (rev 89968)
+++ trunk/Source/WebCore/css/CSSStyleSelector.cpp	2011-06-28 22:33:06 UTC (rev 89969)
@@ -4529,14 +4529,9 @@
         return;
 
     // CSS3 Properties
-    case CSSPropertyWebkitAppearance: {
-        HANDLE_INHERIT_AND_INITIAL(appearance, Appearance)
-        if (!primitiveValue)
-            return;
-        m_style->setAppearance(*primitiveValue);
+    case CSSPropertyWebkitAppearance:
+        HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE(appearance, Appearance)
         return;
-    }
-
     case CSSPropertyWebkitBorderImage:
     case CSSPropertyWebkitMaskBoxImage: {
         if (isInherit) {
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to