Remove SVGElement::rendererIsNeeded

Source/core/svg/SVGTSpanElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2010 Rob Buis <buis@kde.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 24 matching lines @@
 
 DEFINE_NODE_FACTORY(SVGTSpanElement)
 
 RenderObject* SVGTSpanElement::createRenderer(RenderStyle*)
 {
     return new RenderSVGTSpan(this);
 }
 
 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())))
+            || isSVGTextElement(*parent)
+            || isSVGTextPathElement(*parent)
+            || isSVGTSpanElement(*parent)) && isValid())
         return Element::rendererIsNeeded(style);

[esprehn, 2014/06/17 23:09:40]

Remove all the extra parens, instead do:

return isValid() && Element::rendererIsNeeded(style);

+    }
 
     return false;
 }
 
 }