Index: chrome/browser/chromeos/ownership/owner_settings_service_chromeos.h |
diff --git a/chrome/browser/chromeos/ownership/owner_settings_service.h b/chrome/browser/chromeos/ownership/owner_settings_service_chromeos.h |
similarity index 51% |
rename from chrome/browser/chromeos/ownership/owner_settings_service.h |
rename to chrome/browser/chromeos/ownership/owner_settings_service_chromeos.h |
index 3419d4ff1b3e7dbee4934d40695b9576f55c34be..fcbaf57f825d5335a6a61b438fe7379ee7b9a5f0 100644 |
--- a/chrome/browser/chromeos/ownership/owner_settings_service.h |
+++ b/chrome/browser/chromeos/ownership/owner_settings_service_chromeos.h |
@@ -2,27 +2,28 @@ |
// Use of this source code is governed by a BSD-style license that can be |
// found in the LICENSE file. |
-#ifndef CHROME_BROWSER_CHROMEOS_OWNERSHIP_OWNER_SETTINGS_SERVICE_H_ |
-#define CHROME_BROWSER_CHROMEOS_OWNERSHIP_OWNER_SETTINGS_SERVICE_H_ |
+#ifndef CHROME_BROWSER_CHROMEOS_OWNERSHIP_OWNER_SETTINGS_SERVICE_CHROMEOS_H_ |
+#define CHROME_BROWSER_CHROMEOS_OWNERSHIP_OWNER_SETTINGS_SERVICE_CHROMEOS_H_ |
#include <deque> |
#include <vector> |
-#include "base/callback.h" |
+#include "base/callback_forward.h" |
#include "base/compiler_specific.h" |
#include "base/macros.h" |
-#include "base/memory/weak_ptr.h" |
-#include "base/threading/thread_checker.h" |
#include "chrome/browser/chromeos/settings/device_settings_service.h" |
#include "chromeos/dbus/session_manager_client.h" |
#include "chromeos/tpm_token_loader.h" |
-#include "components/keyed_service/core/keyed_service.h" |
-#include "components/ownership/owner_key_util.h" |
+#include "components/ownership/owner_settings_service.h" |
#include "content/public/browser/notification_observer.h" |
#include "content/public/browser/notification_registrar.h" |
class Profile; |
+namespace ownership { |
+class OwnerKeyUtil; |
+} |
+ |
namespace chromeos { |
class SessionManagerOperation; |
@@ -31,31 +32,16 @@ class SessionManagerOperation; |
// |
// TODO (ygorshenin@): move write path for device settings here |
// (crbug.com/230018). |
-class OwnerSettingsService : public DeviceSettingsService::PrivateKeyDelegate, |
- public KeyedService, |
- public content::NotificationObserver, |
- public TPMTokenLoader::Observer, |
- public SessionManagerClient::Observer { |
+class OwnerSettingsServiceChromeOS : public ownership::OwnerSettingsService, |
+ public content::NotificationObserver, |
+ public TPMTokenLoader::Observer, |
+ public SessionManagerClient::Observer { |
public: |
- virtual ~OwnerSettingsService(); |
- |
- base::WeakPtr<OwnerSettingsService> as_weak_ptr() { |
- return weak_factory_.GetWeakPtr(); |
- } |
+ virtual ~OwnerSettingsServiceChromeOS(); |
- // DeviceSettingsService::PrivateKeyDelegate implementation: |
- virtual bool IsOwner() OVERRIDE; |
- virtual void IsOwnerAsync(const IsOwnerCallback& callback) OVERRIDE; |
- virtual bool AssembleAndSignPolicyAsync( |
+ // ownership::OwnerSettingsService implementation: |
+ virtual void SignAndStorePolicyAsync( |
scoped_ptr<enterprise_management::PolicyData> policy, |
- const AssembleAndSignPolicyCallback& callback) OVERRIDE; |
- virtual void SignAndStoreAsync( |
- scoped_ptr<enterprise_management::ChromeDeviceSettingsProto> settings, |
- const base::Closure& callback) OVERRIDE; |
- virtual void SetManagementSettingsAsync( |
- enterprise_management::PolicyData::ManagementMode management_mode, |
- const std::string& request_token, |
- const std::string& device_id, |
const base::Closure& callback) OVERRIDE; |
// NotificationObserver implementation: |
@@ -81,23 +67,22 @@ class OwnerSettingsService : public DeviceSettingsService::PrivateKeyDelegate, |
DeviceSettingsService* device_settings_service); |
private: |
- friend class OwnerSettingsServiceFactory; |
+ friend class OwnerSettingsServiceChromeOSFactory; |
- OwnerSettingsService( |
+ OwnerSettingsServiceChromeOS( |
Profile* profile, |
const scoped_refptr<ownership::OwnerKeyUtil>& owner_key_util); |
- // Reloads private key from profile's NSS slots. Responds via call |
- // to OnPrivateKeyLoaded(). |
- void ReloadPrivateKey(); |
+ // OwnerSettingsService protected interface overrides: |
- // Called when ReloadPrivateKey() completes it's work. |
- void OnPrivateKeyLoaded(scoped_refptr<ownership::PublicKey> public_key, |
- scoped_refptr<ownership::PrivateKey> private_key); |
+ // Reloads private key from profile's NSS slots, responds via |callback|. |
+ virtual void ReloadKeypairImpl(const base::Callback< |
+ void(const scoped_refptr<ownership::PublicKey>& public_key, |
+ const scoped_refptr<ownership::PrivateKey>& private_key)>& callback) |
+ OVERRIDE; |
- // Puts request to perform sign-and-store operation in the queue. |
- void EnqueueSignAndStore(scoped_ptr<enterprise_management::PolicyData> policy, |
- const base::Closure& callback); |
+ // Possibly notifies DeviceSettingsService that owner's keypair is loaded. |
+ virtual void OnPostKeypairLoadedActions() OVERRIDE; |
// Performs next operation in the queue. |
void StartNextOperation(); |
@@ -107,33 +92,12 @@ class OwnerSettingsService : public DeviceSettingsService::PrivateKeyDelegate, |
SessionManagerOperation* operation, |
DeviceSettingsService::Status status); |
- // Called when it's not possible to store settings. |
- void HandleError(DeviceSettingsService::Status status, |
- const base::Closure& callback); |
- |
- // Returns testing instance of OwnerKeyUtil when it's set, otherwise |
- // returns |owner_key_util_|. |
- scoped_refptr<ownership::OwnerKeyUtil> GetOwnerKeyUtil(); |
- |
- // Returns testing instance of DeviceSettingsService when it's set, |
- // otherwise returns pointer to a singleton instance, when it's |
- // initialized. |
- DeviceSettingsService* GetDeviceSettingsService(); |
- |
// Profile this service instance belongs to. |
Profile* profile_; |
// User ID this service instance belongs to. |
std::string user_id_; |
- scoped_refptr<ownership::PublicKey> public_key_; |
- |
- scoped_refptr<ownership::PrivateKey> private_key_; |
- |
- scoped_refptr<ownership::OwnerKeyUtil> owner_key_util_; |
- |
- std::vector<IsOwnerCallback> pending_is_owner_callbacks_; |
- |
// Whether profile still needs to be initialized. |
bool waiting_for_profile_creation_; |
@@ -146,13 +110,11 @@ class OwnerSettingsService : public DeviceSettingsService::PrivateKeyDelegate, |
content::NotificationRegistrar registrar_; |
- base::ThreadChecker thread_checker_; |
- |
- base::WeakPtrFactory<OwnerSettingsService> weak_factory_; |
+ base::WeakPtrFactory<OwnerSettingsServiceChromeOS> weak_factory_; |
- DISALLOW_COPY_AND_ASSIGN(OwnerSettingsService); |
+ DISALLOW_COPY_AND_ASSIGN(OwnerSettingsServiceChromeOS); |
}; |
} // namespace chromeos |
-#endif // CHROME_BROWSER_CHROMEOS_OWNERSHIP_OWNER_SETTINGS_SERVICE_H_ |
+#endif // CHROME_BROWSER_CHROMEOS_OWNERSHIP_OWNER_SETTINGS_SERVICE_CHROMEOS_H_ |