Improve my hasitly added build fix and added a bug https://bugs.webkit.org/show_bug.cgi?id=56288

above addressing this FIXME.

* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::applyProperty):

git-svn-id: svn://svn.chromium.org/blink/trunk@80998 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/WebCore/ChangeLog

@@ -8,6 +8,14 @@
         (WebCore::writeTextRun): If the text box is hyphenated, output the hyphenation
         string.
 
+2011-03-13  David Levin  <levin@chromium.org>
+
+        Improve my hasitly added build fix and added a bug https://bugs.webkit.org/show_bug.cgi?id=56288
+        above addressing this FIXME.
+
+        * css/CSSStyleSelector.cpp:
+        (WebCore::CSSStyleSelector::applyProperty):
+
 2011-03-13  David Levin  <levin@chromium.org>
 
         Build fix adding remaining enum values to switch statement.

third_party/WebKit/Source/WebCore/css/CSSStyleSelector.cpp

@@ -3649,20 +3649,6 @@ void CSSStyleSelector::applyProperty(int id, CSSValue *value)
     // RenderStyle values.  Shorthands (e.g. border, background) occur in this list as well and
     // are only hit when mapping "inherit" or "initial" into front-end values.
     switch (property) {
-    // All properties which are unhandled by the switch are here (included to avoid compile errors about unhandled enum values).
-    case CSSPropertyColor:
-    case CSSPropertyBackgroundColor:
-    case CSSPropertyBorderBottomColor:
-    case CSSPropertyBorderLeftColor:
-    case CSSPropertyBorderRightColor:
-    case CSSPropertyBorderTopColor:
-    case CSSPropertyOutlineColor:
-    case CSSPropertyWebkitColumnRuleColor:
-    case CSSPropertyWebkitTextEmphasisColor:
-    case CSSPropertyWebkitTextFillColor:
-    case CSSPropertyWebkitTextStrokeColor:
-        break;
-
 // ident only properties
     case CSSPropertyBackgroundAttachment:
         HANDLE_BACKGROUND_VALUE(attachment, Attachment, value)
@@ -6193,10 +6179,25 @@ void CSSStyleSelector::applyProperty(int id, CSSValue *value)
         return;
     }
 
+    // FIXME: Move these properties to the appropriate places in this switch statement.
+    case CSSPropertyColor:
+    case CSSPropertyBackgroundColor:
+    case CSSPropertyBorderBottomColor:
+    case CSSPropertyBorderLeftColor:
+    case CSSPropertyBorderRightColor:
+    case CSSPropertyBorderTopColor:
+    case CSSPropertyOutlineColor:
+    case CSSPropertyWebkitColumnRuleColor:
+    case CSSPropertyWebkitTextEmphasisColor:
+    case CSSPropertyWebkitTextFillColor:
+    case CSSPropertyWebkitTextStrokeColor:
+        return;
+
 #if ENABLE(SVG)
     default:
         // Try the SVG properties
         applySVGProperty(id, value);
+        return;
 #endif
     }
 }