Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removing a later event listener while an earlier listener for the same event is running doesn't stop it from being called #25479

Open
pshaughn opened this issue Jan 10, 2020 · 1 comment
Labels
A-content/dom Interacting with the DOM from web content

Comments

@pshaughn
Copy link
Member

Servo currently has no "removed" flag for its event listeners, as specified in https://dom.spec.whatwg.org/#event-listener-removed . I would add it myself as part of #22783 cleanup, but there's a complicating factor: The removed flag lives on an "event listener", lowercase letters, not an EventListener, and Servo event targets currently expose a list of capital-EL Event Listeners to the dispatch algorithm. The lowercase concept is approximately Servo's struct EventListenerEntry, but it's not an exact mapping and code that returns EventListeners can't just be drop-in replaced with code that returns EventListenerEntries.

@pshaughn
Copy link
Member Author

This is almost certainly a cause of other already-open issues, but I didn't happen to see any specific one that's definitely it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-content/dom Interacting with the DOM from web content
Projects
Development

No branches or pull requests

2 participants