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

Unified Diff: ios/chrome/browser/signin/oauth2_token_service_factory.mm

Issue 2685123002: Make OAuth2TokenService and subclasses take delegate by unique_ptr (Closed)
Patch Set: Created 3 years, 10 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: ios/chrome/browser/signin/oauth2_token_service_factory.mm
diff --git a/ios/chrome/browser/signin/oauth2_token_service_factory.mm b/ios/chrome/browser/signin/oauth2_token_service_factory.mm
index cc86bea51d918d1b48dd3c4cd3da42d87e819e8d..fe5f8d6abd72bda0474ad4a216811acf435ed708 100644
--- a/ios/chrome/browser/signin/oauth2_token_service_factory.mm
+++ b/ios/chrome/browser/signin/oauth2_token_service_factory.mm
@@ -51,13 +51,12 @@ OAuth2TokenServiceFactory::BuildServiceInstanceFor(
web::BrowserState* context) const {
ios::ChromeBrowserState* chrome_browser_state =
ios::ChromeBrowserState::FromBrowserState(context);
- ProfileOAuth2TokenServiceIOSDelegate* delegate =
- new ProfileOAuth2TokenServiceIOSDelegate(
- SigninClientFactory::GetForBrowserState(chrome_browser_state),
- base::MakeUnique<ProfileOAuth2TokenServiceIOSProviderImpl>(),
- ios::AccountTrackerServiceFactory::GetForBrowserState(
- chrome_browser_state),
- ios::SigninErrorControllerFactory::GetForBrowserState(
- chrome_browser_state));
- return base::WrapUnique(new ProfileOAuth2TokenService(delegate));
+ auto delegate = base::MakeUnique<ProfileOAuth2TokenServiceIOSDelegate>(
+ SigninClientFactory::GetForBrowserState(chrome_browser_state),
+ base::MakeUnique<ProfileOAuth2TokenServiceIOSProviderImpl>(),
+ ios::AccountTrackerServiceFactory::GetForBrowserState(
+ chrome_browser_state),
+ ios::SigninErrorControllerFactory::GetForBrowserState(
+ chrome_browser_state));
+ return base::MakeUnique<ProfileOAuth2TokenService>(std::move(delegate));
}

Powered by Google App Engine
This is Rietveld 408576698