PART2: Make SignedSettings use proper Value types instead of string all around the place.

chrome/browser/chromeos/login/signed_settings.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_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_SIGNED_SETTINGS_H_
 #pragma once
 
 #include <string>
 #include <vector>
@@ skipping 12 matching lines @@
 // I've filed an issue on me to remove that restriction.
 // http://code.google.com/p/chromium-os/issues/detail?id=6415
 
 // The pattern of use here is that the caller instantiates some
 // subclass of SignedSettings by calling one of the create
 // methods. Then, call Execute() on this object from the UI
 // thread. It'll go off and do work (on the FILE thread and over DBus),
 // and then call the appropriate method of the Delegate you passed in
 // -- again, on the UI thread.
 
+namespace base {
+class Value;
+}  // namespace base
+
 namespace enterprise_management {
 class PolicyFetchResponse;
 class PolicyData;
 }  // namespace enterprise_management
 namespace em = enterprise_management;
 
 namespace chromeos {
 class OwnershipService;
 
 class SignedSettings : public base::RefCountedThreadSafe<SignedSettings>,
@@ skipping 24 matching lines @@
       SignedSettings::Delegate<bool>* d);
 
   static SignedSettings* CreateWhitelistOp(const std::string& email,
                                            bool add_to_whitelist,
                                            SignedSettings::Delegate<bool>* d);
 
   // These are both "property" operations, and only one instance of
   // one type can be in flight at a time.
   static SignedSettings* CreateStorePropertyOp(
       const std::string& name,
-      const std::string& value,
+      const base::Value& value,
       SignedSettings::Delegate<bool>* d);
 
   static SignedSettings* CreateRetrievePropertyOp(
       const std::string& name,
-      SignedSettings::Delegate<std::string>* d);
+      SignedSettings::Delegate<const base::Value*>* d);
 
   // These are both "policy" operations, and only one instance of
   // one type can be in flight at a time.
   static SignedSettings* CreateStorePolicyOp(
       em::PolicyFetchResponse* policy,
       SignedSettings::Delegate<bool>* d);
 
   static SignedSettings* CreateRetrievePolicyOp(
       SignedSettings::Delegate<const em::PolicyFetchResponse&>* d);
 
@@ skipping 41 matching lines @@
   static const char kDevicePolicyType[];
 
   scoped_ptr<Relay> relay_;
   scoped_refptr<SignedSettings> polfetcher_;
   DISALLOW_COPY_AND_ASSIGN(SignedSettings);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_SIGNED_SETTINGS_H_