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

Network and Fetch domain activation #1080

Closed
thiagowfx opened this issue Jul 24, 2023 · 4 comments
Closed

Network and Fetch domain activation #1080

thiagowfx opened this issue Jul 24, 2023 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@thiagowfx
Copy link
Contributor

thiagowfx commented Jul 24, 2023

Currently, the network domain is incorrectly enabled.

The network domain should be enabled via the CDP client/target, only when the browsing context has...

  • ...subscriptions to network. events, and/or
  • ...global subscription to the network namespace

It's important to not do Network.enable globally, otherwise we'd emit too many unnecessary CDP events, when the client may not be interested in them.

Edit(2023-09-11): @Lightning00Blade: It seems that if we never enable interception we always call Fetch.disable on target create, I think it would be good it we omit that that way its a little bit faster for WPT and other test.

(Update by @Lightning00Blade) Investigated the performance cost related to always enabling the Fetch domain can be found at go/webdriver-bidi:network-interception-performance-delta.
The biggest delta that we observed is around 8-9%, when navigating to a page and continuing the request as normal.

@thiagowfx
Copy link
Contributor Author

cc @sadym-chromium FYI as this slightly relates to supporting multiple BiDi/CDP sessions.

@sadym-chromium
Copy link
Collaborator

cc @sadym-chromium FYI as this slightly relates to supporting multiple BiDi/CDP sessions.

ack

thiagowfx pushed a commit that referenced this issue Oct 10, 2023
thiagowfx pushed a commit that referenced this issue Oct 10, 2023
thiagowfx pushed a commit that referenced this issue Oct 10, 2023
Furthermore, update some E2E tests so that they do not rely on CDP
events.

Bug #644, #1080
Co-authored-by: Maksim Sadym <[email protected]>

Re-created from #1400.
@thiagowfx
Copy link
Contributor Author

Fetch is partially resolved via #1409.

@Lightning00Blade
Copy link
Collaborator

This should be resolved and we should be enabling both Network and Fetch only when needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants