New issue
Advanced search Search tips

Issue 659732 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Closed: Dec 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
V1



Sign in to add a comment

Rollout service authpolicy enums

Project Member Reported by rsorokin@chromium.org, Oct 26 2016

Issue description

Components: Enterprise
Labels: Enterprise-Triaged

Comment 3 by tnagel@chromium.org, Nov 21 2016

Labels: -Pri-3 V1 M-57 Pri-2
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 7 2016

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

commit 5e74d62ee9a204f4ce47432219a754897bd80830
Author: tbarzic <tbarzic@chromium.org>
Date: Wed Dec 07 23:08:27 2016

Move ADJoinErrorType from authpolicy to authpolicy::types namespace

authpolicy::ADJoinErrorType conflicts with type defined in cros system
api dbus service constants, and prevents cros_service_api DEPS uprev.

BUG= 659732 

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

[modify] https://crrev.com/5e74d62ee9a204f4ce47432219a754897bd80830/chromeos/dbus/auth_policy_client.cc
[modify] https://crrev.com/5e74d62ee9a204f4ce47432219a754897bd80830/chromeos/dbus/auth_policy_client.h
[modify] https://crrev.com/5e74d62ee9a204f4ce47432219a754897bd80830/chromeos/dbus/fake_auth_policy_client.cc

Project Member

Comment 5 by bugdroid1@chromium.org, Dec 9 2016

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

commit ff0f2ca2e95f208655869178c32cf666c8f6e478
Author: tbarzic <tbarzic@chromium.org>
Date: Fri Dec 09 02:33:16 2016

Roll cros_system_api to pick up new cras constant

Rolling cros_system_api up to:
https://chromium.googlesource.com/chromiumos/platform/system_api/+/4489d28f1ec210c840d4f4827f8210ab27141f2d

BUG= 661861 ,  659732 

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

[modify] https://crrev.com/ff0f2ca2e95f208655869178c32cf666c8f6e478/DEPS

Comment 6 by tnagel@chromium.org, Dec 19 2016

Status: Started (was: Assigned)
Can this be closed?
Project Member

Comment 8 by bugdroid1@chromium.org, Dec 22 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/system_api/+/0995a6af616cd4ef0e94d278ecc92aa2537b16c3

commit 0995a6af616cd4ef0e94d278ecc92aa2537b16c3
Author: Roman Sorokin <rsorokin@chromium.org>
Date: Thu Dec 22 12:55:33 2016

Add *_COUNT member for error enums.

BUG= chromium:659732 
TEST=none

Change-Id: I35af2935949e3aea2964bb80d7fee718b3a0d6b8
Reviewed-on: https://chromium-review.googlesource.com/423088
Commit-Ready: Roman Sorokin <rsorokin@chromium.org>
Tested-by: Roman Sorokin <rsorokin@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>

[modify] https://crrev.com/0995a6af616cd4ef0e94d278ecc92aa2537b16c3/dbus/authpolicy/dbus-constants.h

Project Member

Comment 9 by bugdroid1@chromium.org, Dec 23 2016

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

commit c4cdabc72364a124afe75b2d7bcc7557f300d362
Author: rsorokin <rsorokin@chromium.org>
Date: Fri Dec 23 12:25:56 2016

Roll cros_system_api for authpolicy constants

BUG= 659732 

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

[modify] https://crrev.com/c4cdabc72364a124afe75b2d7bcc7557f300d362/DEPS

Project Member

Comment 10 by bugdroid1@chromium.org, Dec 27 2016

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

commit 461140f54e88128fa2ce229a42fd923a25d99b20
Author: rsorokin <rsorokin@chromium.org>
Date: Tue Dec 27 16:58:32 2016

Rollout service authpolicy enums

BUG= 659732 
TEST=manual

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

[modify] https://crrev.com/461140f54e88128fa2ce229a42fd923a25d99b20/DEPS

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
bulk Verify of Chromad V1 bugs

Sign in to add a comment