lib: implement passive listener behavior per spec by Amemome · Pull Request #59995 · nodejs/node · GitHub

Implement Web API spec-compliant behavior
for passive event listeners
where preventDefault() and returnValue setting are ignored during
passive listener execution.

Add kInPassiveListener symbol to track passive listener state
Modify preventDefault() to ignore calls during
passive listener execution
Add returnValue setter that respects passive listener state
Set/clear passive flag around listener invocation in dispatchEvent

Fixes WPT test cases that were previously failing in
AddEventListenerOptions-passive.any.js and
satisfies expected behaviors:

preventDefault should be ignored
if-and-only-if the passive option is true
returnValue should be ignored
if-and-only-if the passive option is true
passive behavior of one listener should be unaffected
by other listeners

Refs: https://dom.spec.whatwg.org/#dom-event-preventdefault

Review requested:

@nodejs/web-standards

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.44%. Comparing base (36b1cc2) to head (24e6f21).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #59995      +/-   ##
==========================================
- Coverage   88.45%   88.44%   -0.02%     
==========================================
  Files         703      703              
  Lines      207794   207815      +21     
  Branches    40024    40034      +10     
==========================================
- Hits       183805   183792      -13     
- Misses      15971    16015      +44     
+ Partials     8018     8008      -10

Files with missing lines	Coverage Δ
lib/internal/event_target.js	99.16% <100.00%> (+0.01%)	⬆️

... and 31 files with indirect coverage changes

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

CI: https://ci.nodejs.org/job/node-test-pull-request/69502/

lgtm

Commit Queue failed

- Loading data for nodejs/node/pull/59995
✔  Done loading data for nodejs/node/pull/59995
----------------------------------- PR info ------------------------------------
Title      lib: implement passive listener behavior per spec (#59995)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     Amemome:lib/implement-events-passive-listener-spec -> nodejs:main
Labels     author ready, needs-ci, commit-queue-squash
Commits    8
 - lib: implement passive listener preventDefault behavior per spec
 - test: remove passing AddEventListenerOptions-passive.any.js WPT status
 - lib: Ensure kInPassiveListener state is cleaned up on error
 - lib: fix lint error
 - lib: fix passive listener implementation with cancelable checks
 - test: enable passive listener tests after implementation fix
 - test: fix lint error by removing unused common variable assignment
 - Merge branch 'nodejs:main' into lib/implement-events-passive-listener…
Committers 2
 - BCD1me <152237136+Amemome@users.noreply.github.com>
 - GitHub <noreply@github.com>
PR-URL: https://github.com/nodejs/node/pull/59995
Refs: https://dom.spec.whatwg.org/#dom-event-preventdefault
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Reviewed-By: Mattias Buelens <mattias@buelens.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Jason Zhang <xzha4350@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/59995
Refs: https://dom.spec.whatwg.org/#dom-event-preventdefault
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Reviewed-By: Mattias Buelens <mattias@buelens.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Jason Zhang <xzha4350@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Wed, 24 Sep 2025 06:44:19 GMT
   ✔  Approvals: 5
   ✔  - Daeyeon Jeong (@daeyeon): https://github.com/nodejs/node/pull/59995#pullrequestreview-3291751784
   ✔  - Mattias Buelens (@MattiasBuelens): https://github.com/nodejs/node/pull/59995#pullrequestreview-3293254003
   ✔  - Benjamin Gruenbaum (@benjamingr) (TSC): https://github.com/nodejs/node/pull/59995#pullrequestreview-3295749506
   ✔  - Jason Zhang (@jazelly): https://github.com/nodejs/node/pull/59995#pullrequestreview-3296937245
   ✔  - Matteo Collina (@mcollina) (TSC): https://github.com/nodejs/node/pull/59995#pullrequestreview-3297832070
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2025-10-02T01:25:08Z: https://ci.nodejs.org/job/node-test-pull-request/69502/
- Querying data for job/node-test-pull-request/69502/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
 * branch                  main       -> FETCH_HEAD
✔  origin/main is now up-to-date
- Downloading patch for 59995
From https://github.com/nodejs/node
 * branch                  refs/pull/59995/merge -> FETCH_HEAD
✔  Fetched commits as 51df7b92bc0a..2c8534ed1e9b
--------------------------------------------------------------------------------
[main 6d31a3c5b1] lib: implement passive listener preventDefault behavior per spec
 Author: BCD1me <152237136+Amemome@users.noreply.github.com>
 Date: Wed Sep 24 15:29:30 2025 +0900
 1 file changed, 25 insertions(+), 1 deletion(-)
[main 415016f039] test: remove passing AddEventListenerOptions-passive.any.js WPT status
 Author: BCD1me <152237136+Amemome@users.noreply.github.com>
 Date: Wed Sep 24 15:41:15 2025 +0900
 1 file changed, 9 deletions(-)
[main e5f52560b9] lib: Ensure kInPassiveListener state is cleaned up on error
 Author: BCD1me <152237136+Amemome@users.noreply.github.com>
 Date: Sat Sep 27 21:40:25 2025 +0900
 1 file changed, 3 insertions(+), 4 deletions(-)
[main de9f55d0ec] lib: fix lint error
 Author: BCD1me <152237136+Amemome@users.noreply.github.com>
 Date: Sat Sep 27 21:57:18 2025 +0900
 1 file changed, 1 insertion(+), 1 deletion(-)
[main 3e62f50f35] lib: fix passive listener implementation with cancelable checks
 Author: BCD1me <152237136+Amemome@users.noreply.github.com>
 Date: Wed Oct 1 13:29:01 2025 +0900
 1 file changed, 5 insertions(+), 7 deletions(-)
[main 9d2dc1012e] test: enable passive listener tests after implementation fix
 Author: BCD1me <152237136+Amemome@users.noreply.github.com>
 Date: Wed Oct 1 13:34:35 2025 +0900
 1 file changed, 1 deletion(-)
error: commit 2c8534ed1e9ba7316370102515aa168909bb8c83 is a merge but no -m option was given.
fatal: cherry-pick failed
[main fa532ce720] test: fix lint error by removing unused common variable assignment
 Author: BCD1me <152237136+Amemome@users.noreply.github.com>
 Date: Wed Oct 1 14:07:41 2025 +0900
 1 file changed, 1 insertion(+), 1 deletion(-)
   ✘  Failed to apply patches

https://github.com/nodejs/node/actions/runs/18216332994

@Amemome unfortunately our automation is failing to land this. Could you squash your commits and rebase on top of main?

Implements the WHATWG DOM specification for passive event listeners, ensuring that calls to `preventDefault()` are correctly ignored within a passive listener context. An internal `kInPassiveListener` state is added to the Event object to track when a passive listener is executing. The `preventDefault()` method and the `returnValue` setter are modified to check this state, as well as the event's `cancelable` property. This state is reliably cleaned up within a `finally` block to prevent state pollution in case a listener throws an error. This resolves previously failing Web Platform Tests (WPT) in `AddEventListenerOptions-passive.any.js`. Refs: https://dom.spec.whatwg.org/#dom-event-preventdefault

CI: https://ci.nodejs.org/job/node-test-pull-request/69525/

Landed in 6f941fc

Implements the WHATWG DOM specification for passive event listeners, ensuring that calls to `preventDefault()` are correctly ignored within a passive listener context. An internal `kInPassiveListener` state is added to the Event object to track when a passive listener is executing. The `preventDefault()` method and the `returnValue` setter are modified to check this state, as well as the event's `cancelable` property. This state is reliably cleaned up within a `finally` block to prevent state pollution in case a listener throws an error. This resolves previously failing Web Platform Tests (WPT) in `AddEventListenerOptions-passive.any.js`. Refs: https://dom.spec.whatwg.org/#dom-event-preventdefault PR-URL: #59995 Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com> Reviewed-By: Mattias Buelens <mattias@buelens.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Jason Zhang <xzha4350@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

nodejs-github-bot added the needs-ci PRs that need a full CI run. label Sep 24, 2025

daeyeon reviewed Sep 25, 2025

View reviewed changes

Comment thread lib/internal/event_target.js Outdated

Amemome force-pushed the lib/implement-events-passive-listener-spec branch from ee8c8fd to 402d8cf Compare September 27, 2025 12:52

daeyeon approved these changes Oct 2, 2025

View reviewed changes

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 2, 2025

MattiasBuelens approved these changes Oct 2, 2025

View reviewed changes

benjamingr approved these changes Oct 2, 2025

View reviewed changes

jazelly approved these changes Oct 3, 2025

View reviewed changes

mcollina approved these changes Oct 3, 2025

View reviewed changes

mcollina added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 3, 2025

nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Oct 3, 2025

Amemome force-pushed the lib/implement-events-passive-listener-spec branch from 2c8534e to 24e6f21 Compare October 3, 2025 11:09

daeyeon added request-ci Add this label to start a Jenkins CI on a PR. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Oct 3, 2025

github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 3, 2025

daeyeon approved these changes Oct 4, 2025

View reviewed changes

daeyeon added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 4, 2025

nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 4, 2025

nodejs-github-bot merged commit 6f941fc into nodejs:main Oct 4, 2025
60 checks passed

github-actions Bot mentioned this pull request Oct 7, 2025

2025-10-08, Version 24.10.0 (Current) #60136

Merged

github-actions Bot mentioned this pull request Oct 23, 2025

2025-10-28, Version 22.21.1 'Jod' (LTS) #60375

Merged

lib: implement passive listener behavior per spec by Amemome · Pull Request #59995 · nodejs/node · GitHub

Uh oh!

Uh oh!

lib: implement passive listener behavior per spec - #59995

lib: implement passive listener behavior per spec#59995
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
Amemome:lib/implement-events-passive-listener-spec

Amemome commented Sep 24, 2025

Uh oh!

nodejs-github-bot commented Sep 24, 2025

Uh oh!

codecov Bot commented Sep 24, 2025 •
edited
Loading

Uh oh!

Uh oh!

Uh oh!

nodejs-github-bot commented Oct 2, 2025

Uh oh!

mcollina left a comment

Uh oh!

nodejs-github-bot commented Oct 3, 2025

Uh oh!

mcollina commented Oct 3, 2025

Uh oh!

nodejs-github-bot commented Oct 3, 2025

Uh oh!

Uh oh!

nodejs-github-bot commented Oct 4, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

lib: implement passive listener behavior per spec by Amemome · Pull Request #59995 · nodejs/node · GitHub

Uh oh!

Uh oh!

lib: implement passive listener behavior per spec - #59995

lib: implement passive listener behavior per spec#59995nodejs-github-bot merged 1 commit intonodejs:mainnodejs/node:mainfrom Amemome:lib/implement-events-passive-listener-specAmemome/node:lib/implement-events-passive-listener-specCopy head branch name to clipboard

Conversation

Amemome commented Sep 24, 2025

Uh oh!

nodejs-github-bot commented Sep 24, 2025

Uh oh!

codecov Bot commented Sep 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

nodejs-github-bot commented Oct 2, 2025

Uh oh!

mcollina left a comment

Choose a reason for hiding this comment

Uh oh!

nodejs-github-bot commented Oct 3, 2025

Uh oh!

mcollina commented Oct 3, 2025

Uh oh!

nodejs-github-bot commented Oct 3, 2025

Uh oh!

Uh oh!

nodejs-github-bot commented Oct 4, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

lib: implement passive listener behavior per spec#59995
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
Amemome:lib/implement-events-passive-listener-spec

codecov Bot commented Sep 24, 2025 •
edited
Loading