removed safari compatibility check

movsb · Jun 21, 2018 · eb6b296 · eb6b296
1 parent 439d58d
commit eb6b296
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 18 deletions.
diff --git a/gulpfile.js b/gulpfile.js
@@ -78,7 +78,7 @@ var AUTOPREFIXER_CONFIG = {
     'Firefox >= 52', // Last supported on Windows XP
     'Firefox ESR',
     'IE >= 11',
-    'Safari >= 8',
+    'Safari >= 9',
     '> 0.5%',
     'not dead',
   ],

diff --git a/src/display/api.js b/src/display/api.js
@@ -286,10 +286,10 @@ function getDocument(src) {
   }
 
   if (typeof params.disableRange !== 'boolean') {
-    params.disableRange = apiCompatibilityParams.disableRange || false;
+    params.disableRange = false;
   }
   if (typeof params.disableStream !== 'boolean') {
-    params.disableStream = apiCompatibilityParams.disableStream || false;
+    params.disableStream = false;
   }
   if (typeof params.disableAutoFetch !== 'boolean') {
     params.disableAutoFetch = false;

diff --git a/src/display/api_compatibility.js b/src/display/api_compatibility.js
@@ -20,10 +20,7 @@ if (typeof PDFJSDev === 'undefined' || PDFJSDev.test('GENERIC')) {
   const userAgent =
     (typeof navigator !== 'undefined' && navigator.userAgent) || '';
   const isIE = /Trident/.test(userAgent);
-  const isIOS = /\b(iPad|iPhone|iPod)(?=;)/.test(userAgent);
   const isIOSChrome = /CriOS/.test(userAgent);
-  const isSafari = /Safari\//.test(userAgent) &&
-                   !/(Chrome\/|Android\s)/.test(userAgent);
 
   // Checks if possible to use URL.createObjectURL()
   // Support: IE, Chrome on iOS
@@ -35,16 +32,6 @@ if (typeof PDFJSDev === 'undefined' || PDFJSDev.test('GENERIC')) {
     }
   })();
 
-  // Support: Safari 6.0+, iOS
-  (function checkRangeRequests() {
-    // Safari has issues with cached range requests, see issue #3260.
-    // Last tested with version 6.0.4.
-    if (isSafari || isIOS) {
-      compatibilityParams.disableRange = true;
-      compatibilityParams.disableStream = true;
-    }
-  })();
-
   // Support: Node.js
   (function checkFontFaceAndImage() {
     // Node.js is missing native support for `@font-face` and `Image`.

diff --git a/web/app_options.js b/web/app_options.js
@@ -157,12 +157,12 @@ const defaultOptions = {
   },
   disableRange: {
     /** @type {boolean} */
-    value: apiCompatibilityParams.disableRange || false,
+    value: false,
     kind: OptionKind.API,
   },
   disableStream: {
     /** @type {boolean} */
-    value: apiCompatibilityParams.disableStream || false,
+    value: false,
     kind: OptionKind.API,
   },
   isEvalSupported: {