Closed
Bug 1895198
Opened 3 months ago
Closed 2 months ago
Move the assert !LoadWasOpened() in HttpBaseChannel::ExplicitSetUploadStreamLength to HttpChannelChild
Categories
(Core :: Networking, task, P2)
Core
Networking
Tracking
()
RESOLVED
FIXED
127 Branch
Tracking | Status | |
---|---|---|
firefox127 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
Details
(Whiteboard: [webdriver:m11][necko-triaged])
Attachments
(1 file)
For WebDriver BiDi, we need to modify the request body late (currently in http-on-before-connect), but this currently fails on the following assert:
https://searchfox.org/mozilla-central/rev/e69f323af80c357d287fb6314745e75c62eab92a/netwerk/protocol/http/HttpBaseChannel.cpp#1323
After discussing with :valentin, since BiDi only ever calls this from the parent process and this assert is only relevant for the content process, we should be able to split the implementation between HttpBaseChannel and HttpChannelChild, and only keep the assert in the Child version.
Assignee | ||
Updated•3 months ago
|
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Assignee | ||
Updated•3 months ago
|
Component: WebDriver BiDi → Networking
Product: Remote Protocol → Core
Assignee | ||
Comment 1•3 months ago
|
||
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ead376db22e7 Add ExplicitSetUploadStreamLength to HttpChannelChild and move LoadWasOpened assert r=valentin,necko-reviewers
Updated•3 months ago
|
Severity: -- → N/A
Whiteboard: [webdriver:m11] → [webdriver:m11][necko-triaged]
Assignee | ||
Updated•3 months ago
|
Points: --- → 2
Comment 3•2 months ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
status-firefox127:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 127 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•