Revert of TextIteratorAlgorithm should not force layout update in constructor.

third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp

Index: third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
diff --git a/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp b/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
index 7c6df3a1fdbaaae40a98850339ae6b453d95664d..708b95eab321a6b052a4426af7d43d0977e2c802 100644
--- a/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
+++ b/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
@@ -145,11 +145,9 @@
 {
     ASSERT(start.isNotNull());
     ASSERT(end.isNotNull());
-
-    // TODO(dglazkov): TextIterator should not be created for documents that don't have a frame,
-    // but it currently still happens in some cases. See http://crbug.com/591877 for details.
-    ASSERT(!start.document()->view() || !start.document()->view()->needsLayout());
-    ASSERT(!start.document()->needsLayoutTreeUpdate());
+    // Updates layout here since, |Position.compareTo()| and |initialize()|
+    // assume layout tree is up-to-date.
+    start.document()->updateLayoutIgnorePendingStylesheets();
     if (start.compareTo(end) > 0) {
         initialize(end.computeContainerNode(), end.computeOffsetInContainerNode(), start.computeContainerNode(), start.computeOffsetInContainerNode());
         return;
@@ -1091,10 +1089,6 @@
 template<typename Strategy>
 int TextIteratorAlgorithm<Strategy>::rangeLength(const PositionTemplate<Strategy>& start, const PositionTemplate<Strategy>& end, bool forSelectionPreservation)
 {
-    // TODO(dglazkov): The use of updateLayoutIgnorePendingStylesheets needs to be audited.
-    // see http://crbug.com/590369 for more details.
-    start.document()->updateLayoutIgnorePendingStylesheets();
-
     int length = 0;
     TextIteratorBehaviorFlags behaviorFlags = TextIteratorEmitsObjectReplacementCharacter;
     if (forSelectionPreservation)
@@ -1149,10 +1143,6 @@
     if (range.isNull())
         return emptyString();
 
-    // TODO(dglazkov): The use of updateLayoutIgnorePendingStylesheets needs to be audited.
-    // see http://crbug.com/590369 for more details.
-    range.startPosition().document()->updateLayoutIgnorePendingStylesheets();
-
     TextIteratorAlgorithm<Strategy> it(range.startPosition(), range.endPosition(), behavior);
 
     if (it.atEnd())