Reland of "Tracking reference filter mutation via SVGElementProxy"

third_party/WebKit/Source/core/dom/Element.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Peter Kelly (pmk@post.com)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2007 David Smith (catfish.man@gmail.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc.
  * All rights reserved.
  *           (C) 2007 Eric Seidel (eric@webkit.org)
  *
@@ skipping 1730 matching lines @@
     if (isHovered())
       document().hoveredNodeDetached(*this);
     if (inActiveChain())
       document().activeChainNodeDetached(*this);
     document().userActionElements().didDetach(*this);
   }
 
   if (context.clearInvalidation)
     document().styleEngine().styleInvalidator().clearInvalidation(*this);
 
-  if (svgFilterNeedsLayerUpdate())
-    document().unscheduleSVGFilterLayerUpdateHack(*this);
-
   setNeedsResizeObserverUpdate();
 
   DCHECK(needsAttach());
 }
 
 bool Element::pseudoStyleCacheIsInvalid(const ComputedStyle* currentStyle,
                                         ComputedStyle* newStyle) {
   DCHECK_EQ(currentStyle, computedStyle());
   DCHECK(layoutObject());
 
@@ skipping 184 matching lines @@
     return rebuildLayoutTree();
   }
 
   DCHECK(oldStyle);
 
   if (localChange != NoChange)
     updateCallbackSelectors(oldStyle.get(), newStyle.get());
 
   if (LayoutObject* layoutObject = this->layoutObject()) {
     if (localChange != NoChange ||
-        pseudoStyleCacheIsInvalid(oldStyle.get(), newStyle.get()) ||
-        svgFilterNeedsLayerUpdate()) {
+        pseudoStyleCacheIsInvalid(oldStyle.get(), newStyle.get())) {
       layoutObject->setStyle(newStyle.get());
     } else {
       // Although no change occurred, we use the new style so that the cousin
       // style sharing code won't get fooled into believing this style is the
       // same.
       // FIXME: We may be able to remove this hack, see discussion in
       // https://codereview.chromium.org/30453002/
       layoutObject->setStyleInternal(newStyle.get());
     }
   }
@@ skipping 1642 matching lines @@
   if (!document().isHTMLDocument())
     return;
 
   if (!oldId.isEmpty())
     toHTMLDocument(document()).removeExtraNamedItem(oldId);
 
   if (!newId.isEmpty())
     toHTMLDocument(document()).addExtraNamedItem(newId);
 }
 
-void Element::scheduleSVGFilterLayerUpdateHack() {
-  document().scheduleSVGFilterLayerUpdateHack(*this);
-}
-
 ScrollOffset Element::savedLayerScrollOffset() const {
   return hasRareData() ? elementRareData()->savedLayerScrollOffset()
                        : ScrollOffset();
 }
 
 void Element::setSavedLayerScrollOffset(const ScrollOffset& size) {
   if (size.isZero() && !hasRareData())
     return;
   ensureElementRareData().setSavedLayerScrollOffset(size);
 }
@@ skipping 454 matching lines @@
 }
 
 DEFINE_TRACE_WRAPPERS(Element) {
   if (hasRareData()) {
     visitor->traceWrappers(elementRareData());
   }
   ContainerNode::traceWrappers(visitor);
 }
 
 }  // namespace blink