Remove SVGPaint

Source/build/scripts/templates/StyleBuilderFunctions.cpp.tmpl

 {% from 'macros.tmpl' import license %}
 {#
     This file is for property handlers which use the templating engine to
     reduce (handwritten) code duplication.
 
     The `properties' dict can be used to access a property's parameters in
     jinja2 templates (i.e. setter, getter, initial, type_name)
 #}
 #include "config.h"
 #include "StyleBuilderFunctions.h"
@@ skipping 578 matching lines @@
     {{set_value(property)}}(
         svgParentStyle->{{paint_type|lower_first}}Type(),
         svgParentStyle->{{paint_type|lower_first}}Color(),
         svgParentStyle->{{paint_type|lower_first}}Uri(),
         state.applyPropertyToRegularStyle(),
         state.applyPropertyToVisitedLinkStyle());
 }
 
 {{declare_value_function(property_id)}}
 {
-    if (value->isSVGPaint()) {
-        SVGPaint* svgPaint = toSVGPaint(value);
+    String url;
+    if (value->isValueList()) {
+        CSSValueList* list = toCSSValueList(value);
+        if (list->length() < 2)

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

Could you add an assert list->length() > 2? It isn't really a problem if it is
bigger than 2, but it is not expected either.

+            return;
 
-        Color color;
-        if (svgPaint->paintType() == SVGPaint::SVG_PAINTTYPE_CURRENTCOLOR
-                || svgPaint->paintType() == SVGPaint::SVG_PAINTTYPE_URI_CURRENTCOLOR)
-            color = state.style()->color();
-        else
-            color = svgPaint->color();
+        if (!list->item(0)->isPrimitiveValue())
+            return;
 
-        {{set_value(property)}}(svgPaint->paintType(),
-            color,
-            svgPaint->uri(),
+        CSSPrimitiveValue* pValue = toCSSPrimitiveValue(list->item(0));
+        if (!pValue->isURI())
+            return;
+
+        url = pValue->getStringValue();
+        value = list->item(1);
+    }
+    if (value->isPrimitiveValue()) {
+        CSSPrimitiveValue* pValue = toCSSPrimitiveValue(value);
+        Color c;
+        SVGPaintType ptype = SVG_PAINTTYPE_RGBCOLOR;
+        if (pValue->isRGBColor()) {
+            c = pValue->getRGBA32Value();
+            ptype = url.isEmpty() ? SVG_PAINTTYPE_RGBCOLOR : SVG_PAINTTYPE_URI_RGBCOLOR;
+        } else if (pValue->getValueID() == CSSValueCurrentcolor) {
+            c = state.style()->color();
+            ptype = url.isEmpty() ? SVG_PAINTTYPE_CURRENTCOLOR : SVG_PAINTTYPE_URI_CURRENTCOLOR;
+        } else if (pValue->getValueID() == CSSValueNone)
+            ptype = url.isEmpty() ? SVG_PAINTTYPE_NONE : SVG_PAINTTYPE_URI_NONE;
+        else if (pValue->isURI()) {
+            ptype = SVG_PAINTTYPE_URI;
+            url = pValue->getStringValue();
+        }

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

Don't you want to add an else with a return?

+        {{set_value(property)}}(ptype, c, url,
             state.applyPropertyToRegularStyle(),
             state.applyPropertyToVisitedLinkStyle());
     }
 }
 {% endmacro %}
 {{apply_svg_paint('CSSPropertyFill', 'FillPaint')}}
 {{apply_svg_paint('CSSPropertyStroke', 'StrokePaint')}}
 } // namespace WebCore