From 453c4b99dc6ad87de3bc448ae7f188bf745822fa Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sa=C3=BAl=20Ibarra=20Corretg=C3=A9?= Date: Wed, 11 Jul 2018 10:02:31 +0200 Subject: [PATCH] cleanup: drop polyfills which were required for IE11 --- package-lock.json | 19 ---------------- package.json | 2 -- react/features/base/lib-jitsi-meet/_.web.js | 5 +---- webpack.config.js | 24 ++------------------- 4 files changed, 3 insertions(+), 47 deletions(-) diff --git a/package-lock.json b/package-lock.json index cd7cdc528..b767d679d 100644 --- a/package-lock.json +++ b/package-lock.json @@ -4178,25 +4178,6 @@ "babel-types": "^6.24.1" } }, - "babel-polyfill": { - "version": "6.26.0", - "resolved": "https://registry.npmjs.org/babel-polyfill/-/babel-polyfill-6.26.0.tgz", - "integrity": "sha1-N5k3q8Z9eJWXCtxiHyhM2WbPIVM=", - "dev": true, - "requires": { - "babel-runtime": "^6.26.0", - "core-js": "^2.5.0", - "regenerator-runtime": "^0.10.5" - }, - "dependencies": { - "regenerator-runtime": { - "version": "0.10.5", - "resolved": "https://registry.npmjs.org/regenerator-runtime/-/regenerator-runtime-0.10.5.tgz", - "integrity": "sha1-M2w+/BIgrc7dosn6tntaeVWjNlg=", - "dev": true - } - } - }, "babel-preset-env": { "version": "1.6.1", "resolved": "https://registry.npmjs.org/babel-preset-env/-/babel-preset-env-1.6.1.tgz", diff --git a/package.json b/package.json index 4daef0884..b7a3de13d 100644 --- a/package.json +++ b/package.json @@ -80,7 +80,6 @@ "babel-core": "6.26.0", "babel-eslint": "8.0.3", "babel-loader": "7.1.2", - "babel-polyfill": "6.26.0", "babel-preset-env": "1.6.1", "babel-preset-react": "6.24.1", "babel-preset-stage-1": "6.24.1", @@ -102,7 +101,6 @@ "string-replace-loader": "1.3.0", "style-loader": "0.19.0", "uglifyjs-webpack-plugin": "1.2.2", - "whatwg-fetch": "2.0.3", "webpack": "3.9.1", "webpack-dev-server": "2.9.5" }, diff --git a/react/features/base/lib-jitsi-meet/_.web.js b/react/features/base/lib-jitsi-meet/_.web.js index ea111d7f8..28eeac3ca 100644 --- a/react/features/base/lib-jitsi-meet/_.web.js +++ b/react/features/base/lib-jitsi-meet/_.web.js @@ -1,7 +1,4 @@ -/* @flow */ - -// Polyfill URL for Internet Explorer. -import 'url-polyfill'; +// @flow declare var JitsiMeetJS: Object; diff --git a/webpack.config.js b/webpack.config.js index 772de84d4..cccd8d679 100644 --- a/webpack.config.js +++ b/webpack.config.js @@ -89,11 +89,6 @@ const config = { loader: 'expose-loader?$!expose-loader?jQuery', test: /\/node_modules\/jquery\/.*\.js$/ - }, { - // Set scope to window for URL polyfill. - - loader: 'imports-loader?this=>window', - test: /\/node_modules\/url-polyfill\/.*\.js$/ }, { // Allow CSS to be imported into JavaScript. @@ -137,13 +132,7 @@ const config = { module.exports = [ Object.assign({}, config, { entry: { - 'app.bundle': [ - - // babel-polyfill and fetch polyfill are required for IE11. - 'babel-polyfill', - 'whatwg-fetch', - './app.js' - ], + 'app.bundle': './app.js', 'device_selection_popup_bundle': './react/features/settings/popup.js', @@ -153,10 +142,6 @@ module.exports = [ 'dial_in_info_bundle': [ - // babel-polyfill and fetch polyfill are required for IE11. - 'babel-polyfill', - 'whatwg-fetch', - // atlaskit does not support React 16 prop-types './react/features/base/react/prop-types-polyfill.js', @@ -172,12 +157,7 @@ module.exports = [ // JitsiMeetExternalAPI). Object.assign({}, config, { entry: { - 'external_api': [ - - // XXX Required by at least IE11 at the time of this writing. - 'babel-polyfill', - './modules/API/external/index.js' - ] + 'external_api': './modules/API/external/index.js' }, output: Object.assign({}, config.output, { library: 'JitsiMeetExternalAPI',