events: optimize EventTarget.addEventListener

[ronag]

events/eventtarget-add-remove-abort.js nListener=1 n=100000         ***    223.45 %      ±12.03% ±16.12% ±21.21%
events/eventtarget-add-remove-abort.js nListener=10 n=100000        ***    199.33 %       ±9.42% ±12.63% ±16.65%
events/eventtarget-add-remove-abort.js nListener=5 n=100000         ***    221.94 %       ±8.80% ±11.78% ±15.47%

Fixes: #55311

[nodejs-github-bot]

Review requested:

• @nodejs/performance

[ronag]

@benjamingr could we use a Set instead of linked list? At least in the non-weak case?

[KhafraDev]

Is there a reason we can't skip using addEventListener altogether and directly append the listener to the signal's abort algorithms?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.40%. Comparing base (d2ad9b4) to head (c934127).
Report is 53 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #55312      +/-   ##
==========================================
- Coverage   88.41%   88.40%   -0.02%     
==========================================
  Files         652      652              
  Lines      186594   186788     +194     
  Branches    36054    36037      -17     
==========================================
+ Hits       164980   165126     +146     
- Misses      14883    14925      +42     
- Partials     6731     6737       +6     

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

... and 69 files with indirect coverage changes

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[mcollina]

lgtm

[mcollina]

@ronag linting is failing