Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Attempt to fix the Array.prototype.includes and String.prototype.includes polyfills (issue 9514, 9516)

Browse Source 
I don't understand why the previous way importing the polyfills didn't work, and I don't have time to try and figure it out, however this patch seems to fix things.

Fixes 9514.
Fixes 9516.
This commit is contained in:
Jonas Jenwald 2018-02-28 12:04:32 +01:00
parent 97faedba0c
commit cd12a177a9
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/shared/compatibility.js
View File

@ -12,7 +12,7 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
/* eslint-disable no-extend-native, mozilla/use-includes-instead-of-indexOf */
/* eslint-disable mozilla/use-includes-instead-of-indexOf */
/* globals PDFJS */
// Skip compatibility checks for the extensions and if we already ran
@ -131,7 +131,7 @@ PDFJS.compatibilityChecked = true;
if (String.prototype.includes) {
return;
}
String.prototype.includes = require('core-js/fn/string/includes');
require('core-js/fn/string/includes');
})();
// Provides support for Array.prototype.includes in legacy browsers.
@ -140,7 +140,7 @@ PDFJS.compatibilityChecked = true;
if (Array.prototype.includes) {
return;
}
Array.prototype.includes = require('core-js/fn/array/includes');
require('core-js/fn/array/includes');
})();
// Provides support for Math.log2 in legacy browsers.