Handle "url(...) none" and "url(...) currentColor" SVG paint specs

Handle "url(...) none" and "url(...) currentColor" SVG paint specs

Consider all paint types with a URI in paintingResourceFromSVGPaint -
URI_NONE, URI_CURRENTCOLOR and URI_RGBCOLOR_ICCCOLOR were previously
not considered. (The last one, URI_RGBCOLOR_ICCCOLOR, will never appear
out of the parser.)
Add parsing support for the URI_CURRENTCOLOR type, since it's trivial.

TEST=svg/custom/fill-fallback-{none,currentcolor-{1,2}}.svg
BUG=162547
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=164200

[Stephen Chennney, 2013-12-19 16:41:54]

What is the expected behavior when the fallback is "none" and the URL is not
found?

Otherwise the code is good.

[fs, 2013-12-19 17:15:31]

On 2013/12/19 16:41:54, Stephen Chenney wrote:
> What is the expected behavior when the fallback is "none" and the URL is not
> found?

Then the fill used will be 'none', i.e, no fill will be applied. I suppose I
could add a TC for that case as well.

[fs, 2013-12-19 17:26:31]

On 2013/12/19 17:15:31, fs wrote:
> On 2013/12/19 16:41:54, Stephen Chenney wrote:
> > What is the expected behavior when the fallback is "none" and the URL is not
> > found?
> 
> Then the fill used will be 'none', i.e, no fill will be applied. I suppose I
> could add a TC for that case as well.

Couldn't think of ways to make TCs for all the failure modes, but will upload
something at least...

[Stephen Chennney, 2013-12-19 21:48:23]

LGTM. The total failure case is still worth it for crash checking etc. as the
code changes in the future.

[commit-bot: I haz the power, 2013-12-19 21:48:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fs@opera.com/108303009/20001

[commit-bot: I haz the power, 2013-12-19 21:57:25]

Change committed as 164200