events: no stopPropagation call in cancelBubble

PR-URL: #50405 Reviewed-By: Matthew Aitken <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Deokjin Kim <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
nodejs · Feb 27, 2024 · 78f8180 · 78f8180
1 parent ec17fd7
commit 78f8180
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/internal/event_target.js b/lib/internal/event_target.js
@@ -306,7 +306,7 @@ class Event {
     if (!isEvent(this))
       throw new ERR_INVALID_THIS('Event');
     if (value) {
-      this.stopPropagation();
+      this.#propagationStopped = true;
     }
   }
 

diff --git a/test/parallel/test-eventtarget.js b/test/parallel/test-eventtarget.js
@@ -738,3 +738,10 @@ let asyncTest = Promise.resolve();
   controller.abort();
   et.dispatchEvent(new Event('foo'));
 }
+
+{
+  const event = new Event('foo');
+  strictEqual(event.cancelBubble, false);
+  event.cancelBubble = true;
+  strictEqual(event.cancelBubble, true);
+}