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

Issue 757813 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 724708



Sign in to add a comment

EasyUnlock v2: Not searching for phone on the first time the screen is locked after the signing in.

Project Member Reported by sacomoto@chromium.org, Aug 22 2017

Issue description

The first time the screen is locked after a the user signs in EasyUnlock is completely inactive (EasyUnlock is correctly setup, and the login flag is disabled). 

It doesn't show the spinner UI, only the vanilla screen lock UI. It doesn't search for the phone.

The logs are here:

-----------------------------------------------------------

16:14:34.457
easy_unlock_service_regular.cc:669

Loading deferred devices after screen unlock.

16:14:34.460
remote_device_loader.cc:73

Loading 1 remote devices

16:14:34.500
remote_device_loader.cc:104

Derived PSK for LG Nexus 5X, 0 keys remaining.

16:14:34.501
easy_unlock_service.cc:783

Creating ProximityAuthSystem.

16:14:34.502
easy_unlock_service_regular.cc:357

Setting RemoteDevices:
  [{"bluetoothAddress":"A0:91:69:28:41:E6","name":"LG Nexus
5X","permitId":"permit://google.com/easyunlock/v1/sacomotosmartlock4@gmail.=
com","permitRecord":{"data":"CAESRQogZlPhunkd2lLjKwn0LMvPPSP6gtcnX75FTRS-j5=
4bq98SIQCVvMr81YiABOU6-05i2MZ6WJdBqO81SsFNhMilUXmfXA=3D=3D","id":"CAESRQogZ=
lPhunkd2lLjKwn0LMvPPSP6gtcnX75FTRS-j54bq98SIQCVvMr81YiABOU6-05i2MZ6WJdBqO81=
SsFNhMilUXmfXA=3D=3D","type":"license"},"psk":"b6wtNwDRQuPPanru6efTJccQBGCw=
EYJvMgcVhVSqryc=3D","serializedBeaconSeeds":"[\"CiAS9VXvEfBHg-7APbvl4b-NBTF=
-QmRJwrXzesaaoWoDPRCAoMPL3CsYgLCnjOEr\",\"CiBlcv3zxcPVd0ngQ6ViYRaUWZab6o3Kp=
Fk6gqYCDZx58xCAsKeM4SsYgMCLzeUr\",\"CiBCO3DbzHWwppCe-Uup4AYW-0Nba9GlKQgx9jE=
BHRBCchCAwIvN5SsYgNDvjeor\",\"CiASMGb2r2scb8J7SH3tr6vM2xmDu7SG9vHu9uG5oBGlZ=
hCA0O-N6isYgODTzu4r\"]"}]

----------------------------------------------------------------

The second the screen is locked everything works fine.
 

Comment 1 by tengs@chromium.org, Aug 22 2017

Blocking: 724708
Components: UI>ProximityAuth
Labels: M-61
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 22 2017

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

commit ce1cb7b046176acf9fedb093c2d1c669392cf2de
Author: Gustavo Sacomoto <sacomoto@chromium.org>
Date: Tue Aug 22 19:23:23 2017

[EasyUnlock] Fixing the first unlock after user signs in.

Bug:  757813 
Change-Id: I49c69665dafe46df964c91f6af0a6004916f3a67
Reviewed-on: https://chromium-review.googlesource.com/626299
Reviewed-by: Tim Song <tengs@chromium.org>
Commit-Queue: Gustavo Sacomoto <sacomoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#496400}
[modify] https://crrev.com/ce1cb7b046176acf9fedb093c2d1c669392cf2de/chrome/browser/signin/easy_unlock_service_regular.cc

Labels: Merge-Request-61
Project Member

Comment 4 by sheriffbot@chromium.org, Aug 23 2017

Labels: -Merge-Request-61 Merge-Review-61 Hotlist-Merge-Review
This bug requires manual review: We are only 12 days from stable.
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Review-61 Merge-Approved-61
Approving merge to M61 Chrome OS.
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 24 2017

Labels: -merge-approved-61 merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/65ccb11cc71a912f91b4156180e261723d15d7ea

commit 65ccb11cc71a912f91b4156180e261723d15d7ea
Author: Gustavo Sacomoto <sacomoto@chromium.org>
Date: Thu Aug 24 09:16:59 2017

[EasyUnlock] Fixing the first unlock after user signs in.

TBR=sacomoto@chromium.org

(cherry picked from commit ce1cb7b046176acf9fedb093c2d1c669392cf2de)

Bug:  757813 
Change-Id: I49c69665dafe46df964c91f6af0a6004916f3a67
Reviewed-on: https://chromium-review.googlesource.com/626299
Reviewed-by: Tim Song <tengs@chromium.org>
Commit-Queue: Gustavo Sacomoto <sacomoto@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#496400}
Reviewed-on: https://chromium-review.googlesource.com/631682
Reviewed-by: Gustavo Sacomoto <sacomoto@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#851}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/65ccb11cc71a912f91b4156180e261723d15d7ea/chrome/browser/signin/easy_unlock_service_regular.cc

Status: Fixed (was: Untriaged)

Comment 8 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment