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())))
-        return Element::rendererIsNeeded(style);
+    if (!parentNode() || !(isSVGAElement(*parentNode()) || isSVGTextElement(*parentNode())))

[esprehn, 2014/06/20 07:58:26]

It shouldn't be possible to get in here without a parentNode().

If you do want to leave the check please break this if into two statements and
propagate the !.

+        return false;
 
-    return false;
+    if (!isValid())
+        return false;
+    return Element::rendererIsNeeded(style);
 }
 
 void SVGTextPathElement::buildPendingResource()
 {
     clearResourceReferences();
     if (!inDocument())
         return;
 
     AtomicString id;
     Element* target = SVGURIReference::targetElementFromIRIString(hrefString(), treeScope(), &id);
@@ skipping 27 matching lines @@
         clearResourceReferences();
 }
 
 bool SVGTextPathElement::selfHasRelativeLengths() const
 {
     return m_startOffset->currentValue()->isRelative()
         || SVGTextContentElement::selfHasRelativeLengths();
 }
 
 }