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

Unified Diff: chrome/browser/content_settings/content_settings_policy_provider_unittest.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: chrome/browser/content_settings/content_settings_policy_provider_unittest.cc
diff --git a/chrome/browser/content_settings/content_settings_policy_provider_unittest.cc b/chrome/browser/content_settings/content_settings_policy_provider_unittest.cc
index 54363f202253f73fc71a45849a5e7fa43917a57b..23b0938d90f06a5174d4cd495efde8ca0e605854 100644
--- a/chrome/browser/content_settings/content_settings_policy_provider_unittest.cc
+++ b/chrome/browser/content_settings/content_settings_policy_provider_unittest.cc
@@ -48,7 +48,7 @@ TEST_F(PolicyProviderTest, DefaultGeolocationContentSetting) {
// Change the managed value of the default geolocation setting
prefs->SetManagedPref(prefs::kManagedDefaultGeolocationSetting,
- new base::FundamentalValue(CONTENT_SETTING_BLOCK));
+ new base::Value(CONTENT_SETTING_BLOCK));
rule_iterator = provider.GetRuleIterator(CONTENT_SETTINGS_TYPE_GEOLOCATION,
std::string(), false);
@@ -71,7 +71,7 @@ TEST_F(PolicyProviderTest, ManagedDefaultContentSettings) {
PolicyProvider provider(prefs);
prefs->SetManagedPref(prefs::kManagedDefaultPluginsSetting,
- new base::FundamentalValue(CONTENT_SETTING_BLOCK));
+ new base::Value(CONTENT_SETTING_BLOCK));
std::unique_ptr<RuleIterator> rule_iterator(provider.GetRuleIterator(
CONTENT_SETTINGS_TYPE_PLUGINS, std::string(), false));
@@ -98,9 +98,9 @@ TEST_F(PolicyProviderTest, ManagedDefaultPluginSettingsExperiment) {
// ForceDefaultPluginsSettingAsk overrides this to ASK.
prefs->SetManagedPref(prefs::kManagedDefaultPluginsSetting,
- new base::FundamentalValue(CONTENT_SETTING_BLOCK));
+ new base::Value(CONTENT_SETTING_BLOCK));
prefs->SetManagedPref(prefs::kManagedDefaultJavaScriptSetting,
- new base::FundamentalValue(CONTENT_SETTING_BLOCK));
+ new base::Value(CONTENT_SETTING_BLOCK));
std::unique_ptr<RuleIterator> plugin_rule_iterator(provider.GetRuleIterator(
CONTENT_SETTINGS_TYPE_PLUGINS, std::string(), false));
@@ -140,7 +140,7 @@ TEST_F(PolicyProviderTest, ObserveManagedSettingsChange) {
// Set the managed default-content-setting.
prefs->SetManagedPref(prefs::kManagedDefaultImagesSetting,
- new base::FundamentalValue(CONTENT_SETTING_BLOCK));
+ new base::Value(CONTENT_SETTING_BLOCK));
::testing::Mock::VerifyAndClearExpectations(&mock_observer);
EXPECT_CALL(mock_observer,
OnContentSettingChanged(_,
@@ -193,7 +193,7 @@ TEST_F(PolicyProviderTest, GettingManagedContentSettings) {
// enforced via policies and not set by the user or extension. So a call to
// SetWebsiteSetting does nothing.
std::unique_ptr<base::Value> value_block(
- new base::FundamentalValue(CONTENT_SETTING_BLOCK));
+ new base::Value(CONTENT_SETTING_BLOCK));
bool owned = provider.SetWebsiteSetting(yt_url_pattern,
yt_url_pattern,
CONTENT_SETTINGS_TYPE_COOKIES,

Powered by Google App Engine
This is Rietveld 408576698