Only register the notification click handler once (#267)

mrmr1993 · web-flow · commit 8b254f3b299b · 2022-03-02T14:53:33.000+05:30
diff --git a/source/background.js b/source/background.js
@@ -97,13 +97,15 @@ async function onTabUpdated(tabId, changeInfo, tab) {
 	}
 }
 
+function onNotificationClick(id) {
+	openNotification(id);
+}
+
 async function addHandlers() {
 	const {updateCountOnNavigation} = await optionsStorage.getAll();
 
 	if (await queryPermission('notifications')) {
-		browser.notifications.onClicked.addListener(id => {
-			openNotification(id);
-		});
+		browser.notifications.onClicked.addListener(onNotificationClick);
 	}
 
 	if (await queryPermission('tabs')) {

Original file line number	Diff line number	Diff line change
`@@ -97,13 +97,15 @@ async function onTabUpdated(tabId, changeInfo, tab) {`
`97`	`97`	`}`
`98`	`98`	`}`
`99`	`99`
	`100`	`+function onNotificationClick(id) {`
	`101`	`+ openNotification(id);`
	`102`	`+}`
	`103`	`+`
`100`	`104`	`async function addHandlers() {`
`101`	`105`	`const {updateCountOnNavigation} = await optionsStorage.getAll();`
`102`	`106`
`103`	`107`	`if (await queryPermission('notifications')) {`
`104`		`- browser.notifications.onClicked.addListener(id => {`
`105`		`- openNotification(id);`
`106`		`- });`
	`108`	`+ browser.notifications.onClicked.addListener(onNotificationClick);`
`107`	`109`	`}`
`108`	`110`
`109`	`111`	`if (await queryPermission('tabs')) {`