New issue
Advanced search Search tips

Issue 676619 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Undeprecate SigninAllowed

Project Member Reported by pastarmovj@chromium.org, Dec 22 2016

Issue description

SigningAllowed has been marked deprecated because it was essentially 99% the same as SyncDisabled. Now with roaming profiles it makes sense to disable signin but not sync. 

We should make sure that the policies play well nicely with each other and that the documentation for them is consistent.
 
Status: Started (was: Assigned)
The decision here was to instead document the dependency between SyncEnabled and RoamingProfileSupportEnabled policies and keep the SigninAllowed policy deprecated.
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 11 2017

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

commit f143b4e8b90d870ea29a2009b2045a0073cadba5
Author: pastarmovj <pastarmovj@chromium.org>
Date: Tue Jul 11 14:29:08 2017

Update the policy documentation for sync and local sync.

The RoamingProfileSupportEnabled documentation was wrongly referring
to a deprecated policy while the actual dependency on the SyncDisabled
policy was omitted.

Now the proper interconnection between the two policies is described.

BUG= 676619 
TEST=none - documentation only change.

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

[modify] https://crrev.com/f143b4e8b90d870ea29a2009b2045a0073cadba5/components/policy/resources/policy_templates.json

Labels: -M-57 M-61
Status: Fixed (was: Started)
The policy will not be undeprecated instead the documentation was made clear about the dependencies between roaming profiles related policies and the policies controlling sync.

There is no change to the code to be verified this is pure documentation change.

Sign in to add a comment