PART1: Initiated the SignedSettings refactoring.

chrome/browser/automation/testing_automation_provider_chromeos.cc

 // 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.
 
 #include "chrome/browser/automation/testing_automation_provider.h"
 
 #include "base/command_line.h"
 #include "base/i18n/time_formatting.h"
 #include "base/stringprintf.h"
 #include "base/time.h"
@@ skipping 50 matching lines @@
 
 DictionaryValue* GetNetworkInfoDict(const chromeos::Network* network) {
   DictionaryValue* item = new DictionaryValue;
   item->SetString("name", network->name());
   item->SetString("device_path", network->device_path());
   item->SetString("ip_address", network->ip_address());
   item->SetString("status", network->GetStateString());
   return item;
 }
 
-Value* GetProxySetting(const std::string& setting_name) {
+const Value* GetProxySetting(const std::string& setting_name) {
   chromeos::ProxyCrosSettingsProvider settings_provider;

[Mattias Nissler (ping if slow), 2011/09/21 11:12:59]

why aren't we going through CrosSettings here?

[pastarmovj, 2011/09/23 15:19:32]

Done.

   std::string setting_path = "cros.session.proxy.";
   setting_path.append(setting_name);
 
-  if (setting_name == "ignorelist") {
-    Value* value;
-    if (settings_provider.Get(setting_path, &value))
-      return value;
-  } else {
-    Value* setting;
-    if (settings_provider.Get(setting_path, &setting)) {
-      DictionaryValue* setting_dict = static_cast<DictionaryValue*>(setting);
-      Value* value;
-      bool found = setting_dict->Remove("value", &value);
-      delete setting;
-      if (found)
-        return value;
-    }
-  }
-  return NULL;
+  return settings_provider.Get(setting_path);
 }
 
 const char* UpdateStatusToString(chromeos::UpdateStatusOperation status) {
   switch (status) {
     case chromeos::UPDATE_STATUS_IDLE:
       return "idle";
     case chromeos::UPDATE_STATUS_CHECKING_FOR_UPDATE:
       return "checking for update";
     case chromeos::UPDATE_STATUS_UPDATE_AVAILABLE:
       return "update available";
@@ skipping 419 matching lines @@
                                                  IPC::Message* reply_message) {
   const char* settings[] = { "pacurl", "singlehttp", "singlehttpport",
                              "httpurl", "httpport", "httpsurl", "httpsport",
                              "type", "single", "ftpurl", "ftpport",
                              "socks", "socksport", "ignorelist" };
 
   scoped_ptr<DictionaryValue> return_value(new DictionaryValue);
   chromeos::ProxyCrosSettingsProvider settings_provider;
 
   for (size_t i = 0; i < arraysize(settings); ++i) {
-    Value* setting = GetProxySetting(settings[i]);
+    const Value* setting = GetProxySetting(settings[i]);
     if (setting)
-      return_value->Set(settings[i], setting);
+      return_value->Set(settings[i], setting->DeepCopy());
   }
 
   AutomationJSONReply(this, reply_message).SendSuccess(return_value.get());
 }
 
 void TestingAutomationProvider::SetProxySettings(DictionaryValue* args,
                                                 IPC::Message* reply_message) {
   AutomationJSONReply reply(this, reply_message);
   std::string key;
   Value* value;
@@ skipping 592 matching lines @@
 
   // Set up an observer (it will delete itself).
   take_photo_dialog->AddObserver(new PhotoCaptureObserver(
       this, reply_message));
 
   views::Widget* window = browser::CreateViewsWindow(
       browser->window()->GetNativeHandle(), take_photo_dialog);
   window->SetAlwaysOnTop(true);
   window->Show();
 }