From cc49b65a11fbe3cfc9099377ebe3410713a953c0 Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Thu, 31 Dec 2020 13:31:28 +0100
Subject: [PATCH] Use the `once: true` option, rather than manually removing
 the "pdf.js.response" event listener in `FirefoxCom.request`

When this code was originally added, the `once` option didn't exist yet.
---
 web/firefoxcom.js | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/web/firefoxcom.js b/web/firefoxcom.js
index da61ebb6e..fbe9091e6 100644
--- a/web/firefoxcom.js
+++ b/web/firefoxcom.js
@@ -63,15 +63,18 @@ const FirefoxCom = (function FirefoxComClosure() {
     request(action, data, callback) {
       const request = document.createTextNode("");
       if (callback) {
-        document.addEventListener("pdf.js.response", function listener(event) {
-          const node = event.target;
-          const response = event.detail.response;
+        document.addEventListener(
+          "pdf.js.response",
+          event => {
+            const node = event.target;
+            const response = event.detail.response;
 
-          document.documentElement.removeChild(node);
+            document.documentElement.removeChild(node);
 
-          document.removeEventListener("pdf.js.response", listener);
-          return callback(response);
-        });
+            return callback(response);
+          },
+          { once: true }
+        );
       }
       document.documentElement.appendChild(request);