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

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

 /*
  * Copyright (C) 2006 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2007 Rob Buis <buis@kde.org>
  * Copyright (C) 2008 Dirk Schulze <krit@webkit.org>
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 74 matching lines @@
     }
 
     bool applyToFill = mode == ApplyToFillMode;
     SVGPaintType paintType = applyToFill ? svgStyle.fillPaintType() : svgStyle.strokePaintType();
     ASSERT(paintType != SVG_PAINTTYPE_NONE);
 
     Color color;
     bool hasColor = false;
     switch (paintType) {
     case SVG_PAINTTYPE_CURRENTCOLOR:
+    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_CURRENTCOLOR:
     case SVG_PAINTTYPE_URI_RGBCOLOR:
         color = applyToFill ? svgStyle.fillPaintColor() : svgStyle.strokePaintColor();
         hasColor = true;
     default:
         break;
     }
 
     if (style.insideLink() == InsideVisitedLink) {
         // FIXME: This code doesn't support the uri component of the visited link paint, https://bugs.webkit.org/show_bug.cgi?id=70006
         SVGPaintType visitedPaintType = applyToFill ? svgStyle.visitedLinkFillPaintType() : svgStyle.visitedLinkStrokePaintType();
@@ skipping 65 matching lines @@
 LayoutSVGResourcePaintServer::~LayoutSVGResourcePaintServer()
 {
 }
 
 SVGPaintDescription LayoutSVGResourcePaintServer::requestPaintDescription(const LayoutObject& layoutObject, const ComputedStyle& style, LayoutSVGResourceMode resourceMode)
 {
     return requestPaint(layoutObject, style, resourceMode);
 }
 
 }