Invalidate ancestor resources in when relative lengths change

Source/core/svg/SVGElement.cpp

Index: Source/core/svg/SVGElement.cpp
diff --git a/Source/core/svg/SVGElement.cpp b/Source/core/svg/SVGElement.cpp
index e0a312336479d0331318b43a746cbd414eca1f65..df5e14a9780327da99ab0a62ad4132e7bc51936f 100644
--- a/Source/core/svg/SVGElement.cpp
+++ b/Source/core/svg/SVGElement.cpp
@@ -423,7 +423,7 @@ void SVGElement::invalidateRelativeLengthClients(SubtreeLayoutScope* layoutScope
         if (renderer->isSVGResourceContainer())
             toLayoutSVGResourceContainer(renderer)->invalidateCacheAndMarkForLayout(layoutScope);
         else
-            renderer->setNeedsLayoutAndFullPaintInvalidation(LayoutInvalidationReason::Unknown, MarkContainerChain, layoutScope);

[fs, 2015/04/30 13:36:47]

So Unknown -> SvgResourceInvalidated; bad -> weird. Having
SvgResourceInvalidated where it is is weird too though. I'll look into
refactoring that function a bit to unweird this a bit.

+            markForLayoutAndParentResourceInvalidation(renderer, layoutScope);
     }
 
     for (SVGElement* element : m_elementsWithRelativeLengths) {
@@ -958,10 +958,11 @@ bool SVGElement::hasFocusEventListeners() const
         || hasEventListeners(EventTypeNames::focus) || hasEventListeners(EventTypeNames::blur);
 }
 
-void SVGElement::markForLayoutAndParentResourceInvalidation(LayoutObject* renderer)
+void SVGElement::markForLayoutAndParentResourceInvalidation(LayoutObject* renderer, SubtreeLayoutScope* layoutScope)
 {
     ASSERT(renderer);
-    LayoutSVGResourceContainer::markForLayoutAndParentResourceInvalidation(renderer, true);
+    const bool needsLayout = true;
+    LayoutSVGResourceContainer::markForLayoutAndParentResourceInvalidation(renderer, needsLayout, layoutScope);
 }
 
 void SVGElement::invalidateInstances()