[DRAFT] Create RenderFrameHostManager.

content/browser/site_per_process_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 "base/command_line.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "content/browser/frame_host/frame_tree.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/web_contents/web_contents_impl.h"
@@ skipping 450 matching lines @@
 // FrameTreeShape.
 IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest,
                        FrameTreeShape2) {
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
 
   NavigateToURL(shell(),
                 test_server()->GetURL("files/frame_tree/top.html"));
 
   WebContentsImpl* wc = static_cast<WebContentsImpl*>(shell()->web_contents());
-  RenderViewHostImpl* rvh = static_cast<RenderViewHostImpl*>(
-      wc->GetRenderViewHost());
   FrameTreeNode* root = wc->GetFrameTree()->root();
 
   // Check that the root node is properly created with the frame id of the
   // initial navigation.
   ASSERT_EQ(3UL, root->child_count());
   EXPECT_EQ(std::string(), root->frame_name());
-  EXPECT_EQ(rvh->main_frame_id(), root->frame_id());
 
   ASSERT_EQ(2UL, root->child_at(0)->child_count());
   EXPECT_STREQ("1-1-name", root->child_at(0)->frame_name().c_str());
 
   // Verify the deepest node exists and has the right name.
   ASSERT_EQ(2UL, root->child_at(2)->child_count());
   EXPECT_EQ(1UL, root->child_at(2)->child_at(1)->child_count());
   EXPECT_EQ(0UL, root->child_at(2)->child_at(1)->child_at(0)->child_count());
   EXPECT_STREQ("3-1-id",
       root->child_at(2)->child_at(1)->child_at(0)->frame_name().c_str());
 
   // Navigate to about:blank, which should leave only the root node of the frame
   // tree in the browser process.
   NavigateToURL(shell(), test_server()->GetURL("files/title1.html"));
 
   root = wc->GetFrameTree()->root();
   EXPECT_EQ(0UL, root->child_count());
   EXPECT_EQ(std::string(), root->frame_name());
-  EXPECT_EQ(rvh->main_frame_id(), root->frame_id());
 }
 
 // Test that we can navigate away if the previous renderer doesn't clean up its
 // child frames.
 IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest, FrameTreeAfterCrash) {
   ASSERT_TRUE(test_server()->Start());
   NavigateToURL(shell(),
                 test_server()->GetURL("files/frame_tree/top.html"));
 
   // Crash the renderer so that it doesn't send any FrameDetached messages.
   WindowedNotificationObserver crash_observer(
       NOTIFICATION_RENDERER_PROCESS_CLOSED,
       NotificationService::AllSources());
   NavigateToURL(shell(), GURL(kChromeUICrashURL));
   crash_observer.Wait();
 
   // The frame tree should be cleared, and the frame ID should be reset.
   WebContentsImpl* wc = static_cast<WebContentsImpl*>(shell()->web_contents());
-  RenderViewHostImpl* rvh = static_cast<RenderViewHostImpl*>(
-      wc->GetRenderViewHost());
+  //RenderViewHostImpl* rvh = static_cast<RenderViewHostImpl*>(
+  //    wc->GetRenderViewHost());
   FrameTreeNode* root = wc->GetFrameTree()->root();
   EXPECT_EQ(0UL, root->child_count());
-  EXPECT_EQ(FrameTreeNode::kInvalidFrameId, root->frame_id());
-  EXPECT_EQ(rvh->main_frame_id(), root->frame_id());
+  // TODO(creis): We shouldn't be using frame_id here.
+  /*EXPECT_EQ(FrameTreeNode::kInvalidFrameId, root->frame_id());
+  EXPECT_EQ(rvh->main_frame_id(), root->frame_id()); */
 
   // Navigate to a new URL.
   NavigateToURL(shell(), test_server()->GetURL("files/title1.html"));
 
   // The frame ID should now be set.
   EXPECT_EQ(0UL, root->child_count());
-  EXPECT_NE(FrameTreeNode::kInvalidFrameId, root->frame_id());
-  EXPECT_EQ(rvh->main_frame_id(), root->frame_id());
+  // TODO(creis): We shouldn't be using frame_id here.
+  /*EXPECT_NE(FrameTreeNode::kInvalidFrameId, root->frame_id());
+  EXPECT_EQ(rvh->main_frame_id(), root->frame_id()); */
 }
 
 // Test that we can navigate away if the previous renderer doesn't clean up its
 // child frames.
 IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest, NavigateWithLeftoverFrames) {
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
 
   GURL base_url = test_server()->GetURL("files/site_isolation/");
   GURL::Replacements replace_host;
@@ skipping 14 matching lines @@
   ASSERT_EQ(3UL, root->child_count());
 
   // Navigate to a new URL.  We use LoadURL because NavigateToURL will try to
   // wait for the previous navigation to stop.
   TestNavigationObserver tab_observer(wc, 1);
   shell()->LoadURL(base_url.Resolve("blank.html"));
   tab_observer.Wait();
 
   // The frame tree should now be cleared, and the frame ID should be valid.
   EXPECT_EQ(0UL, root->child_count());
-  EXPECT_NE(FrameTreeNode::kInvalidFrameId, root->frame_id());
+  // TODO(creis): We shouldn't be using frame_id here.
+  //EXPECT_NE(FrameTreeNode::kInvalidFrameId, root->frame_id());
 }
 
 // Tests that the |should_replace_current_entry| flag persists correctly across
 // request transfers that began with a cross-process navigation.
 IN_PROC_BROWSER_TEST_F(SitePerProcessBrowserTest,
                        ReplaceEntryCrossProcessThenTranfers) {
   const NavigationController& controller =
       shell()->web_contents()->GetController();
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
@@ skipping 139 matching lines @@
   // There should be two history entries. url2b should have replaced url1. url2b
   // should not have replaced url3b.
   EXPECT_TRUE(controller.GetPendingEntry() == NULL);
   EXPECT_EQ(2, controller.GetEntryCount());
   EXPECT_EQ(1, controller.GetCurrentEntryIndex());
   EXPECT_EQ(url2b, controller.GetEntryAtIndex(0)->GetURL());
   EXPECT_EQ(url3b, controller.GetEntryAtIndex(1)->GetURL());
 }
 
 }  // namespace content