134 lines
7.5 KiB
Diff
134 lines
7.5 KiB
Diff
From: csagan5 <32685696+csagan5@users.noreply.github.com>
|
|
Date: Sat, 28 Dec 2019 10:23:04 +0100
|
|
Subject: Show site settings for cookies
|
|
|
|
Avoid displaying info about intrusive ads
|
|
---
|
|
.../site_settings/SingleWebsiteSettings.java | 59 +++++++++----------
|
|
.../browser_ui/site_settings/Website.java | 14 +++--
|
|
2 files changed, 37 insertions(+), 36 deletions(-)
|
|
|
|
diff --git a/components/browser_ui/site_settings/android/java/src/org/chromium/components/browser_ui/site_settings/SingleWebsiteSettings.java b/components/browser_ui/site_settings/android/java/src/org/chromium/components/browser_ui/site_settings/SingleWebsiteSettings.java
|
|
--- a/components/browser_ui/site_settings/android/java/src/org/chromium/components/browser_ui/site_settings/SingleWebsiteSettings.java
|
|
+++ b/components/browser_ui/site_settings/android/java/src/org/chromium/components/browser_ui/site_settings/SingleWebsiteSettings.java
|
|
@@ -433,6 +433,8 @@ public class SingleWebsiteSettings extends SiteSettingsPreferenceFragment
|
|
setUpSoundPreference(preference);
|
|
} else if (type == ContentSettingsType.JAVASCRIPT) {
|
|
setUpJavascriptPreference(preference);
|
|
+ } else if (type == ContentSettingsType.COOKIES) {
|
|
+ setUpCookiePreference(preference);
|
|
} else if (type == ContentSettingsType.GEOLOCATION) {
|
|
setUpLocationPreference(preference);
|
|
} else if (type == ContentSettingsType.NOTIFICATIONS) {
|
|
@@ -754,18 +756,8 @@ public class SingleWebsiteSettings extends SiteSettingsPreferenceFragment
|
|
}
|
|
|
|
private void setUpAdsInformationalBanner() {
|
|
- // Add the informational banner which shows at the top of the UI if ad blocking is
|
|
- // activated on this site.
|
|
- boolean adBlockingActivated = SiteSettingsCategory.adsCategoryEnabled()
|
|
- && WebsitePreferenceBridge.getAdBlockingActivated(
|
|
- getSiteSettingsClient().getBrowserContextHandle(),
|
|
- mSite.getAddress().getOrigin())
|
|
- && findPreference(getPreferenceKey(ContentSettingsType.ADS)) != null;
|
|
-
|
|
- if (!adBlockingActivated) {
|
|
- removePreferenceSafely(PREF_INTRUSIVE_ADS_INFO);
|
|
- removePreferenceSafely(PREF_INTRUSIVE_ADS_INFO_DIVIDER);
|
|
- }
|
|
+ removePreferenceSafely(PREF_INTRUSIVE_ADS_INFO);
|
|
+ removePreferenceSafely(PREF_INTRUSIVE_ADS_INFO_DIVIDER);
|
|
}
|
|
|
|
private SiteSettingsCategory getWarningCategory() {
|
|
@@ -935,14 +927,31 @@ public class SingleWebsiteSettings extends SiteSettingsPreferenceFragment
|
|
@Nullable
|
|
Integer currentValue =
|
|
mSite.getContentSetting(browserContextHandle, ContentSettingsType.JAVASCRIPT);
|
|
- // If Javascript is blocked by default, then always show a Javascript permission.
|
|
- // To do this, set it to the default value (blocked).
|
|
- if ((currentValue == null)
|
|
- && !WebsitePreferenceBridge.isCategoryEnabled(
|
|
- browserContextHandle, ContentSettingsType.JAVASCRIPT)) {
|
|
- currentValue = ContentSettingValues.BLOCK;
|
|
+ // Always show the Javascript permission
|
|
+ if (currentValue == null) {
|
|
+ currentValue = WebsitePreferenceBridge.isCategoryEnabled(
|
|
+ getSiteSettingsClient().getBrowserContextHandle(),
|
|
+ ContentSettingsType.JAVASCRIPT)
|
|
+ ? ContentSettingValues.ALLOW
|
|
+ : ContentSettingValues.BLOCK;
|
|
}
|
|
- // Not possible to embargo JAVASCRIPT.
|
|
+ setUpListPreference(preference, currentValue, false);
|
|
+ }
|
|
+
|
|
+ private void setUpCookiePreference(Preference preference) {
|
|
+ @ContentSettingValues
|
|
+ @Nullable
|
|
+ Integer currentValue =
|
|
+ mSite.getContentSetting(getSiteSettingsClient().getBrowserContextHandle(), ContentSettingsType.COOKIES);
|
|
+ // Always show the cookies permission
|
|
+ if (currentValue == null) {
|
|
+ currentValue = WebsitePreferenceBridge.isCategoryEnabled(
|
|
+ getSiteSettingsClient().getBrowserContextHandle(),
|
|
+ ContentSettingsType.COOKIES)
|
|
+ ? ContentSettingValues.ALLOW
|
|
+ : ContentSettingValues.BLOCK;
|
|
+ }
|
|
+ // Not possible to embargo COOKIES.
|
|
setUpListPreference(preference, currentValue, false /* isEmbargoed */);
|
|
}
|
|
|
|
@@ -961,22 +970,10 @@ public class SingleWebsiteSettings extends SiteSettingsPreferenceFragment
|
|
setUpListPreference(preference, null, false);
|
|
return;
|
|
}
|
|
- // If the ad blocker is activated, then this site will have ads blocked unless there is an
|
|
- // explicit permission disallowing the blocking.
|
|
- boolean activated = WebsitePreferenceBridge.getAdBlockingActivated(
|
|
- browserContextHandle, mSite.getAddress().getOrigin());
|
|
@ContentSettingValues
|
|
@Nullable
|
|
Integer permission = mSite.getContentSetting(browserContextHandle, ContentSettingsType.ADS);
|
|
|
|
- // If |permission| is null, there is no explicit (non-default) permission set for this site.
|
|
- // If the site is not considered a candidate for blocking, do the standard thing and remove
|
|
- // the preference.
|
|
- if (permission == null && !activated) {
|
|
- setUpListPreference(preference, null, false);
|
|
- return;
|
|
- }
|
|
-
|
|
// However, if the blocking is activated, we still want to show the permission, even if it
|
|
// is in the default state.
|
|
if (permission == null) {
|
|
diff --git a/components/browser_ui/site_settings/android/java/src/org/chromium/components/browser_ui/site_settings/Website.java b/components/browser_ui/site_settings/android/java/src/org/chromium/components/browser_ui/site_settings/Website.java
|
|
--- a/components/browser_ui/site_settings/android/java/src/org/chromium/components/browser_ui/site_settings/Website.java
|
|
+++ b/components/browser_ui/site_settings/android/java/src/org/chromium/components/browser_ui/site_settings/Website.java
|
|
@@ -187,13 +187,17 @@ public final class Website implements Serializable {
|
|
|
|
ContentSettingException exception = getContentSettingException(type);
|
|
if (type == ContentSettingsType.ADS) {
|
|
- // It is possible to set the permission without having an existing exception,
|
|
- // because we can show the BLOCK state even when this permission is set to the
|
|
- // default. In that case, just set an exception now to BLOCK to enable changing the
|
|
- // permission.
|
|
+ // It is possible to set the permission without having an existing exception
|
|
if (exception == null) {
|
|
exception = new ContentSettingException(ContentSettingsType.ADS,
|
|
- getAddress().getOrigin(), ContentSettingValues.BLOCK, "");
|
|
+ getAddress().getOrigin(), value, "");
|
|
+ setContentSettingException(type, exception);
|
|
+ }
|
|
+ } else if (type == ContentSettingsType.COOKIES) {
|
|
+ // It is possible to set the permission without having an existing exception
|
|
+ if (exception == null) {
|
|
+ exception = new ContentSettingException(ContentSettingsType.COOKIES,
|
|
+ getAddress().getOrigin(), value, "");
|
|
setContentSettingException(type, exception);
|
|
}
|
|
} else if (type == ContentSettingsType.JAVASCRIPT) {
|
|
--
|
|
2.17.1
|
|
|