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

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

Issue 1068223002: Implement Site Settings \ Images category. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Sync'ed to latest 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/Website.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/Website.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/Website.java
index b212acb7b5d583907969541919b8252b818e25e3..3a12ec91eeed0b9e062c0bcaea6bc3314c96bf22 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/Website.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/Website.java
@@ -167,6 +167,14 @@ public class Website implements Serializable {
R.string.website_settings_category_ask_before_accessing;
entry.customDisabledSummary = R.string.website_settings_category_block_all;
return entry;
+ case ContentSettingsType.CONTENT_SETTINGS_TYPE_IMAGES:
+ entry = new PermissionDataEntry();
+ entry.iconResourceId = R.drawable.permission_images;
+ entry.titleResourceId = R.string.images_permission_title;
+ entry.explanationResourceId = R.string.images_permission_title;
+ entry.defaultEnabledValue = ContentSetting.ALLOW;
+ entry.defaultDisabledValue = ContentSetting.BLOCK;
+ return entry;
case ContentSettingsType.CONTENT_SETTINGS_TYPE_MEDIASTREAM:
entry = new PermissionDataEntry();
entry.iconResourceId = R.drawable.permission_media;
@@ -232,6 +240,7 @@ public class Website implements Serializable {
private CookieInfo mCookieInfo;
private GeolocationInfo mGeolocationInfo;
private MidiInfo mMidiInfo;
+ private ImagesExceptionInfo mImagesExceptionInfo;
private JavaScriptExceptionInfo mJavaScriptExceptionInfo;
private PopupExceptionInfo mPopupExceptionInfo;
private ProtectedMediaIdentifierInfo mProtectedMediaIdentifierInfo;
@@ -366,6 +375,29 @@ public class Website implements Serializable {
}
/**
+ * Returns what permission governs Images access.
+ */
+ public ContentSetting getImagesPermission() {
+ return mImagesExceptionInfo != null ? mImagesExceptionInfo.getContentSetting() : null;
+ }
+
+ /**
+ * Configure Images permission access setting for this site.
+ */
+ public void setImagesPermission(ContentSetting value) {
+ if (mImagesExceptionInfo != null) {
+ mImagesExceptionInfo.setContentSetting(value);
+ }
+ }
+
+ /**
+ * Sets the Images exception info for this Website.
+ */
+ public void setImagesInfo(ImagesExceptionInfo info) {
+ mImagesExceptionInfo = info;
+ }
+
+ /**
* Returns what permission governs JavaScript access.
*/
public ContentSetting getJavaScriptPermission() {

Powered by Google App Engine
This is Rietveld 408576698