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

chrome/browser/automation/testing_automation_provider_chromeos.cc

Index: chrome/browser/automation/testing_automation_provider_chromeos.cc
diff --git a/chrome/browser/automation/testing_automation_provider_chromeos.cc b/chrome/browser/automation/testing_automation_provider_chromeos.cc
index a82a86c84082c4c9dff9d7a7034539521e0bf393..1d75ab8510b7cc85f931ed32ef38ce11df6df9a7 100644
--- a/chrome/browser/automation/testing_automation_provider_chromeos.cc
+++ b/chrome/browser/automation/testing_automation_provider_chromeos.cc
@@ -519,10 +519,10 @@ void TestingAutomationProvider::GetProxySettings(DictionaryValue* args,
 }
 
 void TestingAutomationProvider::SetProxySettings(DictionaryValue* args,
-                                                IPC::Message* reply_message) {
+                                                 IPC::Message* reply_message) {
   AutomationJSONReply reply(this, reply_message);
   std::string key;
-  Value* value;
+  base::Value* value;
   if (!args->GetString("key", &key) || !args->Get("value", &value)) {
     reply.SendError("Invalid or missing args.");
     return;
@@ -531,8 +531,7 @@ void TestingAutomationProvider::SetProxySettings(DictionaryValue* args,
   std::string setting_path = "cros.session.proxy.";
   setting_path.append(key);
 
-  // ProxyCrosSettingsProvider will own the Value* passed to Set().
-  chromeos::CrosSettings::Get()->Set(setting_path, value->DeepCopy());
+  chromeos::CrosSettings::Get()->Set(setting_path, *value);
   reply.SendSuccess(NULL);
 }