Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(593)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/browser/frame_host/render_frame_host_manager_unittest.cc

Issue 257083002: Check BrowsingInstance before swapping prerenders. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move accounting to RFHM Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« content/browser/browsing_instance.h ('K') | « content/browser/frame_host/render_frame_host_manager.cc ('k') | content/browser/site_instance_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/frame_host/render_frame_host_manager_unittest.cc
diff --git a/content/browser/frame_host/render_frame_host_manager_unittest.cc b/content/browser/frame_host/render_frame_host_manager_unittest.cc
index c0afa6718885eef93b86cdd0888ecee92c5869bc..e0e61b41c0c1a96f4c0032a4ae089de2530150c8 100644
--- a/content/browser/frame_host/render_frame_host_manager_unittest.cc
+++ b/content/browser/frame_host/render_frame_host_manager_unittest.cc
@@ -718,14 +718,8 @@ TEST_F(RenderFrameHostManagerTest, Init) {
scoped_ptr<TestWebContents> web_contents(
TestWebContents::Create(browser_context(), instance));
- FrameTree tree(web_contents->GetFrameTree()->root()->navigator(),
- web_contents.get(), web_contents.get(),
- web_contents.get(), web_contents.get());
- RenderFrameHostManager* manager = tree.root()->render_manager();
-
- manager->Init(browser_context(), instance, MSG_ROUTING_NONE,
- MSG_ROUTING_NONE);
+ RenderFrameHostManager* manager = web_contents->GetRenderManagerForTesting();
RenderViewHostImpl* rvh = manager->current_host();
RenderFrameHostImpl* rfh = manager->current_frame_host();
ASSERT_TRUE(rvh);
@@ -750,15 +744,7 @@ TEST_F(RenderFrameHostManagerTest, Navigate) {
notifications.ListenFor(NOTIFICATION_RENDER_VIEW_HOST_CHANGED,
Source<WebContents>(web_contents.get()));
- // Create.
- FrameTree tree(web_contents->GetFrameTree()->root()->navigator(),
- web_contents.get(), web_contents.get(),
- web_contents.get(), web_contents.get());
- RenderFrameHostManager* manager = tree.root()->render_manager();
-
- manager->Init(browser_context(), instance, MSG_ROUTING_NONE,
- MSG_ROUTING_NONE);
-
+ RenderFrameHostManager* manager = web_contents->GetRenderManagerForTesting();
RenderFrameHostImpl* host;
// 1) The first navigation. --------------------------
@@ -845,14 +831,7 @@ TEST_F(RenderFrameHostManagerTest, NavigateWithEarlyReNavigation) {
notifications.ListenFor(NOTIFICATION_RENDER_VIEW_HOST_CHANGED,
Source<WebContents>(web_contents.get()));
- // Create.
- FrameTree tree(web_contents->GetFrameTree()->root()->navigator(),
- web_contents.get(), web_contents.get(),
- web_contents.get(), web_contents.get());
- RenderFrameHostManager* manager = tree.root()->render_manager();
-
- manager->Init(browser_context(), instance, MSG_ROUTING_NONE,
- MSG_ROUTING_NONE);
+ RenderFrameHostManager* manager = web_contents->GetRenderManagerForTesting();
// 1) The first navigation. --------------------------
const GURL kUrl1("http://www.google.com/");
@@ -1033,13 +1012,8 @@ TEST_F(RenderFrameHostManagerTest, WebUI) {
scoped_ptr<TestWebContents> web_contents(
TestWebContents::Create(browser_context(), instance));
- FrameTree tree(web_contents->GetFrameTree()->root()->navigator(),
- web_contents.get(), web_contents.get(),
- web_contents.get(), web_contents.get());
- RenderFrameHostManager* manager = tree.root()->render_manager();
+ RenderFrameHostManager* manager = web_contents->GetRenderManagerForTesting();
- manager->Init(browser_context(), instance, MSG_ROUTING_NONE,
- MSG_ROUTING_NONE);
EXPECT_FALSE(manager->current_host()->IsRenderViewLive());
const GURL kUrl("chrome://foo");
@@ -1084,12 +1058,8 @@ TEST_F(RenderFrameHostManagerTest, WebUIInNewTab) {
// Create a blank tab.
scoped_ptr<TestWebContents> web_contents1(
TestWebContents::Create(browser_context(), blank_instance));
- FrameTree tree1(web_contents1->GetFrameTree()->root()->navigator(),
- web_contents1.get(), web_contents1.get(),
- web_contents1.get(), web_contents1.get());
- RenderFrameHostManager* manager1 = tree1.root()->render_manager();
- manager1->Init(
- browser_context(), blank_instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
+ RenderFrameHostManager* manager1 =
+ web_contents1->GetRenderManagerForTesting();
// Test the case that new RVH is considered live.
manager1->current_host()->CreateRenderView(base::string16(), -1, -1, false);
@@ -1118,12 +1088,8 @@ TEST_F(RenderFrameHostManagerTest, WebUIInNewTab) {
// Now simulate clicking a link that opens in a new tab.
scoped_ptr<TestWebContents> web_contents2(
TestWebContents::Create(browser_context(), webui_instance));
- FrameTree tree2(web_contents2->GetFrameTree()->root()->navigator(),
- web_contents2.get(), web_contents2.get(),
- web_contents2.get(), web_contents2.get());
- RenderFrameHostManager* manager2 = tree2.root()->render_manager();
- manager2->Init(
- browser_context(), webui_instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
+ RenderFrameHostManager* manager2 =
+ web_contents2->GetRenderManagerForTesting();
// Make sure the new RVH is considered live. This is usually done in
// RenderWidgetHost::Init when opening a new tab from a link.
manager2->current_host()->CreateRenderView(base::string16(), -1, -1, false);
@@ -1412,14 +1378,7 @@ TEST_F(RenderFrameHostManagerTest, NoSwapOnGuestNavigations) {
scoped_ptr<TestWebContents> web_contents(
TestWebContents::Create(browser_context(), instance));
- // Create.
- FrameTree tree(web_contents->GetFrameTree()->root()->navigator(),
- web_contents.get(), web_contents.get(),
- web_contents.get(), web_contents.get());
- RenderFrameHostManager* manager = tree.root()->render_manager();
-
- manager->Init(browser_context(), instance, MSG_ROUTING_NONE,
- MSG_ROUTING_NONE);
+ RenderFrameHostManager* manager = web_contents->GetRenderManagerForTesting();
RenderFrameHostImpl* host;
@@ -1480,14 +1439,7 @@ TEST_F(RenderFrameHostManagerTest, NavigateWithEarlyClose) {
notifications.ListenFor(NOTIFICATION_RENDER_VIEW_HOST_CHANGED,
Source<WebContents>(web_contents.get()));
- // Create.
- FrameTree tree(web_contents->GetFrameTree()->root()->navigator(),
- web_contents.get(), web_contents.get(),
- web_contents.get(), web_contents.get());
- RenderFrameHostManager* manager = tree.root()->render_manager();
-
- manager->Init(browser_context(), instance, MSG_ROUTING_NONE,
- MSG_ROUTING_NONE);
+ RenderFrameHostManager* manager = web_contents->GetRenderManagerForTesting();
// 1) The first navigation. --------------------------
const GURL kUrl1("http://www.google.com/");
« content/browser/browsing_instance.h ('K') | « content/browser/frame_host/render_frame_host_manager.cc ('k') | content/browser/site_instance_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698