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.
This commit is contained in:
Jonas Jenwald 2020-12-31 13:31:28 +01:00
parent f305e042ee
commit cc49b65a11

View file

@ -63,15 +63,18 @@ const FirefoxCom = (function FirefoxComClosure() {
request(action, data, callback) { request(action, data, callback) {
const request = document.createTextNode(""); const request = document.createTextNode("");
if (callback) { if (callback) {
document.addEventListener("pdf.js.response", function listener(event) { document.addEventListener(
"pdf.js.response",
event => {
const node = event.target; const node = event.target;
const response = event.detail.response; 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); document.documentElement.appendChild(request);