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

Side by Side Diff: chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.h

Issue 2603463003: Re-enable fetching component policies for login screen apps (Closed)
Patch Set: Rename the for-testing member 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_CLOUD_POLICY_MANAGER_CHROMEOS_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_CLOUD_POLICY_MANAGER_CHROMEOS_H_
6 #define CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_CLOUD_POLICY_MANAGER_CHROMEOS_H_ 6 #define CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_CLOUD_POLICY_MANAGER_CHROMEOS_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 // policy server. 116 // policy server.
117 StatusUploader* GetStatusUploader() const { return status_uploader_.get(); } 117 StatusUploader* GetStatusUploader() const { return status_uploader_.get(); }
118 118
119 // Passes the pointer to the schema registry that corresponds to the signin 119 // Passes the pointer to the schema registry that corresponds to the signin
120 // profile. 120 // profile.
121 // 121 //
122 // After this method is called, the component cloud policy manager becomes 122 // After this method is called, the component cloud policy manager becomes
123 // associated with this schema registry. 123 // associated with this schema registry.
124 void SetSigninProfileSchemaRegistry(SchemaRegistry* schema_registry); 124 void SetSigninProfileSchemaRegistry(SchemaRegistry* schema_registry);
125 125
126 // Sets whether the component cloud policy service should be created. 126 // Sets whether the component cloud policy should be disabled (by skipping
127 // Defaults to true. 127 // the component cloud policy service creation).
128 void set_is_component_policy_enabled_for_testing( 128 void set_component_policy_disabled_for_testing(
129 bool is_component_policy_enabled) { 129 bool component_policy_disabled_for_testing) {
130 is_component_policy_enabled_ = is_component_policy_enabled; 130 component_policy_disabled_for_testing_ =
131 component_policy_disabled_for_testing;
131 } 132 }
132 133
133 private: 134 private:
134 // Saves the state keys received from |session_manager_client_|. 135 // Saves the state keys received from |session_manager_client_|.
135 void OnStateKeysUpdated(); 136 void OnStateKeysUpdated();
136 137
137 // Initializes requisition settings at OOBE with values from VPD. 138 // Initializes requisition settings at OOBE with values from VPD.
138 void InitializeRequisition(); 139 void InitializeRequisition();
139 140
140 void NotifyConnected(); 141 void NotifyConnected();
(...skipping 27 matching lines...) Expand all
168 PrefService* local_state_; 169 PrefService* local_state_;
169 170
170 std::unique_ptr<chromeos::attestation::AttestationPolicyObserver> 171 std::unique_ptr<chromeos::attestation::AttestationPolicyObserver>
171 attestation_policy_observer_; 172 attestation_policy_observer_;
172 173
173 // Wrapper schema registry that will track the signin profile schema registry 174 // Wrapper schema registry that will track the signin profile schema registry
174 // once it is passed to this class. 175 // once it is passed to this class.
175 std::unique_ptr<ForwardingSchemaRegistry> 176 std::unique_ptr<ForwardingSchemaRegistry>
176 signin_profile_forwarding_schema_registry_; 177 signin_profile_forwarding_schema_registry_;
177 178
178 // Whether the component cloud policy service should be created. 179 // Whether the component cloud policy should be disabled (by skipping the
179 // TODO(emaxx): Change the default to true once both the client and the 180 // component cloud policy service creation).
180 // DMServer are ready for handling policy fetches with the 181 bool component_policy_disabled_for_testing_ = false;
181 // google/chromeos/signinextension type. See crbug.com/666720,
182 // crbug.com/644304 for reference.
183 bool is_component_policy_enabled_ = false;
184 182
185 base::ObserverList<Observer, true> observers_; 183 base::ObserverList<Observer, true> observers_;
186 184
187 DISALLOW_COPY_AND_ASSIGN(DeviceCloudPolicyManagerChromeOS); 185 DISALLOW_COPY_AND_ASSIGN(DeviceCloudPolicyManagerChromeOS);
188 }; 186 };
189 187
190 } // namespace policy 188 } // namespace policy
191 189
192 #endif // CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_CLOUD_POLICY_MANAGER_CHROMEOS_H _ 190 #endif // CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_CLOUD_POLICY_MANAGER_CHROMEOS_H _
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698