New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 845313 link

Starred by 4 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug


Participants' hotlists:
LoginRefresh


Sign in to add a comment

"Signing in..." text missing from views login input field during SmartLock login

Project Member Reported by jhawkins@chromium.org, May 21 2018

Issue description

Sign out of the device.  Sign in using SmartLock.  Expect to see "Signing in..." in the input field, but that text is missing.
 
Labels: M-69
Owner: jdufault@chromium.org
Status: Assigned (was: Untriaged)
Labels: ReleaseBlock-Stable
Owner: jhawkins@chromium.org
You can reassign to me if you don't have time to look.
Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, May 25 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d3433075a470b4c7a4e50ff33b6485989799f7cc

commit d3433075a470b4c7a4e50ff33b6485989799f7cc
Author: James Hawkins <jhawkins@chromium.org>
Date: Fri May 25 17:25:35 2018

LoginAuthUserView: Display "Signing in..." when the user activates SmartLock.

This change also animates the ellipses while the operation is executing.

R=jdufault@chromium.org

Bug: 845313
Test: LoginAuthUserViewUnittest.PressReturnWithTapToUnlockEnabled
Change-Id: I5054a49177049070db5169e4c7e594dfe0a45baf
Reviewed-on: https://chromium-review.googlesource.com/1072804
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Commit-Queue: James Hawkins <jhawkins@chromium.org>
Cr-Commit-Position: refs/heads/master@{#561932}
[modify] https://crrev.com/d3433075a470b4c7a4e50ff33b6485989799f7cc/ash/ash_strings.grd
[modify] https://crrev.com/d3433075a470b4c7a4e50ff33b6485989799f7cc/ash/login/ui/login_auth_user_view.cc
[modify] https://crrev.com/d3433075a470b4c7a4e50ff33b6485989799f7cc/ash/login/ui/login_auth_user_view.h
[modify] https://crrev.com/d3433075a470b4c7a4e50ff33b6485989799f7cc/ash/login/ui/login_auth_user_view_unittest.cc

Status: Fixed (was: Started)
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 18

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f59e3a0112c5b09bee04402bbbcdb89236efd578

commit f59e3a0112c5b09bee04402bbbcdb89236efd578
Author: James Hawkins <jhawkins@chromium.org>
Date: Tue Sep 18 16:03:22 2018

Revert "LoginAuthUserView: Display "Signing in..." when the user activates SmartLock."

This reverts commit d3433075a470b4c7a4e50ff33b6485989799f7cc.

With this change in place, if an auth error occurs after the user
attempts the Smart Lock sign-in/unlock operation, the user is unable to
enter their password because the 'Signing in...' text is still displayed
in the password input field.

Rolling this forward requires handling that auth error state.

Original change's description:
> LoginAuthUserView: Display "Signing in..." when the user activates SmartLock.
>
> This change also animates the ellipses while the operation is executing.
>
> R=jdufault@chromium.org
>
> Bug: 845313
> Test: LoginAuthUserViewUnittest.PressReturnWithTapToUnlockEnabled
> Change-Id: I5054a49177049070db5169e4c7e594dfe0a45baf
> Reviewed-on: https://chromium-review.googlesource.com/1072804
> Reviewed-by: Jacob Dufault <jdufault@chromium.org>
> Commit-Queue: James Hawkins <jhawkins@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#561932}

R=jdufault@chromium.org

Bug:  873723 
Test: none
Change-Id: I0fdf6a491c991983bb0eea1c151d5d30851e4ee5
Reviewed-on: https://chromium-review.googlesource.com/1227293
Commit-Queue: James Hawkins <jhawkins@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Cr-Commit-Position: refs/heads/master@{#592060}
[modify] https://crrev.com/f59e3a0112c5b09bee04402bbbcdb89236efd578/ash/ash_strings.grd
[modify] https://crrev.com/f59e3a0112c5b09bee04402bbbcdb89236efd578/ash/login/ui/login_auth_user_view.cc
[modify] https://crrev.com/f59e3a0112c5b09bee04402bbbcdb89236efd578/ash/login/ui/login_auth_user_view.h
[modify] https://crrev.com/f59e3a0112c5b09bee04402bbbcdb89236efd578/ash/login/ui/login_auth_user_view_unittest.cc

Labels: -Pri-2 -M-69 Merge-Request-70 M-70 Pri-1
Status: Started (was: Fixed)
Labels: -Pri-1 -Merge-Request-70 -M-70 M-69 Pri-2
Status: Fixed (was: Started)
Whoops, wrong bug for merge request.
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-69; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-69 label, otherwise remove Merge-TBD label. Thanks.
Labels: -Merge-TBD
Labels: -ReleaseBlock-Stable -M-69 M-71
Status: Assigned (was: Fixed)
Keeping this issue open to fix properly.
Project Member

Comment 13 by bugdroid1@chromium.org, Sep 26

Labels: merge-merged-3538
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e017ac36996def7653b0b2e86abbab16e52b2448

commit e017ac36996def7653b0b2e86abbab16e52b2448
Author: James Hawkins <jhawkins@chromium.org>
Date: Wed Sep 26 23:56:35 2018

Revert "LoginAuthUserView: Display "Signing in..." when the user activates SmartLock."

This reverts commit d3433075a470b4c7a4e50ff33b6485989799f7cc.

With this change in place, if an auth error occurs after the user
attempts the Smart Lock sign-in/unlock operation, the user is unable to
enter their password because the 'Signing in...' text is still displayed
in the password input field.

Rolling this forward requires handling that auth error state.

Original change's description:
> LoginAuthUserView: Display "Signing in..." when the user activates SmartLock.
>
> This change also animates the ellipses while the operation is executing.
>
> R=jdufault@chromium.org
>
> Bug: 845313
> Test: LoginAuthUserViewUnittest.PressReturnWithTapToUnlockEnabled
> Change-Id: I5054a49177049070db5169e4c7e594dfe0a45baf
> Reviewed-on: https://chromium-review.googlesource.com/1072804
> Reviewed-by: Jacob Dufault <jdufault@chromium.org>
> Commit-Queue: James Hawkins <jhawkins@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#561932}

R=jdufault@chromium.org
TBR=jhawkins@chromium.org

(cherry picked from commit f59e3a0112c5b09bee04402bbbcdb89236efd578)

Bug:  873723 
Test: none
Change-Id: I0fdf6a491c991983bb0eea1c151d5d30851e4ee5
Reviewed-on: https://chromium-review.googlesource.com/1227293
Commit-Queue: James Hawkins <jhawkins@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#592060}
Reviewed-on: https://chromium-review.googlesource.com/1246950
Reviewed-by: James Hawkins <jhawkins@chromium.org>
Cr-Commit-Position: refs/branch-heads/3538@{#691}
Cr-Branched-From: 79f7c91a2b2a2932cd447fa6f865cb6662fa8fa6-refs/heads/master@{#587811}
[modify] https://crrev.com/e017ac36996def7653b0b2e86abbab16e52b2448/ash/ash_strings.grd
[modify] https://crrev.com/e017ac36996def7653b0b2e86abbab16e52b2448/ash/login/ui/login_auth_user_view.cc
[modify] https://crrev.com/e017ac36996def7653b0b2e86abbab16e52b2448/ash/login/ui/login_auth_user_view.h
[modify] https://crrev.com/e017ac36996def7653b0b2e86abbab16e52b2448/ash/login/ui/login_auth_user_view_unittest.cc

Labels: -M-71 -merge-merged-3538 M-72
Cc: jordynass@chromium.org hansberry@chromium.org jlklein@chromium.org shibasheikh@chromium.org khorimoto@chromium.org nohle@chromium.org jhawkins@chromium.org
 Issue 892481  has been merged into this issue.
Labels: -Pri-2 Pri-1
Owner: hansberry@chromium.org
Labels: Hotlist-auth-cleanup

Sign in to add a comment