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

Unified Diff: components/content_settings/core/browser/content_settings_policy_provider.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_policy_provider.cc
diff --git a/components/content_settings/core/browser/content_settings_policy_provider.cc b/components/content_settings/core/browser/content_settings_policy_provider.cc
index 81a059b6e0ddab617c1100274ba45f552bbdb3db..c2967aa41a68010245b9793394eef5705b3f06b6 100644
--- a/components/content_settings/core/browser/content_settings_policy_provider.cc
+++ b/components/content_settings/core/browser/content_settings_policy_provider.cc
@@ -235,10 +235,10 @@ void PolicyProvider::GetContentSettingsFromPreferences(
VLOG_IF(2, !pattern_pair.second.IsValid())
<< "Replacing invalid secondary pattern '"
<< pattern_pair.second.ToString() << "' with wildcard";
- value_map->SetValue(pattern_pair.first, secondary_pattern, content_type,
- ResourceIdentifier(),
- new base::FundamentalValue(
- kPrefsForManagedContentSettingsMap[i].setting));
+ value_map->SetValue(
+ pattern_pair.first, secondary_pattern, content_type,
+ ResourceIdentifier(),
+ new base::Value(kPrefsForManagedContentSettingsMap[i].setting));
}
}
}
@@ -358,7 +358,7 @@ void PolicyProvider::UpdateManagedDefaultSetting(
} else {
value_map_.SetValue(ContentSettingsPattern::Wildcard(),
ContentSettingsPattern::Wildcard(), entry.content_type,
- std::string(), new base::FundamentalValue(setting));
+ std::string(), new base::Value(setting));
}
}

Powered by Google App Engine
This is Rietveld 408576698