Closed
Bug 1899789
Opened 9 months ago
Closed 9 months ago
Add basic bidi network tests for service worker requests
Categories
(Remote Protocol :: WebDriver BiDi, defect, P2)
Remote Protocol
WebDriver BiDi
Tracking
(firefox-esr115 unaffected, firefox126 unaffected, firefox127 unaffected, firefox128 fixed)
RESOLVED
FIXED
128 Branch
Tracking | Status | |
---|---|---|
firefox-esr115 | --- | unaffected |
firefox126 | --- | unaffected |
firefox127 | --- | unaffected |
firefox128 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
(Regression)
Details
(Keywords: regression, Whiteboard: [webdriver:m11], [wptsync upstream])
Attachments
(1 file)
Using http-on-before-connect for BiDi network events actually introduced issues for requests coming from service workers (see blocking bug 1899563).
We should add wdspec tests with service worker requests to catch such regressions.
If the patch currently attached to Bug 1899563 is fine and lands, it should be enough to fix the issue and so this bug would only be about adding tests.
If not, this will require additional fixes in the DevTools NetworkObserver so that we don't hook the ResponseListener as soon as the network event is created.
Comment 1•9 months ago
|
||
Set release status flags based on info from the regressing bug 1850680
status-firefox126:
--- → unaffected
status-firefox127:
--- → unaffected
status-firefox128:
--- → affected
status-firefox-esr115:
--- → unaffected
Assignee | ||
Comment 2•9 months ago
|
||
depends on D212070
Note this needs a full build for the tests to pass due to the dependency on D212070.
Updated•9 months ago
|
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Updated•9 months ago
|
Summary: Cannot receive network events coming from service worker requests → Add basic bidi network tests for service worker requests
Assignee | ||
Updated•9 months ago
|
Severity: -- → S3
Points: --- → 2
Priority: -- → P2
Whiteboard: [webdriver:m11]
Pushed by jdescottes@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d2b4393709e7
[wdspec] Add basic bidi network tests for service worker requests r=webdriver-reviewers,whimboo
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/46579 for changes under testing/web-platform/tests
Whiteboard: [webdriver:m11] → [webdriver:m11], [wptsync upstream]
Comment 5•9 months ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 128 Branch
Upstream PR merged by moz-wptsync-bot
Upstream PR merged by moz-wptsync-bot
You need to log in
before you can comment on or make changes to this bug.
Description
•