Introduce toSVGAnimateElement(), and use it

Introduce toSVGAnimateElement(), and use it

As a step to clean-up static_cast<SVGXXX>, static_cast<SVGAnimateElement*> also can
be changed with toSVGAnimateElement().

BUG=N/A

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

[gyuyoung-inactive, 2013-08-18 15:26:13]

Add Kent Tamura as reviewer.

[tkent, 2013-08-18 22:16:48]

> Additionally this cl adds a toStopElement(Node* node) in order to remove a
> 'static_cast<SVGStopElement*>' which wasn't removed.

This part should be done in a separated CL.

[tkent, 2013-08-18 22:18:44]

https://codereview.chromium.org/23279010/diff/1/Source/core/svg/SVGAnimateEle...
File Source/core/svg/SVGAnimateElement.h (right):

https://codereview.chromium.org/23279010/diff/1/Source/core/svg/SVGAnimateEle...
Source/core/svg/SVGAnimateElement.h:79:
ASSERT_WITH_SECURITY_IMPLICATION(!element ||
element->hasTagName(SVGNames::animateTag));
This is wrong.  We can cast elements with animateColorTag, animateTransformTag,
and setTag to SVGAnimateElement too.

[gyuyoung-inactive, 2013-08-18 23:49:54]

On 2013/08/18 22:18:44, tkent wrote:
>
https://codereview.chromium.org/23279010/diff/1/Source/core/svg/SVGAnimateEle...
> File Source/core/svg/SVGAnimateElement.h (right):
> 
>
https://codereview.chromium.org/23279010/diff/1/Source/core/svg/SVGAnimateEle...
> Source/core/svg/SVGAnimateElement.h:79:
> ASSERT_WITH_SECURITY_IMPLICATION(!element ||
> element->hasTagName(SVGNames::animateTag));
> This is wrong.  We can cast elements with animateColorTag,
animateTransformTag,
> and setTag to SVGAnimateElement too.

Done. Thanks.

[gyuyoung-inactive, 2013-08-18 23:50:27]

On 2013/08/18 22:16:48, tkent wrote:
> > Additionally this cl adds a toStopElement(Node* node) in order to remove a
> > 'static_cast<SVGStopElement*>' which wasn't removed.
> 
> This part should be done in a separated CL.

If you think so, ok, no problem. I'm gonna prepare it on new CL.

[tkent, 2013-08-18 23:57:30]

lgtm

[commit-bot: I haz the power, 2013-08-18 23:57:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/23279010/5001

[commit-bot: I haz the power, 2013-08-19 03:41:53]

Retried try job too often on mac_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_blink_...

[commit-bot: I haz the power, 2013-08-19 03:45:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/23279010/5001

[commit-bot: I haz the power, 2013-08-19 04:45:56]

Change committed as 156284