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

chrome/browser/chromeos/cros_settings.cc

Index: chrome/browser/chromeos/cros_settings.cc
diff --git a/chrome/browser/chromeos/cros_settings.cc b/chrome/browser/chromeos/cros_settings.cc
index 7162907fe0a1679e1cbad666d322872c1c073f01..295f1e00f016dd997750a39b769be19ac979fe42 100644
--- a/chrome/browser/chromeos/cros_settings.cc
+++ b/chrome/browser/chromeos/cros_settings.cc
@@ -47,7 +47,7 @@ void CrosSettings::FireObservers(const char* path) {
   }
 }
 
-void CrosSettings::Set(const std::string& path, Value* in_value) {
+void CrosSettings::Set(const std::string& path, const base::Value& in_value) {
   DCHECK(CalledOnValidThread());
   CrosSettingsProvider* provider;
   provider = GetProvider(path);
@@ -58,23 +58,49 @@ void CrosSettings::Set(const std::string& path, Value* in_value) {
 
 void CrosSettings::SetBoolean(const std::string& path, bool in_value) {
   DCHECK(CalledOnValidThread());
-  Set(path, Value::CreateBooleanValue(in_value));
+  base::FundamentalValue value(in_value);
+  Set(path, value);
 }
 
 void CrosSettings::SetInteger(const std::string& path, int in_value) {
   DCHECK(CalledOnValidThread());
-  Set(path, Value::CreateIntegerValue(in_value));
+  base::FundamentalValue value(in_value);
+  Set(path, value);
 }
 
 void CrosSettings::SetDouble(const std::string& path, double in_value) {
   DCHECK(CalledOnValidThread());
-  Set(path, Value::CreateDoubleValue(in_value));
+  base::FundamentalValue value(in_value);
+  Set(path, value);
 }
 
 void CrosSettings::SetString(const std::string& path,
                              const std::string& in_value) {
   DCHECK(CalledOnValidThread());
-  Set(path, Value::CreateStringValue(in_value));
+  base::StringValue value(in_value);
+  Set(path, value);
+}
+
+void CrosSettings::AppendToList(const std::string& path,
+                                const base::Value& value) {
+  DCHECK(CalledOnValidThread());
+  const base::Value* old_value = GetPref(path);
+  scoped_ptr<base::Value> new_value(new base::ListValue());

[Mattias Nissler (ping if slow), 2011/10/07 11:02:57]

Allocating an object and throwing it away immediately if there's an existing
one? Seems weird.

[pastarmovj, 2011/10/13 11:25:06]

Done.

+  if (old_value)
+    new_value.reset(old_value->DeepCopy());
+  static_cast<base::ListValue*>(new_value.get())->Append(value.DeepCopy());
+  Set(path, *new_value);
+}
+
+void CrosSettings::RemoveFromList(const std::string& path,
+                                  const base::Value& value) {
+  DCHECK(CalledOnValidThread());
+  const base::Value* old_value = GetPref(path);
+  scoped_ptr<base::Value> new_value(new base::ListValue());

[Mattias Nissler (ping if slow), 2011/10/07 11:02:57]

ditto.

[pastarmovj, 2011/10/13 11:25:06]

Done.

+  if (old_value)
+    new_value.reset(old_value->DeepCopy());
+  static_cast<base::ListValue*>(new_value.get())->Remove(value, NULL);
+  Set(path, *new_value);
 }
 
 bool CrosSettings::AddSettingsProvider(CrosSettingsProvider* provider) {