PART1: Initiated the SignedSettings refactoring.

chrome/browser/ui/webui/options/chromeos/system_options_handler.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/ui/webui/options/chromeos/system_options_handler.h"
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/common/json_value_serializer.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/accessibility_util.h"
+#include "chrome/browser/chromeos/cros_settings.h"
 #include "chrome/browser/chromeos/language_preferences.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/webui/options/chromeos/system_settings_provider.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/extensions/extension.h"
 #include "grit/browser_resources.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/locale_settings.h"
 #include "grit/theme_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 
-SystemOptionsHandler::SystemOptionsHandler()
-    : chromeos::CrosOptionsPageUIHandler(
-        new chromeos::SystemSettingsProvider()) {
+SystemOptionsHandler::SystemOptionsHandler() {
 }
 
 SystemOptionsHandler::~SystemOptionsHandler() {
 }
 
 void SystemOptionsHandler::GetLocalizedValues(
     DictionaryValue* localized_strings) {
   DCHECK(localized_strings);
 
   RegisterTitle(localized_strings, "systemPage", IDS_OPTIONS_SYSTEM_TAB_LABEL);
@@ skipping 28 matching lines @@
           IDS_OPTIONS_SETTINGS_LANGUAGES_MODIFIER_KEYS_CUSTOMIZE));
 
   localized_strings->SetString("accessibility_title",
       l10n_util::GetStringUTF16(
           IDS_OPTIONS_SETTINGS_SECTION_TITLE_ACCESSIBILITY));
   localized_strings->SetString("accessibility",
       l10n_util::GetStringUTF16(
           IDS_OPTIONS_SETTINGS_ACCESSIBILITY_DESCRIPTION));
 
   localized_strings->Set("timezoneList",
-      reinterpret_cast<chromeos::SystemSettingsProvider*>(
-          settings_provider_.get())->GetTimezoneList());
+      static_cast<chromeos::SystemSettingsProvider*>(
+          chromeos::CrosSettings::Get()->GetProvider(
+              chromeos::kSystemTimezone))->GetTimezoneList());

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

shouldn't this also go through CrosSettings?

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

I put a todo not to forget it but I think it will be better to do this in the CL
that will introduce some more list support for CrosSettings (the same needed to
wipe out Whitelist functions from UserCrosSettingsProvider).

 }
 
 void SystemOptionsHandler::Initialize() {
   DCHECK(web_ui_);
   PrefService* pref_service = g_browser_process->local_state();
   bool acc_enabled = pref_service->GetBoolean(prefs::kAccessibilityEnabled);
   base::FundamentalValue checked(acc_enabled);
   web_ui_->CallJavascriptFunction(
       "options.SystemOptions.SetAccessibilityCheckboxState", checked);
 }
 
 void SystemOptionsHandler::RegisterMessages() {
   DCHECK(web_ui_);
   web_ui_->RegisterMessageCallback("accessibilityChange",
       NewCallback(this, &SystemOptionsHandler::AccessibilityChangeCallback));
 }
 
 void SystemOptionsHandler::AccessibilityChangeCallback(const ListValue* args) {
   std::string checked_str;
   args->GetString(0, &checked_str);
   bool accessibility_enabled = (checked_str == "true");
 
   chromeos::accessibility::EnableAccessibility(accessibility_enabled, NULL);
 }