Move SVGElement::buildPendingResourcesIfNeeded to SVGTreeScopeResources

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
@@ skipping 93 matching lines @@
   // If the style changes because of a regular property change (not induced by
   // SMIL animations themselves) reset the "computed style without SMIL style
   // properties", so the base value change gets reflected.
   if (change > NoChange || needsStyleRecalc())
     svgRareData()->setNeedsOverrideComputedStyleUpdate();
 }
 
 void SVGElement::buildPendingResourcesIfNeeded() {
   if (!needsPendingResourceHandling() || !isConnected() || inUseShadowTree())
     return;
-
-  SVGTreeScopeResources& treeScopeResources =
-      treeScope().ensureSVGTreeScopedResources();
-  AtomicString resourceId = getIdAttribute();
-  if (!treeScopeResources.hasPendingResource(resourceId))
-    return;
-  // Guaranteed by hasPendingResource.
-  DCHECK(!resourceId.isEmpty());
-
-  // Get pending elements for this id.
-  SVGTreeScopeResources::SVGPendingElements* pendingElements =
-      treeScopeResources.removePendingResource(resourceId);
-  if (!pendingElements || pendingElements->isEmpty())
-    return;
-
-  // Rebuild pending resources for each client of a pending resource that is
-  // being removed.
-  for (Element* clientElement : *pendingElements) {
-    DCHECK(clientElement->hasPendingResources());
-    if (!clientElement->hasPendingResources())
-      continue;
-    // TODO(fs): 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))
-      toSVGUseElement(clientElement)->invalidateShadowTree();
-    else
-      clientElement->buildPendingResource();
-    treeScopeResources.clearHasPendingResourcesIfPossible(clientElement);
-  }
+  treeScope().ensureSVGTreeScopedResources().notifyResourceAvailable(
+      getIdAttribute());
 }
 
 SVGElementRareData* SVGElement::ensureSVGRareData() {
   if (hasSVGRareData())
     return svgRareData();
 
   m_SVGRareData = new SVGElementRareData(this);
   return m_SVGRareData.get();
 }
 
@@ skipping 1148 matching lines @@
   visitor->trace(m_className);
   Element::trace(visitor);
 }
 
 const AtomicString& SVGElement::eventParameterName() {
   DEFINE_STATIC_LOCAL(const AtomicString, evtString, ("evt"));
   return evtString;
 }
 
 }  // namespace blink