use polyfilled DOMException

zloirock · Dec 9, 2021 · 894f81d · 894f81d
1 parent c876280
commit 894f81d
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 13 deletions.
diff --git a/packages/core-js/modules/web.structured-clone.js b/packages/core-js/modules/web.structured-clone.js
@@ -32,6 +32,7 @@ var WebAssembly = global.WebAssembly;
 var CompileError = WebAssembly && WebAssembly.CompileError || Error;
 var LinkError = WebAssembly && WebAssembly.LinkError || Error;
 var RuntimeError = WebAssembly && WebAssembly.RuntimeError || Error;
+var DOMException = getBuiltin('DOMException');
 var Set = getBuiltin('Set');
 var Map = getBuiltin('Map');
 var MapPrototype = Map.prototype;
@@ -48,15 +49,6 @@ var PERFORMANCE_MARK = uid('structuredClone');
 var DATA_CLONE_ERROR = 'DataCloneError';
 var TRANSFERRING = 'Transferring';
 
-// waiting for https://github.com/zloirock/core-js/pull/991
-var DOMException = function (message, name) {
-  try {
-    return new global.DOMException(message, name);
-  } catch (error) {
-    return TypeError(message);
-  }
-};
-
 var checkBasicSemantic = function (structuredCloneImplementation) {
   return !fails(function () {
     var set1 = new global.Set([7]);

diff --git a/packages/core-js/stable/structured-clone.js b/packages/core-js/stable/structured-clone.js
@@ -1,7 +1,11 @@
+require('../modules/es.error.to-string');
 require('../modules/es.array.iterator');
 require('../modules/es.object.to-string');
 require('../modules/es.map');
 require('../modules/es.set');
+require('../modules/web.dom-exception.constructor');
+require('../modules/web.dom-exception.stack');
+require('../modules/web.dom-exception.to-string-tag');
 require('../modules/web.structured-clone');
 var path = require('../internals/path');
 

diff --git a/tests/pure/web.structured-clone.js b/tests/pure/web.structured-clone.js
@@ -13,6 +13,7 @@ import Symbol from 'core-js-pure/es/symbol';
 import Map from 'core-js-pure/es/map';
 import Set from 'core-js-pure/es/set';
 import AggregateError from 'core-js-pure/es/aggregate-error';
+import DOMException from 'core-js-pure/stable/dom-exception';
 
 QUnit.module('structuredClone', () => {
   QUnit.test('identity', assert => {
@@ -356,8 +357,7 @@ QUnit.module('structuredClone', () => {
       });
   });
 
-  // TODO: remove check after https://github.com/zloirock/core-js/pull/991
-  if (fromSource('new DOMException')) QUnit.test('DOMException', assert => {
+  QUnit.test('DOMException', assert => {
     const errors = [
       new DOMException(),
       new DOMException('foo', 'DataCloneError'),

diff --git a/tests/tests/web.structured-clone.js b/tests/tests/web.structured-clone.js
@@ -349,8 +349,7 @@ QUnit.module('structuredClone', () => {
       });
   });
 
-  // TODO: remove DOMException constructor check after https://github.com/zloirock/core-js/pull/991
-  if (fromSource('new DOMException')) QUnit.test('DOMException', assert => {
+  QUnit.test('DOMException', assert => {
     const errors = [
       new DOMException(),
       new DOMException('foo', 'DataCloneError'),