Commit 4038d923 authored by Thomas Tellier's avatar Thomas Tellier Committed by Commit Bot

Removing special focus cases for login screen PIN view

Since the PIN view is not focusable anymore, this is dead code.

Bug: 1043194
Change-Id: Ifd3e4cd8427abaca2bebd76187bba488bd2b4ef5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2153441Reviewed-by: default avatarMaksim Ivanov <emaxx@chromium.org>
Reviewed-by: default avatarDenis Kuznetsov [CET] <antrim@chromium.org>
Commit-Queue: Thomas Tellier <tellier@google.com>
Cr-Commit-Position: refs/heads/master@{#760997}
parent e9a57773
......@@ -487,21 +487,6 @@ void LoginPinView::NotifyAccessibilityLocationChanged() {
}
void LoginPinView::OnPasswordTextChanged(bool is_empty) {
// Disabling the backspace button or the submit button will make it lose
// focus. The previous focusable view is a button in PIN keyboard, which is
// slightly more expected than the user menu.
if (is_empty) {
View* with_focus = nullptr;
if (backspace_->HasFocus())
with_focus = backspace_;
else if (submit_button_->HasFocus())
with_focus = submit_button_;
if (with_focus) {
View* previous = with_focus->GetPreviousFocusableView();
if (previous)
previous->RequestFocus();
}
}
backspace_->SetEnabled(!is_empty);
submit_button_->SetEnabled(!is_empty);
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment