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

Unified Diff: chrome/browser/ui/webui/signin/inline_login_handler_impl.cc

Issue 964563002: Replace SetAuthenticatedUsername with SetAuthenticatedAccountId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@priv
Patch Set: rebased Created 5 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/signin/inline_login_handler_impl.cc
diff --git a/chrome/browser/ui/webui/signin/inline_login_handler_impl.cc b/chrome/browser/ui/webui/signin/inline_login_handler_impl.cc
index a76a764a7e26ec91397597346d83ece34049a63c..697d92f1debc28f5e1f56c011997e2e6c8dae90c 100644
--- a/chrome/browser/ui/webui/signin/inline_login_handler_impl.cc
+++ b/chrome/browser/ui/webui/signin/inline_login_handler_impl.cc
@@ -306,13 +306,10 @@ void InlineSigninHelper::OnClientOAuthSuccess(const ClientOAuthResult& result) {
AboutSigninInternalsFactory::GetForProfile(profile_);
about_signin_internals->OnRefreshTokenReceived("Successful");
- AccountTrackerService* account_tracker =
- AccountTrackerServiceFactory::GetForProfile(profile_);
- std::string account_id =
- account_tracker->PickAccountIdForAccount(gaia_id_, email_);
-
// Prime the account tracker with this combination of gaia id/display email.
- account_tracker->SeedAccountInfo(gaia_id_, email_);
+ std::string account_id =
+ AccountTrackerServiceFactory::GetForProfile(profile_)
+ ->SeedAccountInfo(gaia_id_, email_);
signin_metrics::Source source = signin::GetSourceForPromoURL(current_url_);
@@ -387,7 +384,7 @@ void InlineSigninHelper::OnClientOAuthSuccess(const ClientOAuthResult& result) {
// OneClickSigninSyncStarter will delete itself once the job is done.
new OneClickSigninSyncStarter(
profile_, browser,
- email_, password_, result.refresh_token,
+ gaia_id_, email_, password_, result.refresh_token,
start_mode,
contents,
confirmation_required,
@@ -448,7 +445,7 @@ void InlineSigninHelper::ConfirmEmailAction(
break;
case ConfirmEmailDialogDelegate::START_SYNC:
new OneClickSigninSyncStarter(
- profile_, browser, email_, password_, refresh_token,
+ profile_, browser, gaia_id_, email_, password_, refresh_token,
start_mode, web_contents, confirmation_required, GURL(),
base::Bind(&InlineLoginHandlerImpl::SyncStarterCallback, handler_));
break;

Powered by Google App Engine
This is Rietveld 408576698