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

Unified Diff: components/content_settings/core/browser/content_settings_pref.cc

Issue 2666093002: Remove base::FundamentalValue (Closed)
Patch Set: Rebase Created 3 years, 10 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: components/content_settings/core/browser/content_settings_pref.cc
diff --git a/components/content_settings/core/browser/content_settings_pref.cc b/components/content_settings/core/browser/content_settings_pref.cc
index df76df0657322578264e4aebb82128cf9745c732..f32e3018b4cc0828d4b5b7efa7a596d5f022d82b 100644
--- a/components/content_settings/core/browser/content_settings_pref.cc
+++ b/components/content_settings/core/browser/content_settings_pref.cc
@@ -252,8 +252,7 @@ void ContentSettingsPref::ReadContentSettingsFromPref() {
bool is_integer = j.value().GetAsInteger(&setting);
DCHECK(is_integer);
DCHECK_NE(CONTENT_SETTING_DEFAULT, setting);
- std::unique_ptr<base::Value> setting_ptr(
- new base::FundamentalValue(setting));
+ std::unique_ptr<base::Value> setting_ptr(new base::Value(setting));
value_map_.SetValue(pattern_pair.first,
pattern_pair.second,
content_type_,

Powered by Google App Engine
This is Rietveld 408576698