Remove toSVGStopElement(SVGElement*) because it can be covered by toSVGStopElement(Node*)

Source/core/rendering/svg/SVGRenderTreeAsText.cpp

Index: Source/core/rendering/svg/SVGRenderTreeAsText.cpp
diff --git a/Source/core/rendering/svg/SVGRenderTreeAsText.cpp b/Source/core/rendering/svg/SVGRenderTreeAsText.cpp
index 1eaf384472f5754eba421ce097a5e9e6bed37e4b..4db6b9c551e9c141f744ed90cb2658d0cf964499 100644
--- a/Source/core/rendering/svg/SVGRenderTreeAsText.cpp
+++ b/Source/core/rendering/svg/SVGRenderTreeAsText.cpp
@@ -625,7 +625,7 @@ void writeSVGGradientStop(TextStream& ts, const RenderSVGGradientStop& stop, int
 {
     writeStandardPrefix(ts, stop, indent);
 
-    SVGStopElement* stopElement = toSVGStopElement(stop.node());
+    SVGStopElement* stopElement = toSVGStopElement(toSVGElement(stop.node()));

[tkent, 2013/08/19 22:57:12]

This looks worse.  I don't think we need to remove toSVGStopElement(Node*).

[gyuyoung-inactive, 2013/08/20 05:44:54]

Do you have any reason ? But, I think this patch is trivial. So, if you think we
can keep toSVGStopElement(Node*), I'm willing to close this cl.

[tkent, 2013/08/20 06:09:33]

When one wants to cast a Node to SVGStopElement, he needs to write
toSVGStopElement(toSVGElement(node)).  It's silly.

If you don't want to keep two toSVGStopElement() functions, you should remove
toSVGStopElement(SVGElement*) because toSVGStopElement(Node*) covers much.

     ASSERT(stopElement);
 
     RenderStyle* style = stop.style();