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

Unified Diff: chrome/browser/ui/webui/sync_setup_handler_unittest.cc

Issue 14141006: [components] Switch {RefCounted}ProfileKeyedService to use BrowserContext (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: for review Created 7 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
Index: chrome/browser/ui/webui/sync_setup_handler_unittest.cc
diff --git a/chrome/browser/ui/webui/sync_setup_handler_unittest.cc b/chrome/browser/ui/webui/sync_setup_handler_unittest.cc
index da0eb5352c1b4afc8b833bc0817c1d340e520217..d31fdcaf97a019d7ddfe055c12fcd146757ffe6d 100644
--- a/chrome/browser/ui/webui/sync_setup_handler_unittest.cc
+++ b/chrome/browser/ui/webui/sync_setup_handler_unittest.cc
@@ -327,8 +327,9 @@ class SigninManagerBaseMock : public FakeSigninManagerBase {
MOCK_CONST_METHOD1(IsAllowedUsername, bool(const std::string& username));
};
-static ProfileKeyedService* BuildSigninManagerBaseMock(Profile* profile) {
- return new SigninManagerBaseMock(profile);
+static ProfileKeyedService* BuildSigninManagerBaseMock(
+ content::BrowserContext* profile) {
+ return new SigninManagerBaseMock(static_cast<Profile*>(profile));
}
// The boolean parameter indicates whether the test is run with ClientOAuth
@@ -684,8 +685,9 @@ class SigninManagerMock : public FakeSigninManager {
};
}
-static ProfileKeyedService* BuildSigninManagerMock(Profile* profile) {
- return new SigninManagerMock(profile);
+static ProfileKeyedService* BuildSigninManagerMock(
+ content::BrowserContext* profile) {
+ return new SigninManagerMock(static_cast<Profile*>(profile));
}
class SyncSetupHandlerNonCrosTest : public SyncSetupHandlerTest {

Powered by Google App Engine
This is Rietveld 408576698