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

Unified Diff: chrome/browser/chromeos/login/screens/user_selection_screen.cc

Issue 1191273002: ChromeOS: Show the new POD RE-AUTH ICON when users needs to re-auth (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use CSS selectors instead of JS code. Created 5 years, 6 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/chromeos/login/screens/user_selection_screen.cc
diff --git a/chrome/browser/chromeos/login/screens/user_selection_screen.cc b/chrome/browser/chromeos/login/screens/user_selection_screen.cc
index 814faee0196f4f8509c36bf03975a9ca2b9b8d0e..9ee6910254aa5ee3c248b41a730131f4250d6c69 100644
--- a/chrome/browser/chromeos/login/screens/user_selection_screen.cc
+++ b/chrome/browser/chromeos/login/screens/user_selection_screen.cc
@@ -405,16 +405,18 @@ void UserSelectionScreen::SendUserList() {
}
handler_->LoadUsers(users_list, show_guest_);
+
+ CheckAllUserStatuses();
}
void UserSelectionScreen::HandleGetUsers() {
SendUserList();
}
-void UserSelectionScreen::CheckUserStatus(const std::string& user_id) {
+bool UserSelectionScreen::CheckUserStatus(const std::string& user_id) {
// No checks on lock screen.
if (ScreenLocker::default_screen_locker())
- return;
+ return false;
if (!token_handle_util_.get()) {
token_handle_util_.reset(
@@ -422,10 +424,19 @@ void UserSelectionScreen::CheckUserStatus(const std::string& user_id) {
}
if (token_handle_util_->HasToken(user_id)) {
- token_handle_util_->CheckToken(
+ return token_handle_util_->CheckToken(
user_id, base::Bind(&UserSelectionScreen::OnUserStatusChecked,
weak_factory_.GetWeakPtr()));
}
+ return false;
+}
+
+void UserSelectionScreen::CheckAllUserStatuses() {
+ const user_manager::UserList& users = GetUsers();
+ for (const user_manager::User* user : users) {
+ // ignore return value.
+ CheckUserStatus(user->email());
+ }
}
void UserSelectionScreen::OnUserStatusChecked(

Powered by Google App Engine
This is Rietveld 408576698