feat(@formatjs/intl-locale): expose shouldPolyfill to detect if platf…

…orm needs our polyfill
formatjs · Aug 18, 2020 · 3ed62e2 · 3ed62e2
1 parent 7f62ccd
commit 3ed62e2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/packages/intl-locale/polyfill.ts b/packages/intl-locale/polyfill.ts
@@ -1,5 +1,6 @@
 import {Locale} from './';
-if (!('Locale' in Intl)) {
+import {shouldPolyfill} from './should-polyfill';
+if (shouldPolyfill()) {
   Object.defineProperty(Intl, 'Locale', {
     value: Locale,
     writable: true,

diff --git a/packages/intl-locale/should-polyfill.ts b/packages/intl-locale/should-polyfill.ts
@@ -0,0 +1,8 @@
+export function shouldPolyfill() {
+  return (
+    typeof Intl === 'undefined' ||
+    !('Locale' in Intl) ||
+    // https://bugs.chromium.org/p/v8/issues/detail?id=10682
+    new (Intl as any).Locale('und-x-private').toString() === 'x-private'
+  );
+}