Remove SVGElement::rendererIsNeeded

Source/core/svg/SVGTSpanElement.cpp

Index: Source/core/svg/SVGTSpanElement.cpp
diff --git a/Source/core/svg/SVGTSpanElement.cpp b/Source/core/svg/SVGTSpanElement.cpp
index 080ca5a66f7cc85ef4e8de51ea9f38189342e24b..23b8b73fcebb721f51f7d989d53d2ffa873f3ef6 100644
--- a/Source/core/svg/SVGTSpanElement.cpp
+++ b/Source/core/svg/SVGTSpanElement.cpp
@@ -42,15 +42,16 @@ RenderObject* SVGTSpanElement::createRenderer(RenderStyle*)
 
 bool SVGTSpanElement::rendererIsNeeded(const RenderStyle& style)
 {
-    if (parentNode()
-        && (isSVGAElement(*parentNode())
+    if (Node* parent = parentNode()) {
+        if ((isSVGAElement(*parent)
 #if ENABLE(SVG_FONTS)
-            || isSVGAltGlyphElement(*parentNode())
+            || isSVGAltGlyphElement(*parent)
 #endif
-            || isSVGTextElement(*parentNode())
-            || isSVGTextPathElement(*parentNode())
-            || isSVGTSpanElement(*parentNode())))
-        return Element::rendererIsNeeded(style);
+            || isSVGTextElement(*parent)
+            || isSVGTextPathElement(*parent)
+            || isSVGTSpanElement(*parent)))
+        return isValid() && Element::rendererIsNeeded(style);

[pdr., 2014/06/19 21:33:48]

Can you call SVGElement::rendererIsNeeded here? There's some small duplication
of work due to checking for the parent twice, but it'll be harder to screw this
up in a refactor in the future.

+    }
 
     return false;
 }