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

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

Issue 2785003002: cros: Showing pin on lock is no longer associated with sending user list. (Closed)
Patch Set: Address comments Created 3 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
« no previous file with comments | « no previous file | chrome/browser/resources/chromeos/login/lock.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 33d6629f9d8cd6bb371ddf098375139e71781939..7bcdddb41b92ca8821fe0a764ef0c5c06ec8c9bc 100644
--- a/chrome/browser/chromeos/login/screens/user_selection_screen.cc
+++ b/chrome/browser/chromeos/login/screens/user_selection_screen.cc
@@ -49,7 +49,6 @@ const char kKeyPublicAccount[] = "publicAccount";
const char kKeyLegacySupervisedUser[] = "legacySupervisedUser";
const char kKeyChildUser[] = "childUser";
const char kKeyDesktopUser[] = "isDesktopUser";
-const char kKeyShowPin[] = "showPin";
const char kKeySignedIn[] = "signedIn";
const char kKeyCanRemove[] = "canRemove";
const char kKeyIsOwner[] = "isOwner";
@@ -118,19 +117,6 @@ void AddPublicSessionDetailsToUserDictionaryEntry(
user_dict->Set(kKeyInitialKeyboardLayout, GetCurrentKeyboardLayout());
}
-// Returns true if the PIN keyboard should be displayed for the given |user|.
-bool CanShowPinForUser(user_manager::User* user) {
- if (!user->is_logged_in())
- return false;
-
- quick_unlock::QuickUnlockStorage* quick_unlock_storage =
- quick_unlock::QuickUnlockFactory::GetForUser(user);
- if (!quick_unlock_storage)
- return false;
-
- return quick_unlock_storage->IsPinAuthenticationAvailable();
-}
-
// Returns true if the fingerprint icon should be displayed for the given
// |user|.
bool AllowFingerprintForUser(user_manager::User* user) {
@@ -190,7 +176,6 @@ void UserSelectionScreen::FillUserDictionary(
user_dict->SetBoolean(kKeyChildUser, is_child_user);
user_dict->SetBoolean(kKeyDesktopUser, false);
user_dict->SetInteger(kKeyInitialAuthType, auth_type);
- user_dict->SetBoolean(kKeyShowPin, CanShowPinForUser(user));
user_dict->SetBoolean(kKeySignedIn, user->is_logged_in());
user_dict->SetBoolean(kKeyIsOwner, is_owner);
user_dict->SetBoolean(kKeyIsActiveDirectory, user->IsActiveDirectoryUser());
« no previous file with comments | « no previous file | chrome/browser/resources/chromeos/login/lock.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698