Introduce SVGGraphicsElement IDL interface

Introduce SVGGraphicsElement IDL interface

Introduce SVGGraphicsElement IDL interface as per the latest SVG
specification:
https://svgwg.org/svg2-draft/types.html#InterfaceSVGGraphicsElement

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

[haraken, 2013-06-27 16:04:20]

https://codereview.chromium.org/18053005/diff/1/Source/core/svg/SVGGraphicsEl...
File Source/core/svg/SVGGraphicsElement.idl (right):

https://codereview.chromium.org/18053005/diff/1/Source/core/svg/SVGGraphicsEl...
Source/core/svg/SVGGraphicsElement.idl:40: [RaisesException] SVGMatrix
getTransformToElement([Default=Undefined] optional SVGElement element);

Where were these methods and attributes defined previously? I expected they were
defined in SVGStyledTransformableElement.idl, but there was no such file.

[do-not-use, 2013-06-27 16:05:50]

https://codereview.chromium.org/18053005/diff/1/Source/core/svg/SVGGraphicsEl...
File Source/core/svg/SVGGraphicsElement.idl (right):

https://codereview.chromium.org/18053005/diff/1/Source/core/svg/SVGGraphicsEl...
Source/core/svg/SVGGraphicsElement.idl:40: [RaisesException] SVGMatrix
getTransformToElement([Default=Undefined] optional SVGElement element);
On 2013/06/27 16:04:20, haraken wrote:
> 
> Where were these methods and attributes defined previously? I expected they
were
> defined in SVGStyledTransformableElement.idl, but there was no such file.

They come from SVGLocatable and SVGTransformable interfaces. Sorry, this is only
explained in the bug report.

[haraken, 2013-06-27 16:07:15]

> They come from SVGLocatable and SVGTransformable interfaces. Sorry, this is
only
> explained in the bug report.

Makes sense. LGTM.

[Stephen Chennney, 2013-06-27 16:18:48]

LGTM. It's great that you're doing all this for the project.

https://codereview.chromium.org/18053005/diff/1/Source/core/svg/SVGGraphicsEl...
File Source/core/svg/SVGGraphicsElement.h (right):

https://codereview.chromium.org/18053005/diff/1/Source/core/svg/SVGGraphicsEl...
Source/core/svg/SVGGraphicsElement.h:34: class SVGGraphicsElement : public
SVGStyledLocatableElement, public SVGTransformable, public SVGTests {
You're planning to get rid of the StyledLocatableElement and Transformable,
right? So this is not the final word, but a work in progress?

[do-not-use, 2013-06-27 16:23:19]

On 2013/06/27 16:18:48, Stephen Chenney wrote:
> LGTM. It's great that you're doing all this for the project.
> 
>
https://codereview.chromium.org/18053005/diff/1/Source/core/svg/SVGGraphicsEl...
> File Source/core/svg/SVGGraphicsElement.h (right):
> 
>
https://codereview.chromium.org/18053005/diff/1/Source/core/svg/SVGGraphicsEl...
> Source/core/svg/SVGGraphicsElement.h:34: class SVGGraphicsElement : public
> SVGStyledLocatableElement, public SVGTransformable, public SVGTests {
> You're planning to get rid of the StyledLocatableElement and Transformable,
> right? So this is not the final word, but a work in progress?

Yes, this is a work in progress. I'm trying to do this by chunks.

[commit-bot: I haz the power, 2013-06-27 17:27:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@sisa.samsung.com/18053005/1

[commit-bot: I haz the power, 2013-06-27 19:23:50]

Change committed as 153166