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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleWebsitePreferences.java

Issue 1068223002: Implement Site Settings \ Images category. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address feedback + cleanup Created 5 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: chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleWebsitePreferences.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleWebsitePreferences.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleWebsitePreferences.java
index dbb3d88e957119edf6c313e1e3bf16c624e7579e..0609b60ce306f8603e5f11afcac8730270fa29d8 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleWebsitePreferences.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleWebsitePreferences.java
@@ -54,6 +54,7 @@ public class SingleWebsitePreferences extends PreferenceFragment
// Website permissions (if adding new, see hasPermissionsPreferences and resetSite below):
public static final String PREF_COOKIES_PERMISSION = "cookies_permission_list";
public static final String PREF_FULLSCREEN_PERMISSION = "fullscreen_permission_list";
+ public static final String PREF_IMAGES_PERMISSION = "images_permission_list";
public static final String PREF_JAVASCRIPT_PERMISSION = "javascript_permission_list";
public static final String PREF_LOCATION_ACCESS = "location_access_list";
public static final String PREF_MIDI_SYSEX_PERMISSION = "midi_sysex_permission_list";
@@ -70,6 +71,7 @@ public class SingleWebsitePreferences extends PreferenceFragment
private static final String[] PERMISSION_PREFERENCE_KEYS = {
PREF_COOKIES_PERMISSION,
PREF_FULLSCREEN_PERMISSION,
+ PREF_IMAGES_PERMISSION,
PREF_JAVASCRIPT_PERMISSION,
PREF_LOCATION_ACCESS,
PREF_MIDI_SYSEX_PERMISSION,
@@ -217,8 +219,8 @@ public class SingleWebsitePreferences extends PreferenceFragment
// supports reading the origin or host.
// TODO(mvanouwerkerk): Merge in PopupExceptionInfo? It uses a pattern, and is never
// set on Android.
- // TODO(mvanouwerkerk): Merge in JavaScriptExceptionInfo? It uses a pattern, and is
- // never set on Android.
+ // TODO(mvanouwerkerk): Merge in JavaScriptExceptionInfo? It uses a pattern.
+ // TODO(mvanouwerkerk): Merge in ImagesExceptionInfo? It uses a pattern.
}
}
return merged;
@@ -261,6 +263,8 @@ public class SingleWebsitePreferences extends PreferenceFragment
setUpListPreference(preference, mSite.getCookiePermission());
} else if (PREF_FULLSCREEN_PERMISSION.equals(preference.getKey())) {
setUpListPreference(preference, mSite.getFullscreenPermission());
+ } else if (PREF_IMAGES_PERMISSION.equals(preference.getKey())) {
+ setUpListPreference(preference, mSite.getImagesPermission());
} else if (PREF_JAVASCRIPT_PERMISSION.equals(preference.getKey())) {
setUpListPreference(preference, mSite.getJavaScriptPermission());
} else if (PREF_LOCATION_ACCESS.equals(preference.getKey())) {
@@ -346,6 +350,9 @@ public class SingleWebsitePreferences extends PreferenceFragment
} else if (PREF_FULLSCREEN_PERMISSION.equals(preferenceKey)) {
return Website.PermissionDataEntry.getPermissionDataEntry(
ContentSettingsType.CONTENT_SETTINGS_TYPE_FULLSCREEN);
+ } else if (PREF_IMAGES_PERMISSION.equals(preferenceKey)) {
+ return Website.PermissionDataEntry.getPermissionDataEntry(
+ ContentSettingsType.CONTENT_SETTINGS_TYPE_IMAGES);
} else if (PREF_JAVASCRIPT_PERMISSION.equals(preferenceKey)) {
return Website.PermissionDataEntry.getPermissionDataEntry(
ContentSettingsType.CONTENT_SETTINGS_TYPE_JAVASCRIPT);
@@ -482,6 +489,8 @@ public class SingleWebsitePreferences extends PreferenceFragment
mSite.setCookiePermission(permission);
} else if (PREF_FULLSCREEN_PERMISSION.equals(preference.getKey())) {
mSite.setFullscreenPermission(permission);
+ } else if (PREF_IMAGES_PERMISSION.equals(preference.getKey())) {
+ mSite.setImagesPermission(permission);
} else if (PREF_JAVASCRIPT_PERMISSION.equals(preference.getKey())) {
mSite.setJavaScriptPermission(permission);
} else if (PREF_LOCATION_ACCESS.equals(preference.getKey())) {
@@ -538,6 +547,7 @@ public class SingleWebsitePreferences extends PreferenceFragment
WebsitePreferenceBridge.nativeClearCookieData(mSite.getAddress().getOrigin());
mSite.setFullscreenPermission(null);
mSite.setGeolocationPermission(null);
+ mSite.setImagesPermission(null);
mSite.setJavaScriptPermission(null);
mSite.setMidiPermission(null);
mSite.setPopupPermission(null);

Powered by Google App Engine
This is Rietveld 408576698