DOM-based SVG resource tracking

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 32 matching lines @@
 #include "core/frame/csp/ContentSecurityPolicy.h"
 #include "core/html/HTMLElement.h"
 #include "core/inspector/ConsoleMessage.h"
 #include "core/layout/LayoutObject.h"
 #include "core/layout/svg/LayoutSVGResourceContainer.h"
 #include "core/svg/SVGDocumentExtensions.h"
 #include "core/svg/SVGElementRareData.h"
 #include "core/svg/SVGGraphicsElement.h"
 #include "core/svg/SVGSVGElement.h"
 #include "core/svg/SVGTitleElement.h"
-#include "core/svg/SVGTreeScopeResources.h"
 #include "core/svg/SVGUseElement.h"
 #include "core/svg/properties/SVGProperty.h"
 #include "platform/wtf/AutoReset.h"
 #include "platform/wtf/Threading.h"
 
 namespace blink {
 
 using namespace HTMLNames;
 using namespace SVGNames;
 
@@ skipping 39 matching lines @@
 void SVGElement::WillRecalcStyle(StyleRecalcChange change) {
   if (!HasSVGRareData())
     return;
   // 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 > kNoChange || NeedsStyleRecalc())
     SvgRareData()->SetNeedsOverrideComputedStyleUpdate();
 }
 
-void SVGElement::BuildPendingResourcesIfNeeded() {
-  if (!NeedsPendingResourceHandling() || !isConnected() || InUseShadowTree())
-    return;
-  GetTreeScope().EnsureSVGTreeScopedResources().NotifyResourceAvailable(
-      GetIdAttribute());
-}
-
 SVGElementRareData* SVGElement::EnsureSVGRareData() {
   if (HasSVGRareData())
     return SvgRareData();
 
   svg_rare_data_ = new SVGElementRareData(this);
   return svg_rare_data_.Get();
 }
 
 bool SVGElement::IsOutermostSVGSVGElement() const {
   if (!isSVGSVGElement(*this))
@@ skipping 270 matching lines @@
   if (apply_motion_transform == kIncludeMotionTransform && HasSVGRareData())
     matrix.PreMultiply(*SvgRareData()->AnimateMotionTransform());
 
   return matrix;
 }
 
 Node::InsertionNotificationRequest SVGElement::InsertedInto(
     ContainerNode* root_parent) {
   Element::InsertedInto(root_parent);
   UpdateRelativeLengthsInformation();
-  BuildPendingResourcesIfNeeded();
 
   if (hasAttribute(nonceAttr) && getAttribute(nonceAttr) != g_empty_atom) {
     setNonce(getAttribute(nonceAttr));
     if (RuntimeEnabledFeatures::hideNonceContentAttributeEnabled() &&
         InActiveDocument() &&
         GetDocument().GetContentSecurityPolicy()->HasHeaderDeliveredPolicy()) {
       setAttribute(nonceAttr, g_empty_atom);
     }
   }
-
   return kInsertionDone;
 }
 
 void SVGElement::RemovedFrom(ContainerNode* root_parent) {
   bool was_in_document = root_parent->isConnected();
 
   if (was_in_document && HasRelativeLengths()) {
     // The root of the subtree being removed should take itself out from its
     // parent's relative length set. For the other nodes in the subtree we don't
     // need to do anything: they will get their own removedFrom() notification
@@ skipping 533 matching lines @@
     return;
 
   ToSVGElement(parent)->SendSVGLoadEventToSelfAndAncestorChainIfPossible();
 }
 
 void SVGElement::AttributeChanged(const AttributeModificationParams& params) {
   Element::AttributeChanged(params);
 
   if (params.name == HTMLNames::idAttr) {
     RebuildAllIncomingReferences();
-
-    LayoutObject* object = GetLayoutObject();
-    // Notify resources about id changes, this is important as we cache
-    // resources by id in SVGDocumentExtensions
-    if (object && object->IsSVGResourceContainer()) {
-      ToLayoutSVGResourceContainer(object)->IdChanged(params.old_value,
-                                                      params.new_value);
-    }
-    if (isConnected())
-      BuildPendingResourcesIfNeeded();
     InvalidateInstances();
     return;
   }
 
   // Changes to the style attribute are processed lazily (see
   // Element::getAttribute() and related methods), so we don't want changes to
   // the style attribute to result in extra work here.
   if (params.name == HTMLNames::styleAttr)
     return;
 
@@ skipping 356 matching lines @@
   visitor->Trace(class_name_);
   Element::Trace(visitor);
 }
 
 const AtomicString& SVGElement::EventParameterName() {
   DEFINE_STATIC_LOCAL(const AtomicString, evt_string, ("evt"));
   return evt_string;
 }
 
 }  // namespace blink