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

Unified Diff: chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc

Issue 2593133002: Revert of Add account_type into AccountId (Closed)
Patch Set: Created 4 years 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/chromeos/login/gaia_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc
index 460bf6ecb91cb8276c8a08fe7012ef3c6b1b5e53..5660e1b87073814c93868d46b3544c8357e58c44 100644
--- a/chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/gaia_screen_handler.cc
@@ -88,9 +88,9 @@
// If there's a populated email, we must check first that this user is using
// SAML in order to decide whether to show the interstitial page.
- const user_manager::User* user = user_manager::UserManager::Get()->FindUser(
- user_manager::known_user::GetAccountId(email, std::string() /* id */,
- AccountType::UNKNOWN));
+ const user_manager::User* user =
+ user_manager::UserManager::Get()->FindUser(
+ user_manager::known_user::GetAccountId(email, std::string()));
if (user && user->using_saml())
return GAIA_SCREEN_MODE_SAML_INTERSTITIAL;
@@ -406,7 +406,7 @@
void GaiaScreenHandler::HandleIdentifierEntered(const std::string& user_email) {
if (!Delegate()->IsUserWhitelisted(user_manager::known_user::GetAccountId(
- user_email, std::string() /* id */, AccountType::UNKNOWN)))
+ user_email, std::string() /* gaia_id */)))
ShowWhitelistCheckFailedError();
}
@@ -451,13 +451,12 @@
AccountId GaiaScreenHandler::GetAccountId(
const std::string& authenticated_email,
- const std::string& id,
- const AccountType& account_type) const {
+ const std::string& gaia_id) const {
const std::string canonicalized_email =
gaia::CanonicalizeEmail(gaia::SanitizeEmail(authenticated_email));
- const AccountId account_id = user_manager::known_user::GetAccountId(
- authenticated_email, id, account_type);
+ const AccountId account_id =
+ user_manager::known_user::GetAccountId(authenticated_email, gaia_id);
if (account_id.GetUserEmail() != canonicalized_email) {
LOG(WARNING) << "Existing user '" << account_id.GetUserEmail()
@@ -482,7 +481,7 @@
const std::string sanitized_email = gaia::SanitizeEmail(email);
Delegate()->SetDisplayEmail(sanitized_email);
- UserContext user_context(GetAccountId(email, gaia_id, AccountType::GOOGLE));
+ UserContext user_context(GetAccountId(email, gaia_id));
user_context.SetKey(Key(password));
user_context.SetAuthCode(auth_code);
user_context.SetAuthFlow(using_saml
@@ -567,8 +566,7 @@
DCHECK(!gaia_id.empty());
const std::string sanitized_email = gaia::SanitizeEmail(typed_email);
Delegate()->SetDisplayEmail(sanitized_email);
- UserContext user_context(
- GetAccountId(typed_email, gaia_id, AccountType::GOOGLE));
+ UserContext user_context(GetAccountId(typed_email, gaia_id));
user_context.SetKey(Key(password));
user_context.SetAuthFlow(using_saml
? UserContext::AUTH_FLOW_GAIA_WITH_SAML
@@ -707,7 +705,7 @@
Delegate()->LoadSigninWallpaper();
} else {
Delegate()->LoadWallpaper(user_manager::known_user::GetAccountId(
- populated_email_, std::string() /* id */, AccountType::UNKNOWN));
+ populated_email_, std::string()));
}
input_method::InputMethodManager* imm =

Powered by Google App Engine
This is Rietveld 408576698