src,lib: re-implement DOMException as a cloneable native object

lib/internal/abort_controller.js

@@ -40,7 +40,7 @@ const {
 
 const {
   DOMException,
-} = internalBinding('messaging');
+} = internalBinding('errors');
 
 const {
   clearTimeout,

lib/internal/errors.js

@@ -60,6 +60,12 @@ const {
   URIError,
 } = primordials;
 
+const {
+  makeTransferable,
+  kClone,
+  kDeserialize,
+} = require('internal/worker/js_transferable');
+
 const kIsNodeError = Symbol('kIsNodeError');
 
 const isWindows = process.platform === 'win32';
@@ -828,6 +834,29 @@ class AbortError extends Error {
     super(message, options);
     this.code = 'ABORT_ERR';
     this.name = 'AbortError';
+    // eslint-disable-next-line no-constructor-return
+    return makeTransferable(this);
+  }
+
+  [kClone]() {
+    const name = this.name;
+    const message = this.message;
+    const stack = this.stack;
+    const code = this.code;
+    const cause = this.cause;
+
+    return {
+      data: { name, message, stack, code, cause },
+      deserializeInfo: 'internal/errors:AbortError',
+    };
+  }
+
+  [kDeserialize]({ name, message, stack, code, cause }) {
+    this.name = name;
+    this.message = message;
+    this.stack = stack;
+    this.code = code;
+    this.cause = cause;
   }
 }
 module.exports = {

lib/internal/util.js

@@ -468,12 +468,12 @@ function createDeferredPromise() {
 
 let _DOMException;
 const lazyDOMExceptionClass = () => {
-  _DOMException ??= internalBinding('messaging').DOMException;
+  _DOMException ??= internalBinding('errors').DOMException;
   return _DOMException;
 };
 
 const lazyDOMException = hideStackFrames((message, name) => {
-  _DOMException ??= internalBinding('messaging').DOMException;
+  _DOMException ??= internalBinding('errors').DOMException;
   return new _DOMException(message, name);
 });
 

lib/internal/webstreams/readablestream.js

@@ -40,7 +40,7 @@ const {
 
 const {
   DOMException,
-} = internalBinding('messaging');
+} = internalBinding('errors');
 
 const {
   isArrayBufferView,

lib/internal/webstreams/transfer.js

@@ -1,9 +1,7 @@
 'use strict';
 
 const {
-  ObjectDefineProperties,
   PromiseResolve,
-  ReflectConstruct,
 } = primordials;
 
 const {
@@ -13,7 +11,7 @@ const {
 
 const {
   DOMException,
-} = internalBinding('messaging');
+} = internalBinding('errors');
 
 const {
   ReadableStream,
@@ -34,72 +32,6 @@ const {
 
 const assert = require('internal/assert');
 
-const {
-  makeTransferable,
-  kClone,
-  kDeserialize,
-} = require('internal/worker/js_transferable');
-
-// This class is a bit of a hack. The Node.js implementation of
-// DOMException is not transferable/cloneable. This provides us
-// with a variant that is. Unfortunately, it means playing around
-// a bit with the message, name, and code properties and the
-// prototype. We can revisit this if DOMException is ever made
-// properly cloneable.
-class CloneableDOMException extends DOMException {
-  constructor(message, name) {
-    super(message, name);
-    this[kDeserialize]({
-      message: this.message,
-      name: this.name,
-      code: this.code,
-    });
-    // eslint-disable-next-line no-constructor-return
-    return makeTransferable(this);
-  }
-
-  [kClone]() {
-    return {
-      data: {
-        message: this.message,
-        name: this.name,
-        code: this.code,
-      },
-      deserializeInfo:
-        'internal/webstreams/transfer:InternalCloneableDOMException'
-    };
-  }
-
-  [kDeserialize]({ message, name, code }) {
-    ObjectDefineProperties(this, {
-      message: {
-        configurable: true,
-        enumerable: true,
-        get() { return message; },
-      },
-      name: {
-        configurable: true,
-        enumerable: true,
-        get() { return name; },
-      },
-      code: {
-        configurable: true,
-        enumerable: true,
-        get() { return code; },
-      },
-    });
-  }
-}
-
-function InternalCloneableDOMException() {
-  return makeTransferable(
-    ReflectConstruct(
-      CloneableDOMException,
-      [],
-      DOMException));
-}
-InternalCloneableDOMException[kDeserialize] = () => {};
-
 class CrossRealmTransformReadableSource {
   constructor(port) {
     this[kState] = {
@@ -133,7 +65,7 @@ class CrossRealmTransformReadableSource {
     };
 
     port.onmessageerror = () => {
-      const error = new CloneableDOMException(
+      const error = new DOMException(
         'Internal transferred ReadableStream error',
         'DataCloneError');
       port.postMessage({ type: 'error', value: error });
@@ -156,10 +88,6 @@ class CrossRealmTransformReadableSource {
     try {
       this[kState].port.postMessage({ type: 'error', value: reason });
     } catch (error) {
-      if (error instanceof DOMException) {
-        // eslint-disable-next-line no-ex-assign
-        error = new CloneableDOMException(error.message, error.name);
-      }
       this[kState].port.postMessage({ type: 'error', value: error });
       throw error;
     } finally {
@@ -200,7 +128,7 @@ class CrossRealmTransformWritableSink {
       }
     };
     port.onmessageerror = () => {
-      const error = new CloneableDOMException(
+      const error = new DOMException(
         'Internal transferred ReadableStream error',
         'DataCloneError');
       port.postMessage({ type: 'error', value: error });
@@ -229,10 +157,6 @@ class CrossRealmTransformWritableSink {
     try {
       this[kState].port.postMessage({ type: 'chunk', value: chunk });
     } catch (error) {
-      if (error instanceof DOMException) {
-        // eslint-disable-next-line no-ex-assign
-        error = new CloneableDOMException(error.message, error.name);
-      }
       this[kState].port.postMessage({ type: 'error', value: error });
       this[kState].port.close();
       throw error;
@@ -248,10 +172,6 @@ class CrossRealmTransformWritableSink {
     try {
       this[kState].port.postMessage({ type: 'error', value: reason });
     } catch (error) {
-      if (error instanceof DOMException) {
-        // eslint-disable-next-line no-ex-assign
-        error = new CloneableDOMException(error.message, error.name);
-      }
       this[kState].port.postMessage({ type: 'error', value: error });
       throw error;
     } finally {
@@ -294,6 +214,4 @@ module.exports = {
   newCrossRealmWritableSink,
   CrossRealmTransformWritableSink,
   CrossRealmTransformReadableSource,
-  CloneableDOMException,
-  InternalCloneableDOMException,
 };

lib/internal/webstreams/transformstream.js

@@ -22,7 +22,7 @@ const {
 
 const {
   DOMException,
-} = internalBinding('messaging');
+} = internalBinding('errors');
 
 const {
   createDeferredPromise,

lib/internal/webstreams/writablestream.js

@@ -28,7 +28,7 @@ const {
 
 const {
   DOMException,
-} = internalBinding('messaging');
+} = internalBinding('errors');
 
 const {
   createDeferredPromise,

lib/internal/worker/io.js

@@ -32,8 +32,12 @@ const {
   receiveMessageOnPort: receiveMessageOnPort_,
   stopMessagePort,
   checkMessagePort,
-  DOMException,
 } = internalBinding('messaging');
+
+const {
+  DOMException,
+} = internalBinding('errors');
+
 const {
   getEnvMessagePort
 } = internalBinding('worker');

src/api/environment.cc

@@ -669,7 +669,6 @@ Maybe<bool> InitializePrimordials(Local<Context> context) {
   }
 
   static const char* context_files[] = {"internal/per_context/primordials",
-                                        "internal/per_context/domexception",
                                         "internal/per_context/messageport",
                                         nullptr};