Ensure clean layout for VisibleSelection::selectionFromContentsOfNode

third_party/WebKit/Source/core/editing/VisibleSelection.cpp

Index: third_party/WebKit/Source/core/editing/VisibleSelection.cpp
diff --git a/third_party/WebKit/Source/core/editing/VisibleSelection.cpp b/third_party/WebKit/Source/core/editing/VisibleSelection.cpp
index d8d2debe974a38b3a79b9bfe1fdbea8cfccb0f0c..dec9069a6b27e3deba2082ea9774ff3528bfe98f 100644
--- a/third_party/WebKit/Source/core/editing/VisibleSelection.cpp
+++ b/third_party/WebKit/Source/core/editing/VisibleSelection.cpp
@@ -248,10 +248,6 @@ VisibleSelectionTemplate<Strategy>
 VisibleSelectionTemplate<Strategy>::selectionFromContentsOfNode(Node* node) {
   DCHECK(!Strategy::editingIgnoresContent(node));
 
-  // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets

[tkent, 2016/10/17 07:08:16]

We should add a DCHECK and/or a comment to ensure clean layout.

[Xiaocheng, 2016/10/17 07:36:21]

Done.

-  // needs to be audited. see http://crbug.com/590369 for more details.
-  node->document().updateStyleAndLayoutIgnorePendingStylesheets();
-
   typename SelectionTemplate<Strategy>::Builder builder;
   builder.collapse(PositionTemplate<Strategy>::firstPositionInNode(node))
       .extend(PositionTemplate<Strategy>::lastPositionInNode(node));