Merge pull request #19137 from Snuffleupagus/PDFDocumentProperties-mo…

…ve-getPage Move the `getPage` call in `PDFDocumentProperties` class
mozilla · Dec 1, 2024 · c40a0a6 · c40a0a6
2 parents 9f90bc9 + e5f744d
commit c40a0a6
Showing 1 changed file with 8 additions and 10 deletions.
diff --git a/web/pdf_document_properties.js b/web/pdf_document_properties.js
@@ -112,12 +112,13 @@ class PDFDocumentProperties {
     }
 
     // Get the document properties.
-    const {
-      info,
-      /* metadata, */
-      /* contentDispositionFilename, */
-      contentLength,
-    } = await this.pdfDocument.getMetadata();
+    const [
+      { info, /* metadata, contentDispositionFilename, */ contentLength },
+      pdfPage,
+    ] = await Promise.all([
+      this.pdfDocument.getMetadata(),
+      this.pdfDocument.getPage(currentPageNumber),
+    ]);
 
     const [
       fileName,
@@ -131,10 +132,7 @@ class PDFDocumentProperties {
       this.#parseFileSize(contentLength),
       this.#parseDate(info.CreationDate),
       this.#parseDate(info.ModDate),
-      // eslint-disable-next-line arrow-body-style
-      this.pdfDocument.getPage(currentPageNumber).then(pdfPage => {
-        return this.#parsePageSize(getPageSizeInches(pdfPage), pagesRotation);
-      }),
+      this.#parsePageSize(getPageSizeInches(pdfPage), pagesRotation),
       this.#parseLinearization(info.IsLinearized),
     ]);