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

Unified Diff: chrome/browser/renderer_host/test_render_view_host.cc

Issue 18351: Put back render view host tests (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 11 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
Index: chrome/browser/renderer_host/test_render_view_host.cc
===================================================================
--- chrome/browser/renderer_host/test_render_view_host.cc (revision 8255)
+++ chrome/browser/renderer_host/test_render_view_host.cc (working copy)
@@ -1,41 +1,74 @@
-// Copyright (c) 2009 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 "chrome/browser/renderer_host/test_render_view_host.h"
-
-void TestRenderViewHost::SendNavigate(int page_id, const GURL& url) {
+// Copyright (c) 2009 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 "chrome/browser/renderer_host/test_render_view_host.h"
+
+TestRenderViewHost::TestRenderViewHost(SiteInstance* instance,
+ RenderViewHostDelegate* delegate,
+ int routing_id,
+ base::WaitableEvent* modal_dialog_event)
+ : RenderViewHost(instance, delegate, routing_id, modal_dialog_event),
+ immediate_before_unload(false),
+ render_view_created_(false),
+ delete_counter_(NULL) {
+ set_view(new TestRenderWidgetHostView());
+}
+
+TestRenderViewHost::~TestRenderViewHost() {
+ if (delete_counter_)
+ ++*delete_counter_;
+
+ // Since this isn't a traditional view, we have to delete it.
+ delete view_;
+}
+
+bool TestRenderViewHost::CreateRenderView() {
+ DCHECK(!render_view_created_);
+ render_view_created_ = true;
+ return true;
+}
+
+bool TestRenderViewHost::IsRenderViewLive() const {
+ return render_view_created_;
+}
+
+void TestRenderViewHost::TestOnMessageReceived(const IPC::Message& msg) {
+ OnMessageReceived(msg);
+}
+
+void TestRenderViewHost::SendNavigate(int page_id, const GURL& url) {
ViewHostMsg_FrameNavigate_Params params;
params.is_post = false;
params.page_id = page_id;
params.is_content_filtered = false;
- params.url = url;
- params.should_update_history = true;
- params.transition = PageTransition::LINK;
- params.is_post = false;
-
- ViewHostMsg_FrameNavigate msg(1, params);
- OnMsgNavigate(msg);
-}
-
-void RenderViewHostTestHarness::SetUp() {
- // This will be deleted when the WebContents goes away.
- SiteInstance* instance = SiteInstance::CreateSiteInstance(&profile_);
-
- // Make the SiteInstance use our RenderProcessHost as its own.
- process_ = new MockRenderProcessHost(&profile_);
- instance->set_process_host_id(process_->host_id());
-
- contents_ = new WebContents(&profile_, instance, &rvh_factory_, 12, NULL);
- controller_ = new NavigationController(contents_, &profile_);
-}
-
-void RenderViewHostTestHarness::TearDown() {
- contents_->CloseContents();
- contents_ = NULL;
- controller_ = NULL;
-
- // Make sure that we flush any messages related to WebContents destruction
- // before we destroy the profile.
- MessageLoop::current()->RunAllPending();
-}
+ params.url = url;
+ params.should_update_history = true;
+ params.transition = PageTransition::LINK;
+ params.is_post = false;
+
+ ViewHostMsg_FrameNavigate msg(1, params);
+ OnMsgNavigate(msg);
+}
+
+void RenderViewHostTestHarness::SetUp() {
+ // This will be deleted when the WebContents goes away.
+ SiteInstance* instance = SiteInstance::CreateSiteInstance(&profile_);
+
+ // Make the SiteInstance use our RenderProcessHost as its own.
+ process_ = new MockRenderProcessHost(&profile_);
+ instance->set_process_host_id(process_->host_id());
+
+ contents_ = new WebContents(&profile_, instance, &rvh_factory_, 12, NULL);
+ controller_ = new NavigationController(contents_, &profile_);
+}
+
+void RenderViewHostTestHarness::TearDown() {
+ contents_->CloseContents();
+ contents_ = NULL;
+ controller_ = NULL;
+
+ // Make sure that we flush any messages related to WebContents destruction
+ // before we destroy the profile.
+ MessageLoop::current()->RunAllPending();
+}
« no previous file with comments | « chrome/browser/renderer_host/test_render_view_host.h ('k') | chrome/browser/tab_contents/web_contents_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698