Remove SVGElement::rendererIsNeeded

Source/core/svg/SVGTextPathElement.cpp

 /*
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2010 Rob Buis <rwlbuis@gmail.com>
  *
  * 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 117 matching lines @@
         RenderSVGResource::markForLayoutAndParentResourceInvalidation(object);
 }
 
 RenderObject* SVGTextPathElement::createRenderer(RenderStyle*)
 {
     return new RenderSVGTextPath(this);
 }
 
 bool SVGTextPathElement::rendererIsNeeded(const RenderStyle& style)
 {
-    if (parentNode() && (isSVGAElement(*parentNode()) || isSVGTextElement(*parentNode())))
+    if (parentNode() && (isSVGAElement(*parentNode()) || isSVGTextElement(*parentNode())) && isValid())

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

Use separate if statements and early returns.

if (!isValid())
  return false;

etc.

         return Element::rendererIsNeeded(style);
 
     return false;
 }
 
 void SVGTextPathElement::buildPendingResource()
 {
     clearResourceReferences();
     if (!inDocument())
         return;
@@ skipping 30 matching lines @@
         clearResourceReferences();
 }
 
 bool SVGTextPathElement::selfHasRelativeLengths() const
 {
     return m_startOffset->currentValue()->isRelative()
         || SVGTextContentElement::selfHasRelativeLengths();
 }
 
 }