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

Unified Diff: chrome/browser/content_settings/content_settings_policy_provider.cc

Issue 385263004: Get rid of some uses of CreateIntegerValue (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 5 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.cc
diff --git a/chrome/browser/content_settings/content_settings_policy_provider.cc b/chrome/browser/content_settings/content_settings_policy_provider.cc
index c5f3348ff59f338a2cd97ad32f39ba4ff6dcdad3..371a0b6aad5344f7ce65bdf1f5f41ac08ee74b16 100644
--- a/chrome/browser/content_settings/content_settings_policy_provider.cc
+++ b/chrome/browser/content_settings/content_settings_policy_provider.cc
@@ -289,13 +289,12 @@ void PolicyProvider::GetContentSettingsFromPreferences(
ContentSettingsPattern secondary_pattern =
!pattern_pair.second.IsValid() ? ContentSettingsPattern::Wildcard()
: pattern_pair.second;
- value_map->SetValue(
- pattern_pair.first,
- secondary_pattern,
- content_type,
- NO_RESOURCE_IDENTIFIER,
- base::Value::CreateIntegerValue(
- kPrefsForManagedContentSettingsMap[i].setting));
+ value_map->SetValue(pattern_pair.first,
+ secondary_pattern,
+ content_type,
+ NO_RESOURCE_IDENTIFIER,
+ new base::FundamentalValue(
+ kPrefsForManagedContentSettingsMap[i].setting));
}
}
}
@@ -413,12 +412,11 @@ void PolicyProvider::UpdateManagedDefaultSetting(
content_type,
std::string());
} else {
- value_map_.SetValue(
- ContentSettingsPattern::Wildcard(),
- ContentSettingsPattern::Wildcard(),
- content_type,
- std::string(),
- base::Value::CreateIntegerValue(setting));
+ value_map_.SetValue(ContentSettingsPattern::Wildcard(),
+ ContentSettingsPattern::Wildcard(),
+ content_type,
+ std::string(),
+ new base::FundamentalValue(setting));
}
}

Powered by Google App Engine
This is Rietveld 408576698