removeBetween() -> detach() performance fix.

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)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 1512 matching lines @@
     ContainerNode::removedFrom(insertionPoint);
     if (wasInDocument) {
         if (this == document().cssTarget())
             document().setCSSTarget(nullptr);
 
         if (hasPendingResources())
             document().accessSVGExtensions().removeElementFromPendingResources(this);
 
         if (isUpgradedCustomElement())
             CustomElement::didDetach(this, insertionPoint->document());
+
+        document().styleEngine().styleInvalidator().clearInvalidation(*this);

[esprehn, 2016/01/15 17:07:24]

Can we do a bit check here to avoid this long chain of function calls?

[rune, 2016/01/15 17:22:21]

Done.

     }
 
     document().removeFromTopLayer(this);
 
     clearElementFlag(IsInCanvasSubtree);
 
     if (hasRareData()) {
         ElementRareData* data = elementRareData();
 
         data->clearRestyleFlags();
@@ skipping 2103 matching lines @@
 {
 #if ENABLE(OILPAN)
     if (hasRareData())
         visitor->trace(elementRareData());
     visitor->trace(m_elementData);
 #endif
     ContainerNode::trace(visitor);
 }
 
 } // namespace blink