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

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

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Alp Toker <alp@atoker.com>
  * Copyright (C) 2009 Cameron McCormack <cam@mcc.id.au>
  *
  * 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
@@ skipping 125 matching lines @@
     // Mark pending resources as pending for removal.
     extensions.markPendingResourcesForRemoval(resourceId);
 
     // Rebuild pending resources for each client of a pending resource that is being removed.
     while (Element* clientElement = extensions.removeElementFromPendingResourcesForRemoval(resourceId)) {
         ASSERT(clientElement->hasPendingResources());
         if (clientElement->hasPendingResources()) {
             // 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);
         }
     }
 }
 
 SVGElementRareData* SVGElement::ensureSVGRareData()
 {
     if (hasSVGRareData())
         return svgRareData();
 
@@ skipping 1061 matching lines @@
     Element::trace(visitor);
 }
 
 const AtomicString& SVGElement::eventParameterName()
 {
     DEFINE_STATIC_LOCAL(const AtomicString, evtString, ("evt", AtomicString::ConstructFromLiteral));
     return evtString;
 }
 
 } // namespace blink