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

Unified Diff: ios/chrome/browser/signin/fake_oauth2_token_service_builder.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/fake_oauth2_token_service_builder.mm
diff --git a/ios/chrome/browser/signin/fake_oauth2_token_service_builder.mm b/ios/chrome/browser/signin/fake_oauth2_token_service_builder.mm
index 1afa2d622418e2e4daf253a7cbbfa84c34dd82a4..5d5b0e300f1107c837a477ec19c3b4216679d0b5 100644
--- a/ios/chrome/browser/signin/fake_oauth2_token_service_builder.mm
+++ b/ios/chrome/browser/signin/fake_oauth2_token_service_builder.mm
@@ -17,11 +17,11 @@ std::unique_ptr<KeyedService> BuildFakeOAuth2TokenService(
web::BrowserState* context) {
ios::ChromeBrowserState* browser_state =
ios::ChromeBrowserState::FromBrowserState(context);
- OAuth2TokenServiceDelegate* delegate =
- new ProfileOAuth2TokenServiceIOSDelegate(
+ std::unique_ptr<OAuth2TokenServiceDelegate> delegate =
+ base::MakeUnique<ProfileOAuth2TokenServiceIOSDelegate>(
SigninClientFactory::GetForBrowserState(browser_state),
base::MakeUnique<ProfileOAuth2TokenServiceIOSProviderImpl>(),
ios::AccountTrackerServiceFactory::GetForBrowserState(browser_state),
ios::SigninErrorControllerFactory::GetForBrowserState(browser_state));
- return base::MakeUnique<FakeProfileOAuth2TokenService>(delegate);
+ return base::MakeUnique<FakeProfileOAuth2TokenService>(std::move(delegate));
}

Powered by Google App Engine
This is Rietveld 408576698