Relocate markForLayoutAndParentResourceInvalidation

Source/core/svg/SVGPathElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007 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,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "config.h"
 #include "core/svg/SVGPathElement.h"
 
 #include "core/rendering/svg/RenderSVGPath.h"
-#include "core/rendering/svg/RenderSVGResource.h"
 #include "core/svg/SVGDocumentExtensions.h"
 #include "core/svg/SVGMPathElement.h"
 #include "core/svg/SVGPathSegArcAbs.h"
 #include "core/svg/SVGPathSegArcRel.h"
 #include "core/svg/SVGPathSegClosePath.h"
 #include "core/svg/SVGPathSegCurvetoCubicAbs.h"
 #include "core/svg/SVGPathSegCurvetoCubicRel.h"
 #include "core/svg/SVGPathSegCurvetoCubicSmoothAbs.h"
 #include "core/svg/SVGPathSegCurvetoCubicSmoothRel.h"
 #include "core/svg/SVGPathSegCurvetoQuadraticAbs.h"
@@ skipping 177 matching lines @@
     if (!isSupportedAttribute(attrName)) {
         SVGGeometryElement::svgAttributeChanged(attrName);
         return;
     }
 
     SVGElement::InvalidationGuard invalidationGuard(this);
 
     RenderSVGShape* renderer = toRenderSVGShape(this->renderer());
 
     if (attrName == SVGNames::dAttr) {
-        if (renderer)
+        if (renderer) {
             renderer->setNeedsShapeUpdate();
+            boundsChanged(renderer);
+        }
 
         invalidateMPathDependencies();
+    } else {
+        // 'pathLength' changed.
+        if (renderer)
+            markForLayoutAndParentResourceInvalidation(renderer);
     }
-
-    if (renderer)
-        RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);
 }
 
 void SVGPathElement::invalidateMPathDependencies()
 {
     // <mpath> can only reference <path> but this dependency is not handled in
     // markForLayoutAndParentResourceInvalidation so we update any mpath dependencies manually.
     if (SVGElementSet* dependencies = setOfIncomingReferences()) {
         SVGElementSet::iterator end = dependencies->end();
         for (SVGElementSet::iterator it = dependencies->begin(); it != end; ++it) {
             if (isSVGMPathElement(**it))
@@ skipping 19 matching lines @@
 {
     m_pathSegList->baseValue()->clearByteStream();
 
     invalidateSVGAttributes();
 
     RenderSVGShape* renderer = toRenderSVGShape(this->renderer());
     if (!renderer)
         return;
 
     renderer->setNeedsShapeUpdate();
-    RenderSVGResource::markForLayoutAndParentResourceInvalidation(renderer);
+    boundsChanged(renderer);
 }
 
 FloatRect SVGPathElement::getBBox()
 {
     // By default, getBBox() returns objectBoundingBox but that will include
     // markers so we override it to return just the path's bounding rect.
 
     document().updateLayoutIgnorePendingStylesheets();
 
     // FIXME: Eventually we should support getBBox for detached elements.
     if (!renderer())
         return FloatRect();
 
     RenderSVGShape* renderer = toRenderSVGShape(this->renderer());
     return renderer->path().boundingRect();
 }
 
 } // namespace blink