Split out CSSParser public API

Source/core/editing/EditingStyle.cpp

Index: Source/core/editing/EditingStyle.cpp
diff --git a/Source/core/editing/EditingStyle.cpp b/Source/core/editing/EditingStyle.cpp
index 3a708b34f04fb58881d6de46f42a540e6c32fafd..e76081e52424bec3a63cad4830a7ec153a3f34de 100644
--- a/Source/core/editing/EditingStyle.cpp
+++ b/Source/core/editing/EditingStyle.cpp
@@ -30,7 +30,7 @@
 #include "HTMLNames.h"
 #include "bindings/v8/ExceptionStatePlaceholder.h"
 #include "core/css/CSSComputedStyleDeclaration.h"
-#include "core/css/parser/BisonCSSParser.h"
+#include "core/css/parser/CSSParser.h"
 #include "core/css/CSSRuleList.h"
 #include "core/css/CSSStyleRule.h"
 #include "core/css/CSSValueList.h"
@@ -387,7 +387,8 @@ static RGBA32 cssValueToRGBA(CSSValue* colorValue)
         return primitiveColor->getRGBA32Value();
 
     RGBA32 rgba = 0;
-    BisonCSSParser::parseColor(rgba, colorValue->cssText());
+    // FIXME: Why ignore the return value?

[ojan, 2014/02/19 19:52:31]

Maybe it always returns true. Even if so, I think the FIXME is still valid. It
should ASSERT.

+    CSSParser::parseColor(rgba, colorValue->cssText());
     return rgba;
 }