Check subtree paint invalidation when overflow clip status changes

third_party/WebKit/Source/core/layout/svg/LayoutSVGInline.cpp

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGInline.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGInline.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGInline.cpp
index b36153698b96fbfde3fcff2541428f74de08e66e..ae87f0643ee27427cc3c2ce39cb393bc8e1fea19 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGInline.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGInline.cpp
@@ -143,19 +143,4 @@ void LayoutSVGInline::removeChild(LayoutObject* child)
     LayoutInline::removeChild(child);
 }
 
-void LayoutSVGInline::invalidateTreeIfNeeded(const PaintInvalidationState& paintInvalidationState)

[chrishtr, 2016/05/23 20:10:32]

This looks different than LayoutBoxModelObject::invalidateTreeIfNeeded, which it
inherits from.

[Xianzhu, 2016/05/23 21:35:49]

I noticed this but thought LayoutBoxModelObject::invalidateTreeIfNeeded() is
also applicable to LayoutSVGInline. However, I defer this change for more
verifications. In the new patch this method calls
LayoutObject::invalidateTreeIfNeeded().

-{
-    ASSERT(!needsLayout());
-
-    if (!shouldCheckForPaintInvalidation(paintInvalidationState))
-        return;
-
-    PaintInvalidationState newPaintInvalidationState(paintInvalidationState, *this);
-    PaintInvalidationReason reason = invalidatePaintIfNeeded(newPaintInvalidationState);
-    clearPaintInvalidationFlags(newPaintInvalidationState);
-
-    newPaintInvalidationState.updateForChildren(reason);
-    invalidatePaintOfSubtreesIfNeeded(newPaintInvalidationState);
-}
-
 } // namespace blink