Remove SVGPaint

Source/core/css/parser/BisonCSSParser-in.cpp

 /*
  * Copyright (C) 2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Nicholas Shanks <webkit@nickshanks.com>
  * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2012 Adobe Systems Incorporated. All rights reserved.
  * Copyright (C) 2012 Intel Corporation. All rights reserved.
  *
@@ skipping 999 matching lines @@
         return false;
 
     CSSPrimitiveValue* primitiveValue = toCSSPrimitiveValue(value);
     if (!primitiveValue->isRGBColor())
         return false;
 
     color = primitiveValue->getRGBA32Value();
     return true;
 }
 
+StyleColor BisonCSSParser::colorFromRGBColorString(const String& colorString)

[fs, 2014/07/01 07:52:12]

I suspect this method is better off being folded into it's callers. Could be
left for later of course.

+{
+    // FIXME: Rework css parser so it is more SVG aware.

[krit, 2014/07/01 06:01:55]

fill and stroke will have the same syntax as background shorthand (more or
less). -webkit-mask and background will accept paint servers/ mask operators as
well in the future. I don't think that this comment is correct. The CSS parser
must be reworked to support SVG resources in general.

+    RGBA32 color;
+    if (BisonCSSParser::parseColor(color, colorString.stripWhiteSpace()))
+        return StyleColor(color);
+    // FIXME: This branch catches the string currentColor, but we should error if we have an illegal color value.
+    return StyleColor::currentColor();
+}
+
 bool BisonCSSParser::parseColor(const String& string)
 {
     setupParser("@-internal-decls color:", string, "");
     cssyyparse(this);
     m_rule = nullptr;
 
     return !m_parsedProperties.isEmpty() && m_parsedProperties.first().id() == CSSPropertyColor;
 }
 
 bool BisonCSSParser::parseSystemColor(RGBA32& color, const String& string)
@@ skipping 1072 matching lines @@
     rule->setProperties(createStylePropertySet());
     clearProperties();
 
     StyleRuleViewport* result = rule.get();
     m_parsedRules.append(rule.release());
 
     return result;
 }
 
 }