Set NeedsReattachLayoutTree and ChildNeedsReattachLayoutTree flags on Node

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

Index: third_party/WebKit/Source/core/dom/Node.cpp
diff --git a/third_party/WebKit/Source/core/dom/Node.cpp b/third_party/WebKit/Source/core/dom/Node.cpp
index cf0b32333f04e8690a92d6bdf98790d80c79184c..ef877ff0ab359e3f1364473c8d681fb350f0a670 100644
--- a/third_party/WebKit/Source/core/dom/Node.cpp
+++ b/third_party/WebKit/Source/core/dom/Node.cpp
@@ -692,9 +692,18 @@ void Node::markAncestorsWithChildNeedsStyleRecalc()
 {
     for (ContainerNode* p = parentOrShadowHostNode(); p && !p->childNeedsStyleRecalc(); p = p->parentOrShadowHostNode())
         p->setChildNeedsStyleRecalc();
+    // TODO(nainar): Move this to Node::markAncestorsWithChildNeedsReattachLayoutTree()
     document().scheduleLayoutTreeUpdateIfNeeded();
 }
 
+void Node::markNodeAndAncestorsWithChildNeedsReattachLayoutTree()

[Bugs Nash, 2016/09/29 03:15:24]

This name implies that you're marking the current node with ChildNeedsReattach.
I actually think this logic would be better moved to the
setNeedsReattachLayoutTree method, as marking your ancestors with
ChildNeedsReattach comes hand in hand with marking yourself needing reattach.

[nainar, 2016/09/29 04:05:15]

Done.

+{
+    setNeedsReattachLayoutTree();
+    for (ContainerNode* p = parentOrShadowHostNode(); p && !p->childNeedsReattachLayoutTree(); p = p->parentOrShadowHostNode())
+        p->setChildNeedsReattachLayoutTree();
+}
+
+
 void Node::setNeedsStyleRecalc(StyleChangeType changeType, const StyleChangeReasonForTracing& reason)
 {
     DCHECK(changeType != NoStyleChange);