events: allow null/undefined eventInitDict

PR-URL: #54643 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: LiviaMedeiros <[email protected]> Reviewed-By: Minwoo Jung <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Nov 17, 2024 · 0c1666a · 0c1666a
1 parent 93a098c
commit 0c1666a
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/lib/internal/event_target.js b/lib/internal/event_target.js
@@ -115,14 +115,14 @@ class Event {
    *   composed?: boolean,
    * }} [options]
    */
-  constructor(type, options = kEmptyObject) {
+  constructor(type, options = undefined) {
     if (arguments.length === 0)
       throw new ERR_MISSING_ARGS('type');
-    validateObject(options, 'options');
-    const { bubbles, cancelable, composed } = options;
-    this.#cancelable = !!cancelable;
-    this.#bubbles = !!bubbles;
-    this.#composed = !!composed;
+    if (options != null)
+      validateObject(options, 'options');
+    this.#bubbles = !!options?.bubbles;
+    this.#cancelable = !!options?.cancelable;
+    this.#composed = !!options?.composed;
 
     this[kType] = `${type}`;
     if (options?.[kTrustEvent]) {

diff --git a/test/parallel/test-eventtarget.js b/test/parallel/test-eventtarget.js
@@ -747,3 +747,9 @@ let asyncTest = Promise.resolve();
   event.cancelBubble = true;
   strictEqual(event.cancelBubble, true);
 }
+
+{
+  // A null eventInitDict should not throw an error.
+  new Event('', null);
+  new Event('', undefined);
+}