If add 'id' on pending SVG resource references does not work for non-resources

third_party/WebKit/Source/core/svg/SVGElement.cpp

Index: third_party/WebKit/Source/core/svg/SVGElement.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGElement.cpp b/third_party/WebKit/Source/core/svg/SVGElement.cpp
index 932fc61b351b72c8d8201a7722d290e6bc75fd5b..ffc4f891f7dad39a8e2cde1c2b2b1bb13b511552 100644
--- a/third_party/WebKit/Source/core/svg/SVGElement.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGElement.cpp
@@ -143,10 +143,13 @@ void SVGElement::buildPendingResourcesIfNeeded()
             // FIXME: Ideally we'd always resolve pending resources async instead of inside
             // insertedInto and svgAttributeChanged. For now we only do it for <use> since
             // that would stamp out DOM.
-            if (isSVGUseElement(clientElement))
+            if (isSVGUseElement(clientElement)) {
                 toSVGUseElement(clientElement)->invalidateShadowTree();
-            else
+            } else {
                 clientElement->buildPendingResource();
+                if (LayoutObject* layoutObject = clientElement->layoutObject())
+                    markForLayoutAndParentResourceInvalidation(layoutObject);

[fs, 2016/03/17 17:27:33]

I think we usually have invalidation in the specific buildPendingResource()
method (because their requirements differ etc, so this would invalidate
unnecessarily in some cases.)

[hyunjunekim2, 2016/03/18 07:44:06]

Moved this one to buildPendingResource().

+            }
             extensions.clearHasPendingResourcesIfPossible(clientElement);
         }
     }