diff options
author | Koen <41021050+vrugtehagel@users.noreply.github.com> | 2023-06-05 02:03:44 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-06-04 18:03:44 -0600 |
commit | adf41edda12a26a84cb8b4252404aae2a9e7ae03 (patch) | |
tree | bea8a0a96318df38674620f726fef8d6064f2f0b /cli/worker.rs | |
parent | 34dac6c6efa75f38c29031a65db1ee3332a67259 (diff) |
fix(ext/web): Copy EventTarget list before dispatch (#19360)
Related issue: https://github.com/denoland/deno/issues/19358.
This is a regression that seems to have been introduced in
https://github.com/denoland/deno/pull/18905. It looks to have been a
performance optimization.
The issue is probably easiest described with some code:
```ts
const target = new EventTarget();
const event = new Event("foo");
target.addEventListener("foo", () => {
console.log('base');
target.addEventListener("foo", () => {
console.log('nested');
});
});
target.dispatchEvent(event);
```
Essentially, the second event listener is being attached while the `foo`
event is still being dispatched. It should then not fire that second
event listener, but Deno currently does.
Diffstat (limited to 'cli/worker.rs')
0 files changed, 0 insertions, 0 deletions