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

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

Issue 2621413007: Revert of Re-enable fetching component policies for login screen apps (Closed)
Patch Set: 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 should be disabled (by skipping 126 // Sets whether the component cloud policy service should be created.
127 // the component cloud policy service creation). 127 // Defaults to true.
128 void set_component_policy_disabled_for_testing( 128 void set_is_component_policy_enabled_for_testing(
129 bool component_policy_disabled_for_testing) { 129 bool is_component_policy_enabled) {
130 component_policy_disabled_for_testing_ = 130 is_component_policy_enabled_ = is_component_policy_enabled;
131 component_policy_disabled_for_testing;
132 } 131 }
133 132
134 private: 133 private:
135 // Saves the state keys received from |session_manager_client_|. 134 // Saves the state keys received from |session_manager_client_|.
136 void OnStateKeysUpdated(); 135 void OnStateKeysUpdated();
137 136
138 // Initializes requisition settings at OOBE with values from VPD. 137 // Initializes requisition settings at OOBE with values from VPD.
139 void InitializeRequisition(); 138 void InitializeRequisition();
140 139
141 void NotifyConnected(); 140 void NotifyConnected();
(...skipping 27 matching lines...) Expand all
169 PrefService* local_state_; 168 PrefService* local_state_;
170 169
171 std::unique_ptr<chromeos::attestation::AttestationPolicyObserver> 170 std::unique_ptr<chromeos::attestation::AttestationPolicyObserver>
172 attestation_policy_observer_; 171 attestation_policy_observer_;
173 172
174 // Wrapper schema registry that will track the signin profile schema registry 173 // Wrapper schema registry that will track the signin profile schema registry
175 // once it is passed to this class. 174 // once it is passed to this class.
176 std::unique_ptr<ForwardingSchemaRegistry> 175 std::unique_ptr<ForwardingSchemaRegistry>
177 signin_profile_forwarding_schema_registry_; 176 signin_profile_forwarding_schema_registry_;
178 177
179 // Whether the component cloud policy should be disabled (by skipping the 178 // Whether the component cloud policy service should be created.
180 // component cloud policy service creation). 179 // TODO(emaxx): Change the default to true once both the client and the
181 bool component_policy_disabled_for_testing_ = false; 180 // DMServer are ready for handling policy fetches with the
181 // google/chromeos/signinextension type. See crbug.com/666720,
182 // crbug.com/644304 for reference.
183 bool is_component_policy_enabled_ = false;
182 184
183 base::ObserverList<Observer, true> observers_; 185 base::ObserverList<Observer, true> observers_;
184 186
185 DISALLOW_COPY_AND_ASSIGN(DeviceCloudPolicyManagerChromeOS); 187 DISALLOW_COPY_AND_ASSIGN(DeviceCloudPolicyManagerChromeOS);
186 }; 188 };
187 189
188 } // namespace policy 190 } // namespace policy
189 191
190 #endif // CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_CLOUD_POLICY_MANAGER_CHROMEOS_H _ 192 #endif // CHROME_BROWSER_CHROMEOS_POLICY_DEVICE_CLOUD_POLICY_MANAGER_CHROMEOS_H _
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698