TabContents -> WebContentsImpl, part 11.

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 "content/browser/child_process_security_policy_impl.h"
 #include "content/browser/renderer_host/test_render_view_host.h"
 #include "content/browser/web_contents/navigation_controller_impl.h"
 #include "content/browser/web_contents/test_web_contents.h"
 #include "content/common/view_messages.h"
 #include "content/public/browser/navigation_entry.h"
@@ skipping 18 matching lines @@
 }
 
 // Create a full screen popup RenderWidgetHost and View.
 TEST_F(RenderViewHostTest, CreateFullscreenWidget) {
   int routing_id = process()->GetNextRoutingID();
   test_rvh()->CreateNewFullscreenWidget(routing_id);
 }
 
 // Makes sure that RenderViewHost::is_waiting_for_unload_ack_ is false when
 // reloading a page. If is_waiting_for_unload_ack_ is not false when reloading
-// the tab may get closed out even though the user pressed the reload button.
+// the contents may get closed out even though the user pressed the reload
+// button.
 TEST_F(RenderViewHostTest, ResetUnloadOnReload) {
   const GURL url1("http://foo1");
   const GURL url2("http://foo2");
 
   // This test is for a subtle timing bug. Here's the sequence that triggered
   // the bug:
   // . go to a page.
   // . go to a new page, preferably one that takes a while to resolve, such
   //   as one on a site that doesn't exist.
   //   . After this step is_waiting_for_unload_ack_ has been set to true on
   //     the first RVH.
   // . click stop before the page has been commited.
   // . click reload.
   //   . is_waiting_for_unload_ack_ is still true, and the if the hang monitor
-  //     fires the tab gets closed.
+  //     fires the contents gets closed.
 
   NavigateAndCommit(url1);
   controller().LoadURL(
       url2, content::Referrer(), content::PAGE_TRANSITION_LINK, std::string());
   // Simulate the ClosePage call which is normally sent by the net::URLRequest.
   rvh()->ClosePage();
   // Needed so that navigations are not suspended on the RVH.
   test_rvh()->SendShouldCloseACK(true);
   contents()->Stop();
   controller().Reload(false);
   EXPECT_FALSE(test_rvh()->is_waiting_for_unload_ack_for_testing());
 }
 
 // Ensure we do not grant bindings to a process shared with unprivileged views.
 TEST_F(RenderViewHostTest, DontGrantBindingsToSharedProcess) {
   // Create another view in the same process.
-  scoped_ptr<TestWebContents> new_tab(
+  scoped_ptr<TestWebContents> new_web_contents(
       new TestWebContents(browser_context(), rvh()->GetSiteInstance()));
 
   rvh()->AllowBindings(content::BINDINGS_POLICY_WEB_UI);
   EXPECT_FALSE(rvh()->GetEnabledBindings() & content::BINDINGS_POLICY_WEB_UI);
 }
 
 class MockDraggingRenderViewHostDelegateView
     : public content::RenderViewHostDelegate::View {
  public:
   virtual ~MockDraggingRenderViewHostDelegateView() {}
@@ skipping 123 matching lines @@
   // the code actually expects it to have at least one int para, this this
   // bogus message will not fail at de-serialization but should fail in
   // OnMsgInputEventAck() processing.
   IPC::Message message(0, ViewHostMsg_HandleInputEvent_ACK::ID,
                        IPC::Message::PRIORITY_NORMAL);
   test_rvh()->OnMessageReceived(message);
   EXPECT_EQ(1, process()->bad_msg_count());
 }
 
 #endif