Missing responseCompleted events for redirected document requests
Categories
(Remote Protocol :: WebDriver BiDi, defect, P1)
Tracking
(firefox116 fixed)
Tracking | Status | |
---|---|---|
firefox116 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
(Blocks 1 open bug)
Details
(Whiteboard: [webdriver:m7] [webdriver:relnote])
Attachments
(2 files)
STRs:
- subscribe to network.responseCompleted
- trigger a navigation to
https://google.com
ER:
Should get network.responseCompleted events for https://google.com/
and for https://www.google.com/
AR:
Events are missing
Assignee | ||
Comment 1•6 months ago
|
||
The issue is that for redirects we are trying to translate a browsingContext which is gone to a tabmanager id. This fails and we fail to emit the event.
I think the comment at https://searchfox.org/mozilla-central/rev/962a843f6d96283c45162c788dc72bf217fca7df/remote/shared/listeners/NetworkEventRecord.sys.mjs#280-289 is wrong. We should always be able to trust the tabmanager id to remain identical for a navigation.
Assignee | ||
Comment 2•6 months ago
|
||
Updated•6 months ago
|
Assignee | ||
Comment 3•6 months ago
|
||
Depends on D180854
Assignee | ||
Comment 4•6 months ago
|
||
In case we want to fix this, patches are attached, we'll triage this later.
Updated•6 months ago
|
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/313cfd6c08ee [bidi] Fix responseCompleted for redirects with browsing context swaps r=webdriver-reviewers,whimboo https://hg.mozilla.org/integration/autoland/rev/18cd337d0ef2 [wdspec] Add wdspec network test for navigation and redirect with COOP r=webdriver-reviewers,whimboo
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/40576 for changes under testing/web-platform/tests
Comment 7•6 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/313cfd6c08ee
https://hg.mozilla.org/mozilla-central/rev/18cd337d0ef2
Upstream PR was closed without merging
Upstream PR merged by moz-wptsync-bot
Assignee | ||
Comment 10•4 months ago
|
||
(we forgot priorities points and flags on this one)
Description
•