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

Unified Diff: ui/views/test/webview_test_helper.cc

Issue 10510005: Hide TestRenderViewHostFactory from embedders since it depends on internal content classes. Expose … (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 7 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: ui/views/test/webview_test_helper.cc
===================================================================
--- ui/views/test/webview_test_helper.cc (revision 140261)
+++ ui/views/test/webview_test_helper.cc (working copy)
@@ -5,10 +5,8 @@
#include "ui/views/test/webview_test_helper.h"
#include "base/message_loop.h"
-#include "content/public/test/mock_render_process_host.h"
#include "content/public/test/test_browser_thread.h"
#include "content/public/test/test_content_client_initializer.h"
-#include "content/test/test_render_view_host_factory.h"
#include "ui/views/controls/webview/webview.h"
namespace views {
@@ -20,9 +18,7 @@
// Setup to register a new RenderViewHost factory which manufactures
// mock render process hosts. This ensures that we never create a 'real'
// render view host since support for it doesn't exist in unit tests.
- rph_factory_.reset(new content::MockRenderProcessHostFactory());
- rvh_factory_.reset(
- new content::TestRenderViewHostFactory(rph_factory_.get()));
+ test_content_client_initializer_->CreateTestRenderViewHosts();
ui_thread_.reset(
new content::TestBrowserThread(content::BrowserThread::UI, ui_loop));
« content/public/test/test_content_client_initializer.h ('K') | « ui/views/test/webview_test_helper.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698