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

Unified Diff: chrome/browser/geolocation/chrome_geolocation_permission_context_unittest.cc

Issue 9645003: Introduce abstractions to allow embedders to test RenderViewHost (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge to head. Created 8 years, 9 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/geolocation/chrome_geolocation_permission_context_unittest.cc
diff --git a/chrome/browser/geolocation/chrome_geolocation_permission_context_unittest.cc b/chrome/browser/geolocation/chrome_geolocation_permission_context_unittest.cc
index f4bad4759fb95babbacaf36057204534e4c29145..c1ca818719c91248127850085093356edaae703e 100644
--- a/chrome/browser/geolocation/chrome_geolocation_permission_context_unittest.cc
+++ b/chrome/browser/geolocation/chrome_geolocation_permission_context_unittest.cc
@@ -26,9 +26,11 @@
#include "content/public/browser/notification_service.h"
#include "content/test/mock_geolocation.h"
#include "content/test/test_browser_thread.h"
+#include "content/test/test_renderer_host.h"
#include "testing/gtest/include/gtest/gtest.h"
using content::BrowserThread;
+using content::RenderViewHostTester;
using content::WebContents;
// ClosedDelegateTracker ------------------------------------------------------
@@ -207,7 +209,7 @@ void GeolocationPermissionContextTests::AddNewTab(const GURL& url) {
WebContents::Create(profile(), NULL, MSG_ROUTING_NONE, NULL, NULL);
new_tab->GetController().LoadURL(
url, content::Referrer(), content::PAGE_TRANSITION_TYPED, std::string());
- static_cast<content::TestRenderViewHost*>(new_tab->GetRenderViewHost())->
+ RenderViewHostTester::For(new_tab->GetRenderViewHost())->
SendNavigate(extra_tabs_.size() + 1, url);
extra_tabs_.push_back(new TabContentsWrapper(new_tab));
}

Powered by Google App Engine
This is Rietveld 408576698