New issue
Advanced search Search tips

Issue 671689 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

UserManager::IsCurrentUserOwner returns false for owner

Project Member Reported by xiy...@chromium.org, Dec 6 2016

Issue description

UserManager::IsCurrentUserOwner is borken on ToT and always returns false even for an owner.

In current code, it is updated on NOTIFICATION_OWNERSHIP_STATUS_CHANGED, which is sent in DeviceSettingsService when OwnershipStatus is changed.

This would not work since ownership check could happen on the login screen when there is current user. And when owner signs in, the ownership check already finishes thus OwnershipStatus would not change. Hence, leaving IsCurrentUserOwner not marked properly.
 
Cc: -r...@chromium.org st...@chromium.org

Comment 2 by st...@chromium.org, Mar 3 2017

Cc: r...@chromium.org

Comment 3 by st...@chromium.org, Mar 3 2017

Cc: -st...@chromium.org
Status: Fixed (was: Available)
Fixed in https://codereview.chromium.org/2779973007 for  issue 702308  and  issue 706820 .

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

Status: Archived (was: Fixed)

Comment 6 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)

Sign in to add a comment