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

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: Update after review. 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 ae3542276ad6a9a6b5fc0c85cb12d16021e157b7..36a1e0733826b421eec61a150bb80c1b3308cee5 100644
--- a/chrome/browser/chromeos/login/screens/user_selection_screen.cc
+++ b/chrome/browser/chromeos/login/screens/user_selection_screen.cc
@@ -404,16 +404,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(
@@ -421,10 +423,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