From 209fe604725d81bfbacb760eb005c1ca19c245d2 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Sun, 14 Feb 2021 12:37:37 +0100 Subject: [PATCH] Use a more compact format for the fallback EventBus-listeners in `PDFViewerApplication_initializeJavaScript` Given that these event listeners should essentially never be needed, but are included simply to avoid breakage in edge-cases, it can't hurt to make this code slightly less verbose. --- web/app.js | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/web/app.js b/web/app.js index dc080e7ca..2af3a511e 100644 --- a/web/app.js +++ b/web/app.js @@ -1479,13 +1479,7 @@ const PDFViewerApplication = { // It should be *extremely* rare for metadata to not have been resolved // when this code runs, but ensure that we handle that case here. await new Promise(resolve => { - this.eventBus._on( - "metadataloaded", - evt => { - resolve(); - }, - { once: true } - ); + this.eventBus._on("metadataloaded", resolve, { once: true }); }); if (pdfDocument !== this.pdfDocument) { return; // The document was closed while the metadata resolved. @@ -1498,13 +1492,7 @@ const PDFViewerApplication = { // Hence we'll simply have to trust that the `contentLength` (as provided // by the server), when it exists, is accurate enough here. await new Promise(resolve => { - this.eventBus._on( - "documentloaded", - evt => { - resolve(); - }, - { once: true } - ); + this.eventBus._on("documentloaded", resolve, { once: true }); }); if (pdfDocument !== this.pdfDocument) { return; // The document was closed while the downloadInfo resolved.