Remove the SVGColor and SVGPaint DOM interfaces (were deprecated).

Remove the SVGColor and SVGPaint DOM interfaces (were deprecated).
Spec: http://www.w3.org/TR/SVG11/svgdom.html#ExtendedInterfaces
SVG2 spec removal: https://svgwg.org/svg2-draft/changes.html#svgdom

Change test expectations, serialize colors as CSS component values, aka
the same way as the 'color' property. Opera (Presto) and Firefox already
use that way of serializing SVGColor/SVGPaint values.
CSSOM WD spec for serializing <color> values:
http://dev.w3.org/csswg/cssom/#serialize-a-css-component-value

This is to prepare for rewriting the SVG side, removing the respective
classes there too.

BUG=329518
R=pdr, krit, schenney

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=164179

[Stephen Chennney, 2013-12-18 13:53:14]

Just one test that I think needs to have a check removed. Otherwise LGTM.

https://codereview.chromium.org/114373004/diff/20001/LayoutTests/svg/custom/g...
File LayoutTests/svg/custom/global-constructors-expected.txt (right):

https://codereview.chromium.org/114373004/diff/20001/LayoutTests/svg/custom/g...
LayoutTests/svg/custom/global-constructors-expected.txt:22: FAIL
SVGColor.toString() should be function SVGColor() { [native code] }. Threw
exception ReferenceError: SVGColor is not defined
An oversight?

https://codereview.chromium.org/114373004/diff/20001/LayoutTests/svg/custom/g...
LayoutTests/svg/custom/global-constructors-expected.txt:93: FAIL
SVGPaint.toString() should be function SVGPaint() { [native code] }. Threw
exception ReferenceError: SVGPaint is not defined
Ditto.

[Stephen Chennney, 2013-12-18 13:55:48]

You'll need an LGTM from an owner in this set for the bindings change.

Source/bindings/OWNERS

Create a bug if you can, too.

[Erik Dahlström (inactive), 2013-12-18 14:11:58]

https://codereview.chromium.org/114373004/diff/20001/LayoutTests/svg/custom/g...
File LayoutTests/svg/custom/global-constructors-expected.txt (right):

https://codereview.chromium.org/114373004/diff/20001/LayoutTests/svg/custom/g...
LayoutTests/svg/custom/global-constructors-expected.txt:22: FAIL
SVGColor.toString() should be function SVGColor() { [native code] }. Threw
exception ReferenceError: SVGColor is not defined
On 2013/12/18 13:53:15, Stephen Chenney wrote:
> An oversight?

I was considering whether or not to remove these lines. This way there's one
test for each of the interfaces that were removed, but I don't mind really, if
you think it makes more sense to remove them I'll do that.

[haraken, 2013-12-18 15:10:54]

LGTM for bindings/

[Stephen Chennney, 2013-12-18 15:53:04]

Excellent, except the merge gods are against you. Looks like another rebase
before landing.

[Erik Dahlström (inactive), 2013-12-18 15:58:05]

On 2013/12/18 15:53:04, Stephen Chenney wrote:
> Excellent, except the merge gods are against you. Looks like another rebase
> before landing.

Yeah, filed https://codereview.chromium.org/109373004 for that, will rebase as
soon as that lands.

[commit-bot: I haz the power, 2013-12-19 12:47:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ed@opera.com/114373004/110001

[commit-bot: I haz the power, 2013-12-19 14:10:51]

Change committed as 164179