At the moment we check every string for currentColor, independent of the animation type. We should just check for currentColor on color animation. Also if the string is currentColor, we ask the RenderStyle for the color, transform this color to a string and transform it back again.
Created attachment 82047 [details] Patch
Comment on attachment 82047 [details] Patch Nice optimization. r=me In CSS and <canvas> context, "currentColor" is case-insensitive, I don't know about SVG though. Not a regression in your patch, but something to look at afterwards perhaps.
(In reply to comment #2) > (From update of attachment 82047 [details]) > Nice optimization. r=me > > In CSS and <canvas> context, "currentColor" is case-insensitive, I don't know about SVG though. Not a regression in your patch, but something to look at afterwards perhaps. Checked this right after talking with Andreas. Opera uses case-sensitive, Firefox case-insensitive. Have to take a look into the spec. Thanks for bringing this up.
Committed r78271: <http://trac.webkit.org/changeset/78271>