Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(368)

Unified Diff: chrome/browser/chromeos/settings/system_settings_provider.cc

Issue 2664753002: Remove base::StringValue (Closed)
Patch Set: Rebase Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/settings/system_settings_provider.cc
diff --git a/chrome/browser/chromeos/settings/system_settings_provider.cc b/chrome/browser/chromeos/settings/system_settings_provider.cc
index 3a0668f06bf39398068d978c8a4e4d2a9f8e5cf0..e636b91a50f57d49c30ddba7e394dd28162047d2 100644
--- a/chrome/browser/chromeos/settings/system_settings_provider.cc
+++ b/chrome/browser/chromeos/settings/system_settings_provider.cc
@@ -18,8 +18,8 @@ SystemSettingsProvider::SystemSettingsProvider(
system::TimezoneSettings *timezone_settings =
system::TimezoneSettings::GetInstance();
timezone_settings->AddObserver(this);
- timezone_value_.reset(new base::StringValue(
- timezone_settings->GetCurrentTimezoneID()));
+ timezone_value_.reset(
+ new base::Value(timezone_settings->GetCurrentTimezoneID()));
}
SystemSettingsProvider::~SystemSettingsProvider() {
@@ -61,8 +61,8 @@ bool SystemSettingsProvider::HandlesSetting(const std::string& path) const {
void SystemSettingsProvider::TimezoneChanged(const icu::TimeZone& timezone) {
// Fires system setting change notification.
- timezone_value_.reset(new base::StringValue(
- system::TimezoneSettings::GetTimezoneID(timezone)));
+ timezone_value_.reset(
+ new base::Value(system::TimezoneSettings::GetTimezoneID(timezone)));
NotifyObservers(kSystemTimezone);
}

Powered by Google App Engine
This is Rietveld 408576698