Fix animation of 'color' w/ currentColor for SVG 'fill' and 'stroke'

third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePaintServer.cpp

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePaintServer.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePaintServer.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePaintServer.cpp
index e24a0d299506f74850f56787a3991ec2ef0e52b1..8df6717abecf352ce98915d1b74615ef80e56930 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePaintServer.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourcePaintServer.cpp
@@ -92,8 +92,13 @@ static SVGPaintDescription requestPaint(const LayoutObject& object, const Comput
     bool hasColor = false;
     switch (paintType) {
     case SVG_PAINTTYPE_CURRENTCOLOR:
-    case SVG_PAINTTYPE_RGBCOLOR:
     case SVG_PAINTTYPE_URI_CURRENTCOLOR:
+        color = style.visitedDependentColor(CSSPropertyColor);
+        if (!color.alpha())

[fs, 2015/11/25 17:23:00]

I don't see why you need this.

[hyunjunekim2, 2015/11/26 01:07:26]

And If don't has color(the color wasn't applied), just return svgStyle.{fill or
stroke}.
but has this one, return style.color().

[fs, 2015/11/26 11:30:34]

I suspect this is just a side-effect of the change in
applyValueCSSPropertyColor. Regardless though this only checks if the
alpha-channel of the color is zero - not whether it was applied or not (that's
just a lucky coincidence at best.)

+            color = applyToFill ? svgStyle.fillPaintColor() : svgStyle.strokePaintColor();
+        hasColor = true;
+        break;
+    case SVG_PAINTTYPE_RGBCOLOR:
     case SVG_PAINTTYPE_URI_RGBCOLOR:
         color = applyToFill ? svgStyle.fillPaintColor() : svgStyle.strokePaintColor();
         hasColor = true;