From 4aab3cef4ac9059e6b61fb821d2dbc7fd0047440 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Sun, 18 Jun 2017 12:29:28 +0200 Subject: [PATCH] Remove a redundant `PDFViewer.currentScale` call from `PDFViewerApplication.load` Since this call occurs *before* the `PDFViewer.setDocument` call, it won't actually cause any scale change. Furthermore, moving it should not be necessary, since the `scale` is already used as the fallback case in `PDFViewerApplication.setInitialView` (provided it's non-zero, which isn't even the case in the default viewer). Hence this patch should cause no functional changes at all, since it simply removes a piece of unnecessary code. --- web/app.js | 1 - 1 file changed, 1 deletion(-) diff --git a/web/app.js b/web/app.js index 082335725c506..399062ebb362f 100644 --- a/web/app.js +++ b/web/app.js @@ -907,7 +907,6 @@ var PDFViewerApplication = { this.pdfDocumentProperties.setDocument(pdfDocument, this.url); let pdfViewer = this.pdfViewer; - pdfViewer.currentScale = scale; pdfViewer.setDocument(pdfDocument); let firstPagePromise = pdfViewer.firstPagePromise; let pagesPromise = pdfViewer.pagesPromise;