Title: [121192] trunk/Source/WebCore
Revision
121192
Author
[email protected]
Date
2012-06-25 16:15:44 -0700 (Mon, 25 Jun 2012)

Log Message

Fixing compilation failure in StyleResolver.cpp/CSSParser.cpp
https://bugs.webkit.org/show_bug.cgi?id=89892

Patch by Mike West <[email protected]> on 2012-06-25
Reviewed by Alexis Menard.

Adding CSSPropertyVariable to switch statements in CSSParser and
StyleResolver to fix compilation errors under ninja/clang. I've
added both as new cases to the switches, with FIXME comments for
implementation.

* css/CSSParser.cpp:
(WebCore::CSSParser::parseValue):
* css/StyleResolver.cpp:
(WebCore::StyleResolver::collectMatchingRulesForList):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (121191 => 121192)


--- trunk/Source/WebCore/ChangeLog	2012-06-25 23:03:28 UTC (rev 121191)
+++ trunk/Source/WebCore/ChangeLog	2012-06-25 23:15:44 UTC (rev 121192)
@@ -1,3 +1,20 @@
+2012-06-25  Mike West  <[email protected]>
+
+        Fixing compilation failure in StyleResolver.cpp/CSSParser.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=89892
+
+        Reviewed by Alexis Menard.
+
+        Adding CSSPropertyVariable to switch statements in CSSParser and
+        StyleResolver to fix compilation errors under ninja/clang. I've
+        added both as new cases to the switches, with FIXME comments for
+        implementation.
+
+        * css/CSSParser.cpp:
+        (WebCore::CSSParser::parseValue):
+        * css/StyleResolver.cpp:
+        (WebCore::StyleResolver::collectMatchingRulesForList):
+
 2012-06-25  Jay Civelli  <[email protected]>
 
         Fix for a memory leak with MHTMLArchives.

Modified: trunk/Source/WebCore/css/CSSParser.cpp (121191 => 121192)


--- trunk/Source/WebCore/css/CSSParser.cpp	2012-06-25 23:03:28 UTC (rev 121191)
+++ trunk/Source/WebCore/css/CSSParser.cpp	2012-06-25 23:15:44 UTC (rev 121192)
@@ -2663,6 +2663,11 @@
         m_valueList->next();
         break;
 #endif
+#if ENABLE(CSS_VARIABLES)
+    case CSSPropertyVariable:
+        // FIXME: This should have an actual implementation.
+        return false;
+#endif
     case CSSPropertyBorderBottomStyle:
     case CSSPropertyBorderCollapse:
     case CSSPropertyBorderLeftStyle:

Modified: trunk/Source/WebCore/css/StyleResolver.cpp (121191 => 121192)


--- trunk/Source/WebCore/css/StyleResolver.cpp	2012-06-25 23:03:28 UTC (rev 121191)
+++ trunk/Source/WebCore/css/StyleResolver.cpp	2012-06-25 23:15:44 UTC (rev 121192)
@@ -4160,7 +4160,11 @@
         m_style->setGridItemRow(row);
         return;
     }
-
+#if ENABLE(CSS_VARIABLES)
+    case CSSPropertyVariable:
+        // FIXME: This should have an actual implementation.
+        return;
+#endif
     // These properties are implemented in the StyleBuilder lookup table.
     case CSSPropertyBackgroundAttachment:
     case CSSPropertyBackgroundClip:
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to