Chromium Code Reviews| Index: chrome/browser/ui/ash/multi_user/multi_user_context_menu_chromeos.cc |
| diff --git a/chrome/browser/ui/ash/multi_user/multi_user_context_menu_chromeos.cc b/chrome/browser/ui/ash/multi_user/multi_user_context_menu_chromeos.cc |
| index 7d808f1f88616468fae93f070cfe8f860d40a1f6..2233536322a3fb9395e9dec6593b8e64d627d481 100644 |
| --- a/chrome/browser/ui/ash/multi_user/multi_user_context_menu_chromeos.cc |
| +++ b/chrome/browser/ui/ash/multi_user/multi_user_context_menu_chromeos.cc |
| @@ -76,8 +76,8 @@ scoped_ptr<ui::MenuModel> CreateMultiUserContextMenu(aura::Window* window) { |
| // If this window is not owned, we don't show the menu addition. |
| chrome::MultiUserWindowManager* manager = |
| chrome::MultiUserWindowManager::GetInstance(); |
| - const std::string user_id = manager->GetWindowOwner(window); |
| - if (user_id.empty() || !window) |
| + const AccountId account_id = manager->GetWindowOwner(window); |
|
stevenjb
2015/11/06 17:45:29
const AccountId& ?
Alexander Alekseev
2015/11/06 23:38:59
Done.
|
| + if (!account_id.is_valid() || !window) |
| return model.Pass(); |
| chromeos::MultiUserContextMenuChromeos* menu = |
| new chromeos::MultiUserContextMenuChromeos(window); |
| @@ -105,8 +105,8 @@ void OnAcceptTeleportWarning(const AccountId& account_id, |
| ash::MultiProfileUMA::RecordTeleportAction( |
| ash::MultiProfileUMA::TELEPORT_WINDOW_CAPTION_MENU); |
| - chrome::MultiUserWindowManager::GetInstance()->ShowWindowForUser( |
| - window_, account_id.GetUserEmail()); |
| + chrome::MultiUserWindowManager::GetInstance()->ShowWindowForUser(window_, |
| + account_id); |
| } |
| void ExecuteVisitDesktopCommand(int command_id, aura::Window* window) { |
| @@ -131,9 +131,9 @@ void ExecuteVisitDesktopCommand(int command_id, aura::Window* window) { |
| for (user_manager::UserList::const_iterator it = logged_in_users.begin(); |
| it != logged_in_users.end(); |
| ++it) { |
| - if (multi_user_util::GetProfileFromUserID( |
| - multi_user_util::GetUserIDFromEmail((*it)->email()))->GetPrefs()-> |
| - GetBoolean(prefs::kMultiProfileWarningShowDismissed)) { |
| + if (multi_user_util::GetProfileFromAccountId((*it)->GetAccountId()) |
| + ->GetPrefs() |
| + ->GetBoolean(prefs::kMultiProfileWarningShowDismissed)) { |
| bool active_user_show_option = |
| ProfileManager::GetActiveUserProfile()-> |
| GetPrefs()->GetBoolean(prefs::kMultiProfileWarningShowDismissed); |