Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(129)

Unified Diff: chrome/browser/policy/device_cloud_policy_manager_chromeos.cc

Issue 11415094: Split UserCloudPolicyManager implementation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix DeviceCloudPolicyManagerChromeOSTest.EnrolledDevice failure. Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/device_cloud_policy_manager_chromeos.cc
diff --git a/chrome/browser/policy/device_cloud_policy_manager_chromeos.cc b/chrome/browser/policy/device_cloud_policy_manager_chromeos.cc
index c90d509b50af44d3f7e0e68ed82719f8ae7c1459..81af6aa9604ede5d14faf3239f2dcfbee1d82684 100644
--- a/chrome/browser/policy/device_cloud_policy_manager_chromeos.cc
+++ b/chrome/browser/policy/device_cloud_policy_manager_chromeos.cc
@@ -49,8 +49,8 @@ const char* kMachineInfoSerialNumberKeys[] = {
DeviceCloudPolicyManagerChromeOS::DeviceCloudPolicyManagerChromeOS(
scoped_ptr<DeviceCloudPolicyStoreChromeOS> store,
EnterpriseInstallAttributes* install_attributes)
- : CloudPolicyManager(make_scoped_ptr<CloudPolicyStore>(store.get())),
- device_store_(store.release()), // Hack: retain |store| till here.
+ : CloudPolicyManager(store.get()),
+ device_store_(store.Pass()),
install_attributes_(install_attributes),
device_management_service_(NULL),
local_state_(NULL) {}
@@ -79,7 +79,7 @@ void DeviceCloudPolicyManagerChromeOS::StartEnrollment(
enrollment_handler_.reset(
new EnrollmentHandlerChromeOS(
- device_store_, install_attributes_, CreateClient(), auth_token,
+ device_store_.get(), install_attributes_, CreateClient(), auth_token,
allowed_device_modes,
base::Bind(&DeviceCloudPolicyManagerChromeOS::EnrollmentCompleted,
base::Unretained(this), callback)));

Powered by Google App Engine
This is Rietveld 408576698