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

Unified Diff: chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc

Issue 2471993002: Remove calls to User::email() from chrome/browser/ui/webui/* (Closed)
Patch Set: Created 4 years, 1 month 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/settings/md_settings_localized_strings_provider.cc
diff --git a/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc b/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc
index 39f72880e619d7da05f361c7b6c76c4703d5130e..61e1f7a8552170494edcc93c4b5c5bafcd9c7893 100644
--- a/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc
+++ b/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc
@@ -890,10 +890,12 @@ void AddMultiProfilesStrings(content::WebUIDataSource* html_source,
const user_manager::User* user =
chromeos::ProfileHelper::Get()->GetUserByProfile(profile);
- std::string primary_user_email = user_manager->GetPrimaryUser()->email();
- html_source->AddString("primaryUserEmail", primary_user_email);
- html_source->AddBoolean("isSecondaryUser",
- user && user->email() != primary_user_email);
+ const user_manager::User* primary_user = user_manager->GetPrimaryUser();
+ html_source->AddString("primaryUserEmail",
+ primary_user->GetAccountId().GetUserEmail());
+ html_source->AddBoolean(
+ "isSecondaryUser",
+ user && user->GetAccountId() != primary_user->GetAccountId());
}
#endif

Powered by Google App Engine
This is Rietveld 408576698