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

Side by Side Diff: chrome/browser/policy/cloud/policy_header_service_factory.cc

Issue 2638623006: Merge {Device,User}ActiveDirectoryPolicyManager into a single class (Closed)
Patch Set: Have factories return unique_ptr Created 3 years, 11 months 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/policy/cloud/policy_header_service_factory.h" 5 #include "chrome/browser/policy/cloud/policy_header_service_factory.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
11 #include "build/build_config.h" 11 #include "build/build_config.h"
12 #include "chrome/browser/browser_process.h" 12 #include "chrome/browser/browser_process.h"
13 #include "chrome/browser/profiles/profile.h" 13 #include "chrome/browser/profiles/profile.h"
14 #include "components/keyed_service/content/browser_context_dependency_manager.h" 14 #include "components/keyed_service/content/browser_context_dependency_manager.h"
15 #include "components/keyed_service/core/keyed_service.h" 15 #include "components/keyed_service/core/keyed_service.h"
16 #include "components/policy/core/browser/browser_policy_connector.h" 16 #include "components/policy/core/browser/browser_policy_connector.h"
17 #include "components/policy/core/common/cloud/cloud_policy_store.h" 17 #include "components/policy/core/common/cloud/cloud_policy_store.h"
18 #include "components/policy/core/common/cloud/device_management_service.h" 18 #include "components/policy/core/common/cloud/device_management_service.h"
19 #include "components/policy/core/common/cloud/policy_header_service.h" 19 #include "components/policy/core/common/cloud/policy_header_service.h"
20 20
21 #if defined(OS_CHROMEOS) 21 #if defined(OS_CHROMEOS)
22 #include "chrome/browser/chromeos/policy/active_directory_policy_manager.h"
22 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h" 23 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
23 #include "chrome/browser/chromeos/policy/user_active_directory_policy_manager.h"
24 #include "chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.h" 24 #include "chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos.h"
25 #include "chrome/browser/chromeos/policy/user_policy_manager_factory_chromeos.h" 25 #include "chrome/browser/chromeos/policy/user_policy_manager_factory_chromeos.h"
26 #else 26 #else
27 #include "chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h" 27 #include "chrome/browser/policy/cloud/user_cloud_policy_manager_factory.h"
28 #include "components/policy/core/common/cloud/user_cloud_policy_manager.h" 28 #include "components/policy/core/common/cloud/user_cloud_policy_manager.h"
29 #endif 29 #endif
30 30
31 namespace policy { 31 namespace policy {
32 32
33 namespace { 33 namespace {
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 92
93 CloudPolicyStore* user_store; 93 CloudPolicyStore* user_store;
94 #if defined(OS_CHROMEOS) 94 #if defined(OS_CHROMEOS)
95 Profile* profile = Profile::FromBrowserContext(context); 95 Profile* profile = Profile::FromBrowserContext(context);
96 CloudPolicyManager* cloud_manager = 96 CloudPolicyManager* cloud_manager =
97 UserPolicyManagerFactoryChromeOS::GetCloudPolicyManagerForProfile( 97 UserPolicyManagerFactoryChromeOS::GetCloudPolicyManagerForProfile(
98 profile); 98 profile);
99 if (cloud_manager) { 99 if (cloud_manager) {
100 user_store = cloud_manager->core()->store(); 100 user_store = cloud_manager->core()->store();
101 } else { 101 } else {
102 UserActiveDirectoryPolicyManager* active_directory_manager = 102 ActiveDirectoryPolicyManager* active_directory_manager =
103 UserPolicyManagerFactoryChromeOS:: 103 UserPolicyManagerFactoryChromeOS::
104 GetActiveDirectoryPolicyManagerForProfile(profile); 104 GetActiveDirectoryPolicyManagerForProfile(profile);
105 if (!active_directory_manager) 105 if (!active_directory_manager)
106 return nullptr; 106 return nullptr;
107 user_store = active_directory_manager->store(); 107 user_store = active_directory_manager->store();
108 } 108 }
109 #else 109 #else
110 CloudPolicyManager* manager = 110 CloudPolicyManager* manager =
111 UserCloudPolicyManagerFactory::GetForBrowserContext(context); 111 UserCloudPolicyManagerFactory::GetForBrowserContext(context);
112 if (!manager) 112 if (!manager)
113 return nullptr; 113 return nullptr;
114 user_store = manager->core()->store(); 114 user_store = manager->core()->store();
115 #endif 115 #endif
116 116
117 std::unique_ptr<PolicyHeaderService> service = 117 std::unique_ptr<PolicyHeaderService> service =
118 base::MakeUnique<PolicyHeaderService>( 118 base::MakeUnique<PolicyHeaderService>(
119 connector->device_management_service()->GetServerUrl(), 119 connector->device_management_service()->GetServerUrl(),
120 kPolicyVerificationKeyHash, user_store); 120 kPolicyVerificationKeyHash, user_store);
121 return new PolicyHeaderServiceWrapper(std::move(service)); 121 return new PolicyHeaderServiceWrapper(std::move(service));
122 } 122 }
123 123
124 // static 124 // static
125 PolicyHeaderServiceFactory* PolicyHeaderServiceFactory::GetInstance() { 125 PolicyHeaderServiceFactory* PolicyHeaderServiceFactory::GetInstance() {
126 return base::Singleton<PolicyHeaderServiceFactory>::get(); 126 return base::Singleton<PolicyHeaderServiceFactory>::get();
127 } 127 }
128 128
129 } // namespace policy 129 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698