Revert of TextIteratorAlgorithm should not force layout update in constructor.

content/renderer/render_view_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/command_line.h"
@@ skipping 1401 matching lines @@
     // Update the status of our IME back-end.
     // TODO(hbono): we should verify messages to be sent from the back-end.
     view()->UpdateTextInputState(ShowIme::HIDE_IME, ChangeSource::FROM_NON_IME);
     ProcessPendingMessages();
     render_thread_->sink().ClearMessages();
 
     if (ime_message->result) {
       // Retrieve the content of this page and compare it with the expected
       // result.
       const int kMaxOutputCharacters = 128;
-      view()->GetWebView()->updateAllLifecyclePhases();
       base::string16 output = WebFrameContentDumper::dumpFrameTreeAsText(
           GetMainFrame(), kMaxOutputCharacters);
       EXPECT_EQ(base::WideToUTF16(ime_message->result), output);
     }
   }
 }
 
 // Test that the RenderView::OnSetTextDirection() function can change the text
 // direction of the selected input element.
 TEST_F(RenderViewImplTest, OnSetTextDirection) {
@@ skipping 30 matching lines @@
         "var result = document.getElementById('result');"
         "var node = document.getElementById('test');"
         "var style = getComputedStyle(node, null);"
         "result.innerText ="
         "    node.getAttribute('dir') + ',' +"
         "    style.getPropertyValue('direction');");
 
     // Copy the document content to std::wstring and compare with the
     // expected result.
     const int kMaxOutputCharacters = 16;
-    view()->GetWebView()->updateAllLifecyclePhases();
     base::string16 output = WebFrameContentDumper::dumpFrameTreeAsText(
         GetMainFrame(), kMaxOutputCharacters);
     EXPECT_EQ(base::WideToUTF16(kTextDirection[i].expected_result), output);
   }
 }
 
 // Crashy, http://crbug.com/53247.
 TEST_F(RenderViewImplTest, DISABLED_DidFailProvisionalLoadWithErrorForError) {
   GetMainFrame()->enableViewSourceMode(true);
   WebURLError error;
@@ skipping 310 matching lines @@
 
   TestRenderFrame* subframe =
       static_cast<TestRenderFrame*>(RenderFrameImpl::FromWebFrame(
           view()->webview()->findFrameByName("frame")));
   subframe->Navigate(common_params, StartNavigationParams(), request_params);
   FrameLoadWaiter(subframe).Wait();
 
   // Copy the document content to std::wstring and compare with the
   // expected result.
   const int kMaxOutputCharacters = 256;
-  view()->GetWebView()->updateAllLifecyclePhases();
   std::string output = base::UTF16ToUTF8(
       base::StringPiece16(WebFrameContentDumper::dumpFrameTreeAsText(
           GetMainFrame(), kMaxOutputCharacters)));
   EXPECT_EQ(output, "hello \n\nworld");
 }
 
 // This test ensures that a RenderFrame object is created for the top level
 // frame in the RenderView.
 TEST_F(RenderViewImplTest, BasicRenderFrame) {
   EXPECT_TRUE(view()->main_render_frame_);
@@ skipping 102 matching lines @@
   common_params.navigation_type = FrameMsg_Navigate_Type::NORMAL;
   common_params.url = GURL("data:text/html,test data");
   TestRenderFrame* main_frame = static_cast<TestRenderFrame*>(frame());
   main_frame->Navigate(common_params, StartNavigationParams(),
                        RequestNavigationParams());
 
   // An error occurred.
   main_frame->didFailProvisionalLoad(web_frame, error,
                                      blink::WebStandardCommit);
   const int kMaxOutputCharacters = 22;
-  view()->GetWebView()->updateAllLifecyclePhases();
   EXPECT_EQ("", base::UTF16ToASCII(base::StringPiece16(
                     WebFrameContentDumper::dumpFrameTreeAsText(
                         web_frame, kMaxOutputCharacters))));
 }
 
 #if defined(OS_ANDROID)
 // Crashing on Android: http://crbug.com/311341
 #define MAYBE_DoesNotSuppress DISABLED_DoesNotSuppress
 #else
 #define MAYBE_DoesNotSuppress DoesNotSuppress
@@ skipping 15 matching lines @@
   main_frame->Navigate(common_params, StartNavigationParams(),
                        RequestNavigationParams());
 
   // An error occurred.
   main_frame->didFailProvisionalLoad(web_frame, error,
                                      blink::WebStandardCommit);
 
   // The error page itself is loaded asynchronously.
   FrameLoadWaiter(main_frame).Wait();
   const int kMaxOutputCharacters = 22;
-  view()->GetWebView()->updateAllLifecyclePhases();
   EXPECT_EQ("A suffusion of yellow.",
             base::UTF16ToASCII(
                 base::StringPiece16(WebFrameContentDumper::dumpFrameTreeAsText(
                     web_frame, kMaxOutputCharacters))));
 }
 
 #if defined(OS_ANDROID)
 // Crashing on Android: http://crbug.com/311341
 #define MAYBE_HttpStatusCodeErrorWithEmptyBody \
   DISABLED_HttpStatusCodeErrorWithEmptyBody
@@ skipping 15 matching lines @@
   main_frame->Navigate(common_params, StartNavigationParams(),
                        RequestNavigationParams());
 
   // Emulate a 4xx/5xx main resource response with an empty body.
   main_frame->didReceiveResponse(1, response);
   main_frame->didFinishDocumentLoad(web_frame, true);
 
   // The error page itself is loaded asynchronously.
   FrameLoadWaiter(main_frame).Wait();
   const int kMaxOutputCharacters = 22;
-  view()->GetWebView()->updateAllLifecyclePhases();
   EXPECT_EQ("A suffusion of yellow.",
             base::UTF16ToASCII(
                 base::StringPiece16(WebFrameContentDumper::dumpFrameTreeAsText(
                     web_frame, kMaxOutputCharacters))));
 }
 
 // Ensure the render view sends favicon url update events correctly.
 TEST_F(RenderViewImplTest, SendFaviconURLUpdateEvent) {
   // An event should be sent when a favicon url exists.
   LoadHTML("<html>"
@@ skipping 470 matching lines @@
       FROM_HERE,
       base::Bind(&DevToolsAgentTest::CloseWhilePaused, base::Unretained(this)));
   ExecuteJavaScriptForTests("debugger;");
 
   // CloseWhilePaused should resume execution and continue here.
   EXPECT_FALSE(IsPaused());
   Detach();
 }
 
 }  // namespace content