refactor: obj.prop?.() is safe, there's no .call

.eslintrc.js

@@ -41,10 +41,6 @@ const INJECTED_RULES = {
     'error', {
       selector: 'ObjectExpression > ExperimentalSpreadProperty',
       message: 'Object spread adds a polyfill in injected* even if unused by it',
-    }, {
-      selector: 'OptionalCallExpression > MemberExpression',
-      message: 'Optional call on property uses .call(), which may be spoofed/broken in an unsafe environment',
-      // TODO: write a Babel plugin to use safeCall for this.
     }, {
       selector: 'ArrayPattern',
       message: 'Destructuring via Symbol.iterator may be spoofed/broken in an unsafe environment',

src/injected/web/notifications.js

@@ -5,15 +5,13 @@ const notifications = createNullObj();
 
 addHandlers({
   NotificationClicked(id) {
-    const fn = notifications[id]?.onclick;
-    if (fn) fn();
+    notifications[id]?.onclick?.();
   },
   NotificationClosed(id) {
     const options = notifications[id];
     if (options) {
       delete notifications[id];
-      const fn = options.ondone;
-      if (fn) fn();
+      options.ondone?.();
     }
   },
 });

src/injected/web/tabs.js

@@ -9,8 +9,7 @@ addHandlers({
     if (item) {
       item.closed = true;
       delete tabs[key];
-      const fn = item.onclose;
-      if (fn) fn();
+      item.onclose?.();
     }
   },
 });