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

Unified Diff: components/content_settings/core/browser/content_settings_registry.h

Issue 1902703002: Only Register() desktop-only content settings types on desktop platforms (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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_registry.h
diff --git a/components/content_settings/core/browser/content_settings_registry.h b/components/content_settings/core/browser/content_settings_registry.h
index bfa532fae656a8456e0796acef4f7a3decf938ad..0b9873e1cae609632930c2ff02a9b00f4ad5563f 100644
--- a/components/content_settings/core/browser/content_settings_registry.h
+++ b/components/content_settings/core/browser/content_settings_registry.h
@@ -64,6 +64,19 @@ class ContentSettingsRegistry {
WebsiteSettingsInfo::ScopingType scoping_type,
ContentSettingsInfo::IncognitoBehavior incognito_behavior);
+ // Register a content setting that is only used on desktop platforms. It won't
+ // be registered on other platforms meaning that type information won't be
+ // available, a pref won't be registered, etc.
+ void RegisterDesktop(
+ ContentSettingsType type,
+ const std::string& name,
+ ContentSetting initial_default_value,
+ WebsiteSettingsInfo::SyncStatus sync_status,
+ const std::vector<std::string>& whitelisted_schemes,
+ const std::set<ContentSetting>& valid_settings,
+ WebsiteSettingsInfo::ScopingType scoping_type,
+ ContentSettingsInfo::IncognitoBehavior incognito_behavior);
+
Map content_settings_info_;
WebsiteSettingsRegistry* website_settings_registry_;

Powered by Google App Engine
This is Rietveld 408576698