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 3fbca07ed2309b40a5af8f7ae71d9535f2a7811f..a7f1da58fe60553b17da319b59605f37ddc3cf8e 100644
--- a/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
+++ b/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
@@ -146,9 +146,11 @@ TextIteratorAlgorithm<Strategy>::TextIteratorAlgorithm(const PositionTemplate<St
 {
     ASSERT(start.isNotNull());
     ASSERT(end.isNotNull());
-    // Updates layout here since, |Position.compareTo()| and |initialize()|
-    // assume layout tree is up-to-date.
-    start.document()->updateLayoutIgnorePendingStylesheets();
+
+    // 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());
     if (start.compareTo(end) > 0) {
         initialize(end.computeContainerNode(), end.computeOffsetInContainerNode(), start.computeContainerNode(), start.computeOffsetInContainerNode());
         return;
@@ -1090,6 +1092,10 @@ PositionTemplate<Strategy> TextIteratorAlgorithm<Strategy>::endPositionInCurrent
 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)
@@ -1144,6 +1150,10 @@ static String createPlainText(const EphemeralRangeTemplate<Strategy>& range, Tex
     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())