diff --git a/index.android.js b/index.android.js index 12035cbe4..56408336f 100644 --- a/index.android.js +++ b/index.android.js @@ -1,26 +1,2 @@ -// The type field of react-native application loader's React Element is created -// as number and not Symbol, because it's not been defined by the polyfill yet. -// We import the application renderer, before Symbol is defined, in order to use -// number types as well. Otherwise this will result in the invariant exception, -// because fiber thingy will not recognise root react-native component as React -// Element, but as an Object. -// -// See node_modules/react-native/Libraries/polyfills/babelHelpers.js -// :babelHelpers.createRawReactElement - that's where first react-native element -// is created (super early - it's the app loader). -// -// See node_modules/react-native/Libraries/Renderer/ReactNativeFiber-dev.js -// and look for REACT_ELEMENT_TYPE definition - it's defined later when Symbol -// has been defined and type will not match. -// -// As an alternative solution we could stop using/polyfilling Symbols and -// replace with classpath string constants or some kind of a wrapper around -// that. - -import 'react-native/Libraries/ReactNative/renderApplication'; - -// Android doesn't provide Symbol -import 'es6-symbol/implement'; - import './react/index.native'; diff --git a/package-lock.json b/package-lock.json index b278e875d..96b8cd4fa 100644 --- a/package-lock.json +++ b/package-lock.json @@ -5727,6 +5727,7 @@ "version": "1.0.0", "resolved": "https://registry.npmjs.org/d/-/d-1.0.0.tgz", "integrity": "sha1-dUu1v+VUUdpppYuU1F9MWwRi1Y8=", + "dev": true, "requires": { "es5-ext": "^0.10.9" } @@ -6231,6 +6232,7 @@ "version": "0.10.39", "resolved": "https://registry.npmjs.org/es5-ext/-/es5-ext-0.10.39.tgz", "integrity": "sha512-AlaXZhPHl0po/uxMx1tyrlt1O86M6D5iVaDH8UgLfgek4kXTX6vzsRfJQWC2Ku+aG8pkw1XWzh9eTkwfVrsD5g==", + "dev": true, "requires": { "es6-iterator": "~2.0.3", "es6-symbol": "~3.1.1" @@ -6240,6 +6242,7 @@ "version": "2.0.3", "resolved": "https://registry.npmjs.org/es6-iterator/-/es6-iterator-2.0.3.tgz", "integrity": "sha1-p96IkUGgWpSwhUQDstCg+/qY87c=", + "dev": true, "requires": { "d": "1", "es5-ext": "^0.10.35", @@ -6277,6 +6280,7 @@ "version": "3.1.1", "resolved": "https://registry.npmjs.org/es6-symbol/-/es6-symbol-3.1.1.tgz", "integrity": "sha1-vwDvT9q2uhtG7Le2KbTH7VcVzHc=", + "dev": true, "requires": { "d": "1", "es5-ext": "~0.10.14" diff --git a/package.json b/package.json index 098b66ad1..b4ae09bb4 100644 --- a/package.json +++ b/package.json @@ -36,8 +36,6 @@ "@atlaskit/tooltip": "9.1.1", "@webcomponents/url": "0.7.1", "autosize": "1.18.13", - "es6-iterator": "2.0.3", - "es6-symbol": "3.1.1", "i18next": "8.4.3", "i18next-browser-languagedetector": "2.0.0", "i18next-xhr-backend": "1.4.2", diff --git a/react/features/base/lib-jitsi-meet/native/polyfills-browser.js b/react/features/base/lib-jitsi-meet/native/polyfills-browser.js index 3671e703b..69ac94047 100644 --- a/react/features/base/lib-jitsi-meet/native/polyfills-browser.js +++ b/react/features/base/lib-jitsi-meet/native/polyfills-browser.js @@ -1,4 +1,3 @@ -import Iterator from 'es6-iterator'; import BackgroundTimer from 'react-native-background-timer'; import '@webcomponents/url'; // Polyfill for URL constructor @@ -123,18 +122,6 @@ function _visitNode(node, callback) { global.removeEventListener = () => {}; } - // Array.prototype[@@iterator] - // - // Required by: - // - for...of statement use(s) in lib-jitsi-meet - const arrayPrototype = Array.prototype; - - if (typeof arrayPrototype['@@iterator'] === 'undefined') { - arrayPrototype['@@iterator'] = function() { - return new Iterator(this); - }; - } - // document // // Required by: