From 745cb73c65e315e44bf299a09f8d272d46a3b3f3 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Sun, 12 Nov 2017 17:43:07 +0100 Subject: [PATCH] Remove `PDFJS.disableRange`/`PDFJS.disableStream` code for now unsupported browsers in `src/shared/compatibility.js` We're currently disabling range requests and streaming for a number of configurations. A couple of those will no longer be supported (with PDF.js version 2.0), hence we ought to be able to clean up the compatibility code slightly. --- src/shared/compatibility.js | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/src/shared/compatibility.js b/src/shared/compatibility.js index 117d68345..4822755cd 100644 --- a/src/shared/compatibility.js +++ b/src/shared/compatibility.js @@ -25,10 +25,8 @@ var globalScope = require('./global_scope'); var userAgent = (typeof navigator !== 'undefined' && navigator.userAgent) || ''; var isAndroid = /Android/.test(userAgent); -var isAndroidPre3 = /Android\s[0-2][^\d]/.test(userAgent); var isAndroidPre5 = /Android\s[0-4][^\d]/.test(userAgent); var isChrome = userAgent.indexOf('Chrom') >= 0; -var isChromeWithRangeBug = /Chrome\/(39|40)\./.test(userAgent); var isIOSChrome = userAgent.indexOf('CriOS') >= 0; var isIE = userAgent.indexOf('Trident') >= 0; var isIOS = /\b(iPad|iPhone|iPod)(?=;)/.test(userAgent); @@ -470,19 +468,12 @@ PDFJS.compatibilityChecked = true; PDFJS.locale = navigator.userLanguage || 'en-US'; })(); -// Support: Safari 6.0+, Android<3.0, Chrome 39/40, iOS +// Support: Safari 6.0+, iOS (function checkRangeRequests() { // Safari has issues with cached range requests see: // https://github.com/mozilla/pdf.js/issues/3260 // Last tested with version 6.0.4. - - // Older versions of Android (pre 3.0) has issues with range requests, see: - // https://github.com/mozilla/pdf.js/issues/3381. - // Make sure that we only match webkit-based Android browsers, - // since Firefox/Fennec works as expected. - - // Range requests are broken in Chrome 39 and 40, https://crbug.com/442318 - if (isSafari || isAndroidPre3 || isChromeWithRangeBug || isIOS) { + if (isSafari || isIOS) { PDFJS.disableRange = true; PDFJS.disableStream = true; }