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

Issue 725142 link

Starred by 1 user

Issue metadata

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

Blocking:
issue 724708



Sign in to add a comment

Unlock not working when there are multiple users signed in.

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

Issue description

It seems that ProximityAuthSystem::OnFocusedUserChanged() gets called with an invalid user (EmptyAccountId). This cases the remote device life cycle to be destroyed.

This call was triggered by ProximityAuthSystem::OnScreenDidLock(), which seemed to be triggered by 

https://cs.chromium.org/chromium/src/components/proximity_auth/screenlock_bridge.cc?q=ScreenlockBridge+package:%5Echromium$&dr=CSs&l=139

This was working before https://codereview.chromium.org/2861913002/, which removes the invalid account check from ProximityAuthSystem::OnFocusedUserChanged().
 

Comment 1 by tengs@chromium.org, May 23 2017

Blocking: 724708
Project Member

Comment 2 by bugdroid1@chromium.org, May 23 2017

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

commit 9265ff00a94993e1db869cb7e540117102bc85d5
Author: sacomoto <sacomoto@chromium.org>
Date: Tue May 23 21:34:54 2017

Do not destroy the life cycle when the user is invalid.

BUG= 725142 

Review-Url: https://codereview.chromium.org/2899803003
Cr-Commit-Position: refs/heads/master@{#474064}

[modify] https://crrev.com/9265ff00a94993e1db869cb7e540117102bc85d5/components/proximity_auth/proximity_auth_system.cc

Status: Fixed (was: Assigned)

Comment 4 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Sign in to add a comment