Commit c44d59af authored by Illia Klimov's avatar Illia Klimov Committed by Commit Bot

Notifications permission is not seen in site settings.

This patch fixes a bug that adds an empty space instead of notifications
permission in site settings. Reproducible only on Android 7.1 and less.

Bug: 1099645
Change-Id: I8d283b186ce562a1598ce72ff431863d45c77197
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2273103
Commit-Queue: Andy Paicu <andypaicu@chromium.org>
Reviewed-by: default avatarAndy Paicu <andypaicu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#783867}
parent e2bb80dc
......@@ -512,6 +512,7 @@ public class SingleWebsiteSettings extends SiteSettingsPreferenceFragment
return true;
});
} else {
setUpListPreference(preference, value, isEmbargoed);
if (isPermissionControlledByDSE(ContentSettingsType.NOTIFICATIONS) && value != null) {
updatePreferenceForDSESetting(preference);
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment