No extra space emission in TextIterator::HandleReplacedElement

third_party/WebKit/Source/web/tests/WebFrameTest.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 9006 matching lines @@
       &client, nullptr, nullptr, remote_frame, WebSandboxFlags::kNone);
   SwapAndVerifyFirstChildConsistency("remote->local", MainFrame(), local_frame);
 
   // FIXME: This almost certainly fires more load events on the iframe element
   // than it should.
   // Finally, make sure an embedder triggered load in the local frame swapped
   // back in works.
   FrameTestHelpers::LoadFrame(local_frame, base_url_ + "subframe-hello.html");
   std::string content =
       WebFrameContentDumper::DumpWebViewAsText(WebView(), 1024).Utf8();
-  EXPECT_EQ("  \n\nhello\n\nb \n\na\n\nc", content);
+  EXPECT_EQ("\n\nhello\n\nb \n\na\n\nc", content);
 
   // Manually reset to break WebViewHelper's dependency on the stack allocated
   // TestWebFrameClient.
   Reset();
   remote_frame->Close();
 }
 
 void WebFrameTest::SwapAndVerifyMiddleChildConsistency(
     const char* const message,
     WebFrame* parent,
@@ skipping 23 matching lines @@
   SwapAndVerifyMiddleChildConsistency("remote->local", MainFrame(),
                                       local_frame);
 
   // FIXME: This almost certainly fires more load events on the iframe element
   // than it should.
   // Finally, make sure an embedder triggered load in the local frame swapped
   // back in works.
   FrameTestHelpers::LoadFrame(local_frame, base_url_ + "subframe-hello.html");
   std::string content =
       WebFrameContentDumper::DumpWebViewAsText(WebView(), 1024).Utf8();
-  EXPECT_EQ("  \n\na\n\nhello\n\nc", content);
+  EXPECT_EQ("\n\na\n\nhello\n\nc", content);
 
   // Manually reset to break WebViewHelper's dependency on the stack allocated
   // TestWebFrameClient.
   Reset();
   remote_frame->Close();
 }
 
 void WebFrameTest::SwapAndVerifyLastChildConsistency(const char* const message,
                                                      WebFrame* parent,
                                                      WebFrame* new_child) {
@@ skipping 17 matching lines @@
       &client, nullptr, nullptr, remote_frame, WebSandboxFlags::kNone);
   SwapAndVerifyLastChildConsistency("remote->local", MainFrame(), local_frame);
 
   // FIXME: This almost certainly fires more load events on the iframe element
   // than it should.
   // Finally, make sure an embedder triggered load in the local frame swapped
   // back in works.
   FrameTestHelpers::LoadFrame(local_frame, base_url_ + "subframe-hello.html");
   std::string content =
       WebFrameContentDumper::DumpWebViewAsText(WebView(), 1024).Utf8();
-  EXPECT_EQ("  \n\na\n\nb \n\na\n\nhello", content);
+  EXPECT_EQ("\n\na\n\nb \n\na\n\nhello", content);
 
   // Manually reset to break WebViewHelper's dependency on the stack allocated
   // TestWebFrameClient.
   Reset();
   remote_frame->Close();
 }
 
 TEST_F(WebFrameSwapTest, DetachProvisionalFrame) {
   FrameTestHelpers::TestWebRemoteFrameClient remote_frame_client;
   WebRemoteFrameImpl* remote_frame = WebRemoteFrameImpl::Create(
@@ skipping 59 matching lines @@
       &client, nullptr, nullptr, remote_frame, WebSandboxFlags::kNone);
   SwapAndVerifySubframeConsistency("remote->local", target_frame, local_frame);
 
   // FIXME: This almost certainly fires more load events on the iframe element
   // than it should.
   // Finally, make sure an embedder triggered load in the local frame swapped
   // back in works.
   FrameTestHelpers::LoadFrame(local_frame, base_url_ + "subframe-hello.html");
   std::string content =
       WebFrameContentDumper::DumpWebViewAsText(WebView(), 1024).Utf8();
-  EXPECT_EQ("  \n\na\n\nhello\n\nc", content);
+  EXPECT_EQ("\n\na\n\nhello\n\nc", content);
 
   // Manually reset to break WebViewHelper's dependency on the stack allocated
   // TestWebFrameClient.
   Reset();
   remote_frame->Close();
   child_remote_frame->Close();
 }
 
 TEST_F(WebFrameSwapTest, SwapPreservesGlobalContext) {
   v8::HandleScope scope(v8::Isolate::GetCurrent());
@@ skipping 2947 matching lines @@
 
   // Failing the original child frame navigation and trying to render fallback
   // content shouldn't crash. It should return NoLoadInProgress. This is so the
   // caller won't attempt to replace the correctly empty frame with an error
   // page.
   EXPECT_EQ(WebLocalFrame::NoLoadInProgress,
             child->MaybeRenderFallbackContent(WebURLError()));
 }
 
 }  // namespace blink