[SVG2] Make ownerSVGElement and viewportElement nullable.

[SVG2] Make ownerSVGElement and viewportElement nullable.

Spec: https://svgwg.org/svg2-draft/types.html#InterfaceSVGElement

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

[fs, 2014-05-26 13:05:42]

LGTM

(This only codifies current behavior after all - i.e. there should be no change
to the generated bindings.)

https://codereview.chromium.org/303483005/diff/1/Source/core/svg/SVGElement.idl
File Source/core/svg/SVGElement.idl (right):

https://codereview.chromium.org/303483005/diff/1/Source/core/svg/SVGElement.i...
Source/core/svg/SVGElement.idl:28: [TypeChecking=Interface|Nullable] readonly
attribute SVGElement? viewportElement;
Note: This extended attribute will not "do anything" in this case (because of
the readonly - i.e. no setter), but it makes sense to add it nonetheless (IMHO).

[Erik Dahlström (inactive), 2014-05-26 13:13:34]

https://codereview.chromium.org/303483005/diff/1/Source/core/svg/SVGElement.idl
File Source/core/svg/SVGElement.idl (right):

https://codereview.chromium.org/303483005/diff/1/Source/core/svg/SVGElement.i...
Source/core/svg/SVGElement.idl:28: [TypeChecking=Interface|Nullable] readonly
attribute SVGElement? viewportElement;
On 2014/05/26 13:05:42, fs wrote:
> Note: This extended attribute will not "do anything" in this case (because of
> the readonly - i.e. no setter), but it makes sense to add it nonetheless
(IMHO).

I followed the lead from HTMLMediaElement.idl, which had the same. I'll leave it
as is.

[Erik Dahlström (inactive), 2014-05-26 13:13:46]

The CQ bit was checked by ed@opera.com

[commit-bot: I haz the power, 2014-05-26 13:14:06]

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

[commit-bot: I haz the power, 2014-05-26 13:46:04]

Change committed as 174809