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

Unified Diff: chrome/browser/chromeos/settings/session_manager_operation.h

Issue 769703003: SetManagementSettings() is moved to OwnerSettingsServiceChromeOS. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanup. Created 6 years 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/chromeos/settings/session_manager_operation.h
diff --git a/chrome/browser/chromeos/settings/session_manager_operation.h b/chrome/browser/chromeos/settings/session_manager_operation.h
index 356025e9514612c2d9204a1a266c1e21b8078041..6c3fe0c50120a5a9d6d6272e60b2b0c03b193e70 100644
--- a/chrome/browser/chromeos/settings/session_manager_operation.h
+++ b/chrome/browser/chromeos/settings/session_manager_operation.h
@@ -11,7 +11,6 @@
#include "base/memory/scoped_ptr.h"
#include "chrome/browser/chromeos/policy/device_cloud_policy_validator.h"
#include "chrome/browser/chromeos/settings/device_settings_service.h"
-#include "components/ownership/owner_settings_service.h"
#include "net/cert/x509_util_nss.h"
namespace enterprise_management {
@@ -68,13 +67,6 @@ class SessionManagerOperation {
force_key_load_ = force_key_load;
}
- void set_username(const std::string& username) { username_ = username; }
-
- void set_owner_settings_service(const base::WeakPtr<
- ownership::OwnerSettingsService>& owner_settings_service) {
- owner_settings_service_ = owner_settings_service;
- }
-
protected:
// Runs the operation. The result is reported through |callback_|.
virtual void Run() = 0;
@@ -93,8 +85,6 @@ class SessionManagerOperation {
return session_manager_client_;
}
- base::WeakPtr<ownership::OwnerSettingsService> owner_settings_service_;
-
private:
// Loads the owner key from disk. Must be run on a thread that can do I/O.
static scoped_refptr<ownership::PublicKey> LoadPublicKey(
@@ -121,7 +111,6 @@ class SessionManagerOperation {
scoped_refptr<ownership::PublicKey> public_key_;
bool force_key_load_;
- std::string username_;
bool is_loading_;
scoped_ptr<enterprise_management::PolicyData> policy_data_;
@@ -173,35 +162,6 @@ class StoreSettingsOperation : public SessionManagerOperation {
DISALLOW_COPY_AND_ASSIGN(StoreSettingsOperation);
};
-// Signs device settings and stores the resulting blob to session_manager.
-class SignAndStoreSettingsOperation : public SessionManagerOperation {
- public:
- // Creates a new sign-and-store operation.
- SignAndStoreSettingsOperation(
- const Callback& callback,
- scoped_ptr<enterprise_management::PolicyData> new_policy);
- virtual ~SignAndStoreSettingsOperation();
-
- // SessionManagerOperation:
- virtual void Run() override;
-
- private:
- void StartSigning(bool has_private_key);
-
- // Stores the signed device settings blob.
- void StoreDeviceSettings(
- scoped_ptr<enterprise_management::PolicyFetchResponse> policy_response);
-
- // Handles the result of the store operation and triggers the load.
- void HandleStoreResult(bool success);
-
- scoped_ptr<enterprise_management::PolicyData> new_policy_;
-
- base::WeakPtrFactory<SignAndStoreSettingsOperation> weak_factory_;
-
- DISALLOW_COPY_AND_ASSIGN(SignAndStoreSettingsOperation);
-};
-
} // namespace chromeos
#endif // CHROME_BROWSER_CHROMEOS_SETTINGS_SESSION_MANAGER_OPERATION_H_

Powered by Google App Engine
This is Rietveld 408576698