Chromium Code Reviews| Index: chrome/browser/ui/webui/options/chromeos/system_options_handler.cc |
| diff --git a/chrome/browser/ui/webui/options/chromeos/system_options_handler.cc b/chrome/browser/ui/webui/options/chromeos/system_options_handler.cc |
| index 4f343f9bd24f434dfbe4e46067bef984d837cd12..434d45d4cacc68848fdc70c64f59ee9188610d1c 100644 |
| --- a/chrome/browser/ui/webui/options/chromeos/system_options_handler.cc |
| +++ b/chrome/browser/ui/webui/options/chromeos/system_options_handler.cc |
| @@ -15,6 +15,7 @@ |
| #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" |
| @@ -30,9 +31,7 @@ |
| #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() { |
| @@ -81,8 +80,9 @@ void SystemOptionsHandler::GetLocalizedValues( |
| 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
|
| } |
| void SystemOptionsHandler::Initialize() { |