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

chrome/browser/chromeos/login/signed_settings.h

Index: chrome/browser/chromeos/login/signed_settings.h
diff --git a/chrome/browser/chromeos/login/signed_settings.h b/chrome/browser/chromeos/login/signed_settings.h
index a423aaed70ccd5234a6ca2e29bce56d7e5f9abc2..0b6623907aa616ee9abc83aca5d7c6ed064a57f2 100644
--- a/chrome/browser/chromeos/login/signed_settings.h
+++ b/chrome/browser/chromeos/login/signed_settings.h
@@ -10,6 +10,7 @@
 #include <vector>
 
 #include "base/memory/ref_counted.h"
+#include "base/values.h"

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

forward declaration should suffice?

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

Done.

 #include "chrome/browser/chromeos/login/owner_manager.h"
 
 // There are two categories of operations that can be performed on the
@@ -74,12 +75,12 @@ class SignedSettings : public base::RefCountedThreadSafe<SignedSettings>,
   // 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.