[SPInvalidation] Update paint properties when SVG resource change

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

Index: third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceContainer.cpp
diff --git a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceContainer.cpp b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceContainer.cpp
index 5d72405397bbbfec2fa59de0ec37940204a2ee26..6bfb64b4d40e99dad8fb6f6fbf6be7633253c801 100644
--- a/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceContainer.cpp
+++ b/third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceContainer.cpp
@@ -178,6 +178,8 @@ void LayoutSVGResourceContainer::markClientForInvalidation(
       // entire <text>/<tspan>/... subtree.
       client->setShouldDoFullPaintInvalidation(
           PaintInvalidationSVGResourceChange);
+      // Invalidate paint properties to update effects if any.

[pdr., 2016/12/16 21:13:37]

This invalidation is very different from the one below. Are both required for
the test to pass?

I think both this change and the ones in PaintLayerResourceInfo make sense, just
want to make sure a new test passes.

[Xianzhu, 2016/12/16 23:00:04]

Added a test for this. However the test doesn't fail in
SlimmingPaintInvalidation mode without this patch because of a bug of
PaintPropertyTreeBuilder that it fails to create an effect node for an SVG
element referencing a filter resource (so FindPaintPropertiesToUpdate doesn't
find the under-invalidation). This doesn't block SlimmingPaintInvalidation
because we can still paint the filter correctly. Will file a bug for this.
 

+      client->setNeedsPaintPropertyUpdate();
       break;
     case ParentOnlyInvalidation:
       break;