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

chrome/browser/chromeos/login/signed_settings_helper.h

Index: chrome/browser/chromeos/login/signed_settings_helper.h
diff --git a/chrome/browser/chromeos/login/signed_settings_helper.h b/chrome/browser/chromeos/login/signed_settings_helper.h
index 406ba7bb937ef396fc6dab07a6842ea9c638b769..25664006d5229fe8b32bdf66bac47d7d48b60903 100644
--- a/chrome/browser/chromeos/login/signed_settings_helper.h
+++ b/chrome/browser/chromeos/login/signed_settings_helper.h
@@ -8,6 +8,7 @@
 
 #include <string>
 
+#include "base/values.h"

[Mattias Nissler (ping if slow), 2011/11/18 14:24:42]

forward decl?

[pastarmovj, 2011/11/22 12:13:58]

Done.

 #include "chrome/browser/chromeos/login/signed_settings.h"
 
 namespace enterprise_management {
@@ -42,13 +43,13 @@ class SignedSettingsHelper {
     virtual void OnStorePropertyCompleted(
         SignedSettings::ReturnCode code,
         const std::string& name,
-        const std::string& value) {}
+        const base::Value& value) {}
 
     // Callback of RetrievePropertyOp.
     virtual void OnRetrievePropertyCompleted(
         SignedSettings::ReturnCode code,
         const std::string& name,
-        const std::string& value) {}
+        const base::Value* value) {}
 
     // Callback of StorePolicyOp.
     virtual void OnStorePolicyCompleted(
@@ -70,7 +71,7 @@ class SignedSettingsHelper {
                                 bool add_to_whitelist,
                                 Callback* callback) = 0;
   virtual void StartStorePropertyOp(const std::string& name,
-                                    const std::string& value,
+                                    const base::Value& value,
                                     Callback* callback) = 0;
   virtual void StartRetrieveProperty(const std::string& name,
                                      Callback* callback) = 0;