Signed settings refactoring: Proper caching and more tests.

chrome/browser/chromeos/login/signed_settings_cache.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_CHROMEOS_LOGIN_SIGNED_SETTINGS_TEMP_STORAGE_H_
-#define CHROME_BROWSER_CHROMEOS_LOGIN_SIGNED_SETTINGS_TEMP_STORAGE_H_
+#ifndef CHROME_BROWSER_CHROMEOS_LOGIN_SIGNED_SETTINGS_CACHE_H_
+#define CHROME_BROWSER_CHROMEOS_LOGIN_SIGNED_SETTINGS_CACHE_H_
 #pragma once
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/values.h"
+#include "chrome/browser/policy/proto/chrome_device_policy.pb.h"
+#include "chrome/browser/policy/proto/device_management_backend.pb.h"
+
+namespace em = enterprise_management;
 
 class PrefService;
 
 namespace chromeos {
 
 // There is need (proxy settings at OOBE stage) to store settings
 // (that are normally go into SignedSettings storage)

[Mattias Nissler (ping if slow), 2011/11/30 11:22:44]

s/are//

[pastarmovj, 2011/11/30 17:21:16]

Done.

 // before owner has been assigned (hence no key is available).
 // This class serves as a transient storage in that case.
-class SignedSettingsTempStorage {
+class SignedSettingsCache {

[Mattias Nissler (ping if slow), 2011/11/30 11:22:44]

Only statics? Convert to namespace.

[pastarmovj, 2011/11/30 17:21:16]

Done.

  public:
   // Registers required pref section.
   static void RegisterPrefs(PrefService* local_state);
 
-  static bool Store(const std::string& name,
-                    const base::Value& value,
+  static bool Store(const em::PolicyData &policy,
                     PrefService* local_state);
-  static bool Retrieve(const std::string& name,
-                       base::Value** value,
+  static bool Retrieve(em::PolicyData *policy,
                        PrefService* local_state);
 
   // Call this after owner has been assigned to persist settings
   // into SignedSettings storage.
-  static void Finalize(PrefService* local_state);
+  static void Finalize(PrefService* local_state,
+                       const em::PolicyFetchResponse& policy);
 
  private:
-  SignedSettingsTempStorage() {}
-  DISALLOW_COPY_AND_ASSIGN(SignedSettingsTempStorage);
+  SignedSettingsCache() {}
+  DISALLOW_COPY_AND_ASSIGN(SignedSettingsCache);
 };
 
 }  // namespace chromeos
 
-#endif  // CHROME_BROWSER_CHROMEOS_LOGIN_SIGNED_SETTINGS_TEMP_STORAGE_H_
+#endif  // CHROME_BROWSER_CHROMEOS_LOGIN_SIGNED_SETTINGS_CACHE_H_