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

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

Issue 103633002: Add RenderFrameHostDelegate and plumb it through all the necessary classes. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 7 years 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
Index: content/browser/frame_host/render_frame_host_manager_unittest.cc
===================================================================
--- content/browser/frame_host/render_frame_host_manager_unittest.cc (revision 238504)
+++ content/browser/frame_host/render_frame_host_manager_unittest.cc (working copy)
@@ -568,7 +568,7 @@
scoped_ptr<TestWebContents> web_contents(
TestWebContents::Create(browser_context(), instance));
RenderFrameHostManager manager(web_contents.get(), web_contents.get(),
- web_contents.get());
+ web_contents.get(), web_contents.get());
manager.Init(browser_context(), instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
@@ -594,7 +594,7 @@
// Create.
RenderFrameHostManager manager(web_contents.get(), web_contents.get(),
- web_contents.get());
+ web_contents.get(), web_contents.get());
manager.Init(browser_context(), instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
@@ -686,7 +686,7 @@
// Create.
RenderFrameHostManager manager(web_contents.get(), web_contents.get(),
- web_contents.get());
+ web_contents.get(), web_contents.get());
manager.Init(browser_context(), instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
@@ -829,7 +829,7 @@
scoped_ptr<TestWebContents> web_contents(
TestWebContents::Create(browser_context(), instance));
RenderFrameHostManager manager(web_contents.get(), web_contents.get(),
- web_contents.get());
+ web_contents.get(), web_contents.get());
manager.Init(browser_context(), instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
EXPECT_FALSE(manager.current_host()->IsRenderViewLive());
@@ -876,7 +876,7 @@
scoped_ptr<TestWebContents> web_contents1(
TestWebContents::Create(browser_context(), blank_instance));
RenderFrameHostManager manager1(web_contents1.get(), web_contents1.get(),
- web_contents1.get());
+ web_contents1.get(), web_contents1.get());
manager1.Init(
browser_context(), blank_instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
// Test the case that new RVH is considered live.
@@ -906,7 +906,7 @@
scoped_ptr<TestWebContents> web_contents2(
TestWebContents::Create(browser_context(), webui_instance));
RenderFrameHostManager manager2(web_contents2.get(), web_contents2.get(),
- web_contents2.get());
+ web_contents2.get(), web_contents2.get());
manager2.Init(
browser_context(), webui_instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
// Make sure the new RVH is considered live. This is usually done in
@@ -1192,7 +1192,7 @@
// Create.
RenderFrameHostManager manager(web_contents.get(), web_contents.get(),
- web_contents.get());
+ web_contents.get(), web_contents.get());
manager.Init(browser_context(), instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
@@ -1257,7 +1257,7 @@
// Create.
RenderFrameHostManager manager(web_contents.get(), web_contents.get(),
- web_contents.get());
+ web_contents.get(), web_contents.get());
manager.Init(browser_context(), instance, MSG_ROUTING_NONE, MSG_ROUTING_NONE);
« no previous file with comments | « content/browser/frame_host/render_frame_host_manager.cc ('k') | content/browser/frame_host/test_render_frame_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698