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 a7f1da58fe60553b17da319b59605f37ddc3cf8e..3fbca07ed2309b40a5af8f7ae71d9535f2a7811f 100644
--- a/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
+++ b/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
@@ -146,11 +146,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;
@@ -1092,10 +1090,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)
@@ -1150,10 +1144,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())