Replace the remaining occurences, in src/display/api.js, of var with let/const

src/display/api.js

@@ -13,6 +13,7 @@
  * limitations under the License.
  */
 /* globals requirejs, __non_webpack_require__ */
+/* eslint no-var: error */
 
 import {
   assert, createPromiseCapability, deprecated, getVerbosityLevel, info,
@@ -106,7 +107,7 @@ if (typeof PDFJSDev !== 'undefined' && PDFJSDev.test('GENERIC')) {
  */
 
 /** @type IPDFStreamFactory */
-var createPDFNetworkStream;
+let createPDFNetworkStream;
 
 /**
  * Sets the function that instantiates a IPDFStream as an alternative PDF data
@@ -221,9 +222,9 @@ function setPDFNetworkStreamFactory(pdfNetworkStreamFactory) {
  * @return {PDFDocumentLoadingTask}
  */
 function getDocument(src) {
-  var task = new PDFDocumentLoadingTask();
+  const task = new PDFDocumentLoadingTask();
 
-  var source;
+  let source;
   if (typeof src === 'string') {
     source = { url: src, };
   } else if (isArrayBuffer(src)) {
@@ -239,15 +240,12 @@ function getDocument(src) {
       throw new Error(
         'Invalid parameter object: need either .data, .range or .url');
     }
-
     source = src;
   }
+  const params = Object.create(null);
+  let rangeTransport = null, worker = null;
 
-  let params = Object.create(null);
+  for (const key in source) {
-  var rangeTransport = null;
-  let worker = null;
-
-  for (var key in source) {
     if (key === 'url' && typeof window !== 'undefined') {
       // The full path is required in the 'url' field.
       params[key] = new URL(source[key], window.location).href;
@@ -260,7 +258,7 @@ function getDocument(src) {
       continue;
     } else if (key === 'data' && !(source[key] instanceof Uint8Array)) {
       // Converting string or array-like data to Uint8Array.
-      var pdfBytes = source[key];
+      const pdfBytes = source[key];
       if (typeof pdfBytes === 'string') {
         params[key] = stringToBytes(pdfBytes);
       } else if (typeof pdfBytes === 'object' && pdfBytes !== null &&
@@ -329,7 +327,7 @@ function getDocument(src) {
                                  new PDFWorker(workerParams);
     task._worker = worker;
   }
-  var docId = task.docId;
+  const docId = task.docId;
   worker.promise.then(function() {
     if (task.destroyed) {
       throw new Error('Loading aborted');
@@ -360,9 +358,9 @@ function getDocument(src) {
         });
       }
 
-      var messageHandler = new MessageHandler(docId, workerId, worker.port);
+      const messageHandler = new MessageHandler(docId, workerId, worker.port);
       messageHandler.postMessageTransfers = worker.postMessageTransfers;
-      var transport = new WorkerTransport(messageHandler, task, networkStream,
+      const transport = new WorkerTransport(messageHandler, task, networkStream,
                                             params);
       task._transport = transport;
       messageHandler.send('Ready', null);
@@ -1257,11 +1255,10 @@ class LoopbackPort {
       if (cloned.has(value)) { // already cloned the object
         return cloned.get(value);
       }
-      var result;
+      let buffer, result;
-      var buffer;
       if ((buffer = value.buffer) && isArrayBuffer(buffer)) {
         // We found object with ArrayBuffer (typed array).
-        var transferable = transfers && transfers.includes(buffer);
+        const transferable = transfers && transfers.includes(buffer);
         if (value === buffer) {
           // Special case when we are faking typed arrays in compatibility.js.
           result = value;
@@ -1278,8 +1275,8 @@ class LoopbackPort {
       cloned.set(value, result); // adding to cache now for cyclic references
       // Cloning all value and object properties, however ignoring properties
       // defined via getter.
-      for (var i in value) {
+      for (const i in value) {
-        var desc, p = value;
+        let desc, p = value;
         while (!(desc = Object.getOwnPropertyDescriptor(p, i))) {
           p = Object.getPrototypeOf(p);
         }
@@ -1299,8 +1296,8 @@ class LoopbackPort {
       return;
     }
 
-    var cloned = new WeakMap();
+    const cloned = new WeakMap();
-    var e = { data: cloneValue(obj), };
+    const e = { data: cloneValue(obj), };
     this._deferred.then(() => {
       this._listeners.forEach(function(listener) {
         listener.call(this, e);
@@ -1313,7 +1310,7 @@ class LoopbackPort {
   }
 
   removeEventListener(name, listener) {
-    var i = this._listeners.indexOf(listener);
+    const i = this._listeners.indexOf(listener);
     this._listeners.splice(i, 1);
   }
 
@@ -2485,11 +2482,10 @@ const InternalRenderTask = (function InternalRenderTaskClosure() {
   return InternalRenderTask;
 })();
 
-var version, build;
+const version = (typeof PDFJSDev !== 'undefined' ?
-if (typeof PDFJSDev !== 'undefined') {
+                 PDFJSDev.eval('BUNDLE_VERSION') : null);
-  version = PDFJSDev.eval('BUNDLE_VERSION');
+const build = (typeof PDFJSDev !== 'undefined' ?
-  build = PDFJSDev.eval('BUNDLE_BUILD');
+               PDFJSDev.eval('BUNDLE_BUILD') : null);
-}
 
 export {
   getDocument,