Closed Bug 1899789 Opened 4 months ago Closed 4 months ago

Add basic bidi network tests for service worker requests

Categories

(Remote Protocol :: WebDriver BiDi, defect, P2)

defect
Points:
2

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.

Set release status flags based on info from the regressing bug 1850680

depends on D212070

Note this needs a full build for the tests to pass due to the dependency on D212070.

Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Summary: Cannot receive network events coming from service worker requests → Add basic bidi network tests for service worker requests
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]
Status: ASSIGNED → RESOLVED
Closed: 4 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.

Attachment

General

Created:
Updated:
Size: