Remove branch on childNeedsDistributionRecalc

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

Index: third_party/WebKit/Source/core/dom/Text.cpp
diff --git a/third_party/WebKit/Source/core/dom/Text.cpp b/third_party/WebKit/Source/core/dom/Text.cpp
index 04845a1644e032a15a674bbc13771da76953f6ad..60c166060d8e7fcd013872fc244893f6f883d7d7 100644
--- a/third_party/WebKit/Source/core/dom/Text.cpp
+++ b/third_party/WebKit/Source/core/dom/Text.cpp
@@ -280,11 +280,6 @@ bool Text::textLayoutObjectIsNeeded(const ComputedStyle& style,
   if (style.preserveNewline())
     return true;
 
-  // childNeedsDistributionRecalc() here is rare, only happens JS calling
-  // surroundContents() etc. from DOMNodeInsertedIntoDocument etc.
-  if (document().childNeedsDistributionRecalc())
-    return true;

[rune, 2017/02/06 22:58:52]

It looks like this could still happen, looking at the code.

updateTextLayoutObject() -> shouldUpdateLayoutByReattaching() ->
textLayoutObjectIsNeeded() in Text.cpp looks dubious because it may be triggered
by setting characterdata from a script at a point where
distribution/style/layout is dirty, and textLayoutObjectIsNeeded() seems to be
expected to be called during layout (re-)attachment.

(disclaimer: haven't tested/debugged to confirm this)

-
   // Avoiding creation of a layoutObject for the text node is a non-essential
   // memory optimization.  So to avoid blowing up on very wide DOMs, we limit
   // the number of siblings to visit.