Move ClosePage() from RenderViewHost to WebContents.

content/browser/renderer_host/render_view_host_unittest.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 "base/path_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "content/browser/child_process_security_policy_impl.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/browser/renderer_host/render_message_filter.h"
 #include "content/browser/renderer_host/render_view_host_delegate_view.h"
@@ skipping 80 matching lines @@
   //   . After this step IsWaitingForUnloadACK returns true on the first RVH.
   // . click stop before the page has been commited.
   // . click reload.
   //   . IsWaitingForUnloadACK still returns true, and if the hang monitor fires
   //     the contents gets closed.
 
   NavigateAndCommit(url1);
   controller().LoadURL(
       url2, Referrer(), ui::PAGE_TRANSITION_LINK, std::string());
   // Simulate the ClosePage call which is normally sent by the net::URLRequest.
-  rvh()->ClosePage();
+  static_cast<RenderViewHostImpl*>(rvh())->ClosePage();

[Charlie Reis, 2015/04/21 17:28:08]

Would test_rvh() work?

[nasko, 2015/04/29 17:09:27]

Done.

   // Needed so that navigations are not suspended on the RFH.
   main_test_rfh()->SendBeforeUnloadACK(true);
   contents()->Stop();
   controller().Reload(false);
   EXPECT_FALSE(main_test_rfh()->IsWaitingForUnloadACK());
 }
 
 // Ensure we do not grant bindings to a process shared with unprivileged views.
 TEST_F(RenderViewHostTest, DontGrantBindingsToSharedProcess) {
   // Create another view in the same process.
@@ skipping 199 matching lines @@
   const std::string data_url = "data:image/gif;base64,"
       "R0lGODlhAQABAIAAAAUEBAAAACwAAAAAAQABAAACAkQBADs=";
 
   tester->Reset();
   tester->Test(data_url);
   EXPECT_EQ(tester->UrlString(), data_url);
   EXPECT_TRUE(tester->IsDownloaded());
 }
 
 }  // namespace content