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

Faster event listener collection #2163

Closed
paulirish opened this issue May 5, 2017 · 0 comments
Closed

Faster event listener collection #2163

paulirish opened this issue May 5, 2017 · 0 comments

Comments

@paulirish
Copy link
Member

paulirish commented May 5, 2017

we have Two options for our remaining uses of event listener collection:

  1. DOMDebugger.getEventListeners now has a pierce option (added for us) so that we dont need to ask for each individual element's listener
  2. pavel was exploring reporting mutation listeners as violations: https://codereview.chromium.org/2863183003 not sure if this will land

also separately..

scriptParsed snapshot
instead of leaving scriptParsed open for the entire run we just use it for a quick snapshot.

between the Debugger.enable command and its callback/resolution, we'll get all the scriptParsed events flushed at us. So we can instantly disable it right after it resolves.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants