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

Unified Diff: chrome/browser/android/preferences/pref_service_bridge.cc

Issue 2785913002: [subresource_filter] Add the Content Settings UI for Android (Closed)
Patch Set: git cl try Created 3 years, 9 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/android/preferences/pref_service_bridge.cc
diff --git a/chrome/browser/android/preferences/pref_service_bridge.cc b/chrome/browser/android/preferences/pref_service_bridge.cc
index ea8b02f8299c0108cdcca023e62d1d30eb473af8..11282c80e0666039472b920f87aee3998cd1cba0 100644
--- a/chrome/browser/android/preferences/pref_service_bridge.cc
+++ b/chrome/browser/android/preferences/pref_service_bridge.cc
@@ -143,7 +143,8 @@ static jboolean IsContentSettingEnabled(JNIEnv* env,
// Before we migrate functions over to this central function, we must verify
// that the functionality provided below is correct.
DCHECK(content_settings_type == CONTENT_SETTINGS_TYPE_JAVASCRIPT ||
- content_settings_type == CONTENT_SETTINGS_TYPE_POPUPS);
+ content_settings_type == CONTENT_SETTINGS_TYPE_POPUPS ||
+ content_settings_type == CONTENT_SETTINGS_TYPE_SUBRESOURCE_FILTER);
ContentSettingsType type =
static_cast<ContentSettingsType>(content_settings_type);
return GetBooleanForContentSetting(type);
@@ -156,7 +157,8 @@ static void SetContentSettingEnabled(JNIEnv* env,
// Before we migrate functions over to this central function, we must verify
// that the new category supports ALLOW/BLOCK pairs and, if not, handle them.
DCHECK(content_settings_type == CONTENT_SETTINGS_TYPE_JAVASCRIPT ||
- content_settings_type == CONTENT_SETTINGS_TYPE_POPUPS);
+ content_settings_type == CONTENT_SETTINGS_TYPE_POPUPS ||
+ content_settings_type == CONTENT_SETTINGS_TYPE_SUBRESOURCE_FILTER);
HostContentSettingsMap* host_content_settings_map =
HostContentSettingsMapFactory::GetForProfile(GetOriginalProfile());

Powered by Google App Engine
This is Rietveld 408576698