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

Issue 683150 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Policies are not getting merged based on source priority

Project Member Reported by hunyadym@chromium.org, Jan 20 2017

Issue description

.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 23 2017

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

commit 877b2113a4375f07ec1cd07a7ab4c417d957a37e
Author: hunyadym <hunyadym@chromium.org>
Date: Mon Jan 23 16:52:14 2017

Use source parameter too to determine policy priority.

This fixes the problem of Enterprise default policies sometimes
overwriting Active Directory-set policies.

BUG= 683150 

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

[modify] https://crrev.com/877b2113a4375f07ec1cd07a7ab4c417d957a37e/components/policy/core/common/policy_map.cc
[modify] https://crrev.com/877b2113a4375f07ec1cd07a7ab4c417d957a37e/components/policy/core/common/policy_map.h
[modify] https://crrev.com/877b2113a4375f07ec1cd07a7ab4c417d957a37e/components/policy/core/common/policy_map_unittest.cc

Labels: Merge-Request-57
Status: Fixed (was: Started)
Project Member

Comment 3 by sheriffbot@chromium.org, Jan 24 2017

Labels: -Merge-Request-57 Hotlist-Merge-Approved Merge-Approved-57
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-57 M-57 merge-merged-2987
This has been merged:

https://chromium.googlesource.com/chromium/src.git/+/4553996f1fa4725d1fcab5214c5889b7172fd5b5
Status: Verified (was: Fixed)
bulk Verify of older or not-user-facing Chromad bugs

Sign in to add a comment