aboutsummaryrefslogtreecommitdiff
path: root/ext/bg/js/permissions-main.js
diff options
context:
space:
mode:
authortoasted-nutbread <toasted-nutbread@users.noreply.github.com>2021-01-30 15:05:08 -0500
committerGitHub <noreply@github.com>2021-01-30 15:05:08 -0500
commit828c4b11c14a13abe6ce352177ece341471e374f (patch)
treee08bd57aba64b95593dab87349127016df48c50f /ext/bg/js/permissions-main.js
parenta802666d8dde251d996fee66b4d96f69c044edf4 (diff)
Permissions requesting fixes (#1334)
* Fix permissions requests not occurring during user interaction * Update toggle after changing permissions * Update functions * Update clipboard requests
Diffstat (limited to 'ext/bg/js/permissions-main.js')
-rw-r--r--ext/bg/js/permissions-main.js48
1 files changed, 36 insertions, 12 deletions
diff --git a/ext/bg/js/permissions-main.js b/ext/bg/js/permissions-main.js
index 0d45ace6..cd03dd72 100644
--- a/ext/bg/js/permissions-main.js
+++ b/ext/bg/js/permissions-main.js
@@ -36,21 +36,47 @@ async function isAllowedFileSchemeAccess() {
return await new Promise((resolve) => chrome.extension.isAllowedFileSchemeAccess(resolve));
}
-async function hasPermissions(permissions) {
- return await new Promise((resolve) => chrome.permissions.contains({permissions}, resolve));
+function hasPermissions(permissions) {
+ return new Promise((resolve) => chrome.permissions.contains({permissions}, (result) => {
+ const e = chrome.runtime.lastError;
+ resolve(!e && result);
+ }));
}
-async function setPermissionsGranted(permissions, shouldHave) {
- const has = await hasPermissions(permissions);
- if (shouldHave === has) { return has; }
-
- return await (
+function setPermissionsGranted(permissions, shouldHave) {
+ return (
shouldHave ?
- new Promise((resolve) => chrome.permissions.request({permissions}, resolve)) :
- new Promise((resolve) => chrome.permissions.remove({permissions}, (v) => resolve(!v)))
+ new Promise((resolve, reject) => chrome.permissions.request({permissions}, (result) => {
+ const e = chrome.runtime.lastError;
+ if (e) {
+ reject(new Error(e.message));
+ } else {
+ resolve(result);
+ }
+ })) :
+ new Promise((resolve, reject) => chrome.permissions.remove({permissions}, (result) => {
+ const e = chrome.runtime.lastError;
+ if (e) {
+ reject(new Error(e.message));
+ } else {
+ resolve(!result);
+ }
+ }))
);
}
+function setupPermissionCheckbox(checkbox, permissions) {
+ checkbox.addEventListener('change', (e) => {
+ updatePermissionCheckbox(checkbox, permissions, e.currentTarget.checked);
+ }, false);
+}
+
+async function updatePermissionCheckbox(checkbox, permissions, value) {
+ checkbox.checked = !value;
+ const hasPermission = await setPermissionsGranted(permissions, value);
+ checkbox.checked = hasPermission;
+}
+
(async () => {
try {
const documentFocusController = new DocumentFocusController();
@@ -79,9 +105,7 @@ async function setPermissionsGranted(permissions, shouldHave) {
permissionsCheckboxes[i].checked = permissions[i];
}
- permissionsCheckboxes[0].addEventListener('change', (e) => {
- setPermissionsGranted(['clipboardRead'], e.currentTarget.checked);
- });
+ setupPermissionCheckbox(permissionsCheckboxes[0], ['clipboardRead']);
await promiseTimeout(100);