From 614e502227214e92debe8c43776cf8cccd9f700f Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Sun, 3 Feb 2019 14:09:24 +0100
Subject: [PATCH] [api-minor] Remove the `document.currentScript` polyfill

This polyfill is currently used in only *one* file, i.e. `src/display/api.js`, and only when trying to build a *fallback* `workerSrc` path.

Given that the global `workerSrc` should *always* be set[1] when using the PDF.js library[2], and that the fallback `workerSrc` should only be regarded as a best-effort solution anyway, there isn't a particularily strong reason to keep the compatibility code in my opinion.

---
[1] Other supported options include setting the global `workerPort`, or passing in a `PDFWorker` instance as part of the `getDocument` call.

[2] Which is clearly mentioned in the JSDocs in `src/display/worker_options.js`.
---
 src/display/api.js          |  3 ++-
 src/shared/compatibility.js | 19 -------------------
 2 files changed, 2 insertions(+), 20 deletions(-)

diff --git a/src/display/api.js b/src/display/api.js
index eb4089276..811a2fe61 100644
--- a/src/display/api.js
+++ b/src/display/api.js
@@ -90,7 +90,8 @@ if (typeof PDFJSDev !== 'undefined' && PDFJSDev.test('GENERIC')) {
     });
   }) : null;
 
-  if (!fallbackWorkerSrc && typeof document !== 'undefined') {
+  if (!fallbackWorkerSrc && typeof document === 'object' &&
+      'currentScript' in document) {
     const pdfjsFilePath = document.currentScript && document.currentScript.src;
     if (pdfjsFilePath) {
       fallbackWorkerSrc =
diff --git a/src/shared/compatibility.js b/src/shared/compatibility.js
index 13122af13..2efb4d1ec 100644
--- a/src/shared/compatibility.js
+++ b/src/shared/compatibility.js
@@ -54,25 +54,6 @@ const hasDOM = typeof window === 'object' && typeof document === 'object';
   };
 })();
 
-// Provides document.currentScript support
-// Support: IE, Chrome<29.
-(function checkCurrentScript() {
-  if (!hasDOM) {
-    return;
-  }
-  if ('currentScript' in document) {
-    return;
-  }
-  Object.defineProperty(document, 'currentScript', {
-    get() {
-      var scripts = document.getElementsByTagName('script');
-      return scripts[scripts.length - 1];
-    },
-    enumerable: true,
-    configurable: true,
-  });
-})();
-
 // Provides support for ChildNode.remove in legacy browsers.
 // Support: IE.
 (function checkChildNodeRemove() {