1926294 - Add tests for network requests using keepAlive: true

Status: NEW

(Remote Protocol :: WebDriver BiDi, task, P3)

Description

[Julian Descottes [:jdescottes]]

Similar to devtools Bug 1926042, we currently skip fetch requests with keepAlive: true, because we can't find the browsing context id associated with them.

Because of this our network module will bail out of emitting the events at: https://searchfox.org/mozilla-central/rev/dfc3050503739883ce6d17e8365bc35410106aba/remote/webdriver-bidi/modules/root/network.sys.mjs#1691-1698

const browsingContext = lazy.TabManager.getBrowsingContextById(
  request.contextId
);
if (!browsingContext) {
  // Do not emit events if the context id does not match any existing
  // browsing context.
  return;
}

keepAlive was enabled in Firefox for all channels in Bug 1923044

Comment 1

[Andrey Lushnikov]

just in case, there's a speculative fix to add the missing association: https://github.com/aslushnikov/juggler/commit/30b05ab1132a6023c4a281bf568496f05c4381c8

Comment 3

[Julian Descottes [:jdescottes]]

The keepAlive requests should now create the expected events thanks to

• Bug 1897424 - Add capability to send NotifyNetworkMonitorAlternateStack for requests via PFetch originating from the main thread.

We can keep this bug to add a test but I think it can decrease to P3.