[SVG] A minor clean up related to event listener attributes

[SVG] A minor clean up related to event listener attributes

onbegin, onend and onrepeat attributes are valid only for animation elements.
So, it is better for them to be in SVGSMILElement instead of SVGElement.

R=pdr@chromium.org, schenney@chromium.org, dschulze@chromium.org, fmalita@chromium.org, rob.buis@samsung.com
BUG=

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

[pavane, 2014-01-02 22:20:43]

Please have a look at this patch.

[rwlbuis, 2014-01-02 22:31:52]

On 2014/01/02 22:20:43, pavane wrote:
> Please have a look at this patch.

Good find! I think a test for this would be nice as well, so we know onbegin
etc. on something other than ‘animate’, ‘animateColor’, ‘animateMotion’,
‘animateTransform’, ‘set’ gets ignored.

[pavane, 2014-01-03 06:42:02]

On 2014/01/02 22:31:52, rwlbuis wrote:
> On 2014/01/02 22:20:43, pavane wrote:
> > Please have a look at this patch.
> 
> Good find! I think a test for this would be nice as well, so we know onbegin
> etc. on something other than ‘animate’, ‘animateColor’, ‘animateMotion’,
> ‘animateTransform’, ‘set’ gets ignored.

Rob, it is already ignored, just that it adds a event listener which is never
listened to. 
Functionality wise it doesn't do much difference. I am afraid there is no way to
test this.
Any thoughts on testing this?

[rwlbuis, 2014-01-03 17:13:38]

On 2014/01/03 06:42:02, pavane wrote:
> On 2014/01/02 22:31:52, rwlbuis wrote:
> > On 2014/01/02 22:20:43, pavane wrote:
> > > Please have a look at this patch.
> > 
> > Good find! I think a test for this would be nice as well, so we know onbegin
> > etc. on something other than ‘animate’, ‘animateColor’, ‘animateMotion’,
> > ‘animateTransform’, ‘set’ gets ignored.
> 
> Rob, it is already ignored, just that it adds a event listener which is never
> listened to. 
> Functionality wise it doesn't do much difference. I am afraid there is no way
to
> test this.
> Any thoughts on testing this?

You are right, I don't see an easy way to test it.

lgtm (I am not a reviewer though).

[pavane, 2014-01-06 20:22:17]

can somebody rubber stamp this patch?

Need a reviewer lgtm

[pdr., 2014-01-06 20:40:51]

On 2014/01/06 20:22:17, pavane wrote:
> can somebody rubber stamp this patch?
> 
> Need a reviewer lgtm

LGTM!

[commit-bot: I haz the power, 2014-01-06 20:53:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pavan.e@samsung.com/106263004/1

[commit-bot: I haz the power, 2014-01-07 00:04:31]

Change committed as 164534