Sync the SVGSVGElement interface with the spec

Sync the SVGSVGElement interface with the spec

Add use counters for the bits that have been removed from SVG 2.

The referenceElement arguments are not nullable in the spec, but that
seems to be a spec problem: https://github.com/w3c/svgwg/issues/16

BUG=460722, 313655
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=195466

[philipj_slow, 2015-05-18 13:03:15]

philipj@opera.com changed reviewers:
+ fs@opera.com

[philipj_slow, 2015-05-18 13:03:16]

PTAL

[fs, 2015-05-18 13:18:03]

lgtm w/ tiny request

https://codereview.chromium.org/1140343002/diff/1/Source/core/svg/SVGSVGEleme...
File Source/core/svg/SVGSVGElement.idl (right):

https://codereview.chromium.org/1140343002/diff/1/Source/core/svg/SVGSVGEleme...
Source/core/svg/SVGSVGElement.idl:50: // TODO(philipj): The referenceElement
arguments should not be nullable.
I think this is a spec omission actually (this argument adds an additional
restriction to the result, and null is mentioned in the prose), so might be
worthwhile to add something mentioning that.

[philipj_slow, 2015-05-18 13:33:30]

undo referenceElement

[philipj_slow, 2015-05-18 13:34:49]

https://codereview.chromium.org/1140343002/diff/1/Source/core/svg/SVGSVGEleme...
File Source/core/svg/SVGSVGElement.idl (right):

https://codereview.chromium.org/1140343002/diff/1/Source/core/svg/SVGSVGEleme...
Source/core/svg/SVGSVGElement.idl:50: // TODO(philipj): The referenceElement
arguments should not be nullable.
On 2015/05/18 13:18:03, fs wrote:
> I think this is a spec omission actually (this argument adds an additional
> restriction to the result, and null is mentioned in the prose), so might be
> worthwhile to add something mentioning that.

I reverted this bit and updated the description.

[philipj_slow, 2015-05-18 13:35:16]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-05-18 13:35:17]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/1140343002/#ps20001
(title: "undo referenceElement")

[commit-bot: I haz the power, 2015-05-18 13:35:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1140343002/20001

[fs, 2015-05-18 13:36:20]

On 2015/05/18 13:34:49, philipj_UTC2 wrote:
>
https://codereview.chromium.org/1140343002/diff/1/Source/core/svg/SVGSVGEleme...
> File Source/core/svg/SVGSVGElement.idl (right):
> 
>
https://codereview.chromium.org/1140343002/diff/1/Source/core/svg/SVGSVGEleme...
> Source/core/svg/SVGSVGElement.idl:50: // TODO(philipj): The referenceElement
> arguments should not be nullable.
> On 2015/05/18 13:18:03, fs wrote:
> > I think this is a spec omission actually (this argument adds an additional
> > restriction to the result, and null is mentioned in the prose), so might be
> > worthwhile to add something mentioning that.
> 
> I reverted this bit and updated the description.

I edited typo for you (refernceElement).

[philipj_slow, 2015-05-18 13:38:00]

On 2015/05/18 13:36:20, fs wrote:
> On 2015/05/18 13:34:49, philipj_UTC2 wrote:
> >
>
https://codereview.chromium.org/1140343002/diff/1/Source/core/svg/SVGSVGEleme...
> > File Source/core/svg/SVGSVGElement.idl (right):
> > 
> >
>
https://codereview.chromium.org/1140343002/diff/1/Source/core/svg/SVGSVGEleme...
> > Source/core/svg/SVGSVGElement.idl:50: // TODO(philipj): The referenceElement
> > arguments should not be nullable.
> > On 2015/05/18 13:18:03, fs wrote:
> > > I think this is a spec omission actually (this argument adds an additional
> > > restriction to the result, and null is mentioned in the prose), so might
be
> > > worthwhile to add something mentioning that.
> > 
> > I reverted this bit and updated the description.
> 
> I edited typo for you (refernceElement).

Thanks :)

[commit-bot: I haz the power, 2015-05-18 15:12:52]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=195466