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

Unified Diff: trunk/src/chrome/browser/ui/ash/multi_user/multi_user_context_menu_chromeos.cc

Issue 260783002: Revert 267158 "CleanUp: Introduce UserInfo. Move session_state s..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 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: trunk/src/chrome/browser/ui/ash/multi_user/multi_user_context_menu_chromeos.cc
===================================================================
--- trunk/src/chrome/browser/ui/ash/multi_user/multi_user_context_menu_chromeos.cc (revision 267249)
+++ trunk/src/chrome/browser/ui/ash/multi_user/multi_user_context_menu_chromeos.cc (working copy)
@@ -5,7 +5,7 @@
#include "chrome/browser/ui/ash/multi_user/multi_user_context_menu.h"
#include "ash/multi_profile_uma.h"
-#include "ash/session/session_state_delegate.h"
+#include "ash/session_state_delegate.h"
#include "ash/shell.h"
#include "base/bind.h"
#include "base/callback.h"
@@ -87,13 +87,13 @@
new chromeos::MultiUserContextMenuChromeos(window);
model.reset(menu);
for (int user_index = 1; user_index < logged_in_users; ++user_index) {
- const ash::UserInfo* user_info = delegate->GetUserInfo(user_index);
- menu->AddItem(user_index == 1 ? IDC_VISIT_DESKTOP_OF_LRU_USER_2
- : IDC_VISIT_DESKTOP_OF_LRU_USER_3,
- l10n_util::GetStringFUTF16(
- IDS_VISIT_DESKTOP_OF_LRU_USER,
- user_info->GetDisplayName(),
- base::ASCIIToUTF16(user_info->GetEmail())));
+ menu->AddItem(
+ user_index == 1 ? IDC_VISIT_DESKTOP_OF_LRU_USER_2 :
+ IDC_VISIT_DESKTOP_OF_LRU_USER_3,
+ l10n_util::GetStringFUTF16(
+ IDS_VISIT_DESKTOP_OF_LRU_USER,
+ delegate->GetUserDisplayName(user_index),
+ base::ASCIIToUTF16(delegate->GetUserEmail(user_index))));
}
}
return model.Pass();
@@ -118,11 +118,8 @@
// When running the multi user mode on Chrome OS, windows can "visit"
// another user's desktop.
const std::string& user_id =
- ash::Shell::GetInstance()
- ->session_state_delegate()
- ->GetUserInfo(IDC_VISIT_DESKTOP_OF_LRU_USER_2 == command_id ? 1
- : 2)
- ->GetUserID();
+ ash::Shell::GetInstance()->session_state_delegate()->GetUserID(
+ IDC_VISIT_DESKTOP_OF_LRU_USER_2 == command_id ? 1 : 2);
base::Callback<void(bool)> on_accept =
base::Bind(&OnAcceptTeleportWarning, user_id, window);

Powered by Google App Engine
This is Rietveld 408576698