Chromium Code Reviews
DescriptionRevert of Implemented consumer management unenrollment. (patchset #15 id:300001 of https://codereview.chromium.org/751703003/)
Reason for revert:
This seems to be leaking, see:
http://build.chromium.org/p/chromium.memory/buildstatus?builder=Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29&number=5886
Original issue's description:
> Implemented consumer management unenrollment.
>
> To implement this feature:
> - Added ConsumerUnenrollmentHandler and ConsumerUnenrollmentHandlerFactory.
> - Added DevcieCloudPolicyManagerChromeOS::Unregister() and Disconnect().
> - Added CloudPolicyService::Unregister().
> - Added unenrollment related stages in ConsumerManagementStage.
> - Extended ConsumerManagementNotification to show unenroll notifications.
>
> Also, the device cloud policy initializer is restarted after the unenrollment
> so that we can implement reboot-free enrollment in the future. To implement
> this feature:
> - Added OnDeviceCloudPolicyManagerDisconnected().
> - Added BrowserPolicyConnectChromeOS::RestartDeviceCloudPolicyInitializer()
> - Fixed the initiailzer so that it uses ConsumerManagementService to check if
> the device is consumer-managed rather using DeviceCloudPolicyStore, which
> doesn't load policies for consumer-managed devices.
>
> BUG=chromium:353050
> TEST=unit_tests
>
> Committed: https://crrev.com/051ed4f27b9361534e25311ad1987f30335faa16
> Cr-Commit-Position: refs/heads/master@{#311643}
TBR=bartfab@chromium.org,mnissler@chromium.org,ygorshenin@chromium.org,stevenjb@chromium.org,stevenjb@google.com,davidyu@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=chromium:353050
Committed: https://crrev.com/96bd0b01b6d0e993be38a06b1c0af69ae44f6acb
Cr-Commit-Position: refs/heads/master@{#311646}
Patch Set 1 #
Created: 5 years, 11 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|