Implement SendOrder for WebTransportSendStreams
Categories
(Core :: Networking: HTTP, enhancement, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox119 | --- | fixed |
People
(Reporter: jesup, Assigned: jesup)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
(Keywords: dev-doc-complete, Whiteboard: [necko-triaged])
Attachments
(1 file)
We don't currently implement SendOrder for outgoing streams. We'll need to accept it on creation, and pipe it through IPC to the protocol API when creating a stream. This may requite protocol API changes, and perhaps internal implementation changes in the protocol to prioritize by sendorder.
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Updated•1 year ago
|
Updated•1 year ago
|
Comment 1•1 year ago
|
||
Noting WIP doc if anyone wants to pick this up while I've backburnered my work on it.
Sorry for the google doc.
https://docs.google.com/document/d/1tyLBdFguyqTUuxom3Yo3IHXIlFDAmz_F2ZtWkvPi5Gw/edit#heading=h.lp6n8jdjm4e7
Assignee | ||
Comment 3•1 year ago
|
||
Note: the neqo implementation of this does not currently provide 'fairness'
between streams at the same priority
Updated•1 year ago
|
Comment 4•11 months ago
|
||
MDN docs note - support for this accidentally added to BCD. Now removed in https://github.com/mdn/browser-compat-data/pull/19983 but will need to be re-added when this completes (added dev-doc-needed).
Updated•8 months ago
|
Updated•8 months ago
|
Pushed by rjesup@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cd2b009cdee4 Implement WebTransport SendOrder at the DOM level r=kershaw,necko-reviewers
Comment 6•8 months ago
|
||
Backed out for causing bustage on Http3Session.cpp
- backout: https://hg.mozilla.org/integration/autoland/rev/fa2de257a6a77734d8b003f5af0dc8e01bd07359
- push: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&revision=cd2b009cdee4ebd5b41bbeb196d3de8ca49f2bb7
- failure log: https://treeherder.mozilla.org/logviewer?job_id=427608304&repo=autoland&lineNumber=23031
[task 2023-08-31T18:21:39.255Z] 18:21:39 INFO - In file included from Unified_cpp_protocol_http1.cpp:119:
[task 2023-08-31T18:21:39.255Z] 18:21:39 ERROR - /builds/worker/checkouts/gecko/netwerk/protocol/http/Http3Session.cpp:2487:14: error: unused variable 'rv' [-Werror,-Wunused-variable]
[task 2023-08-31T18:21:39.255Z] 18:21:39 INFO - nsresult rv = mHttp3Connection->WebTransportSetSendOrder(
[task 2023-08-31T18:21:39.255Z] 18:21:39 INFO - ^
[task 2023-08-31T18:21:39.255Z] 18:21:39 INFO - 1 error generated.
[task 2023-08-31T18:21:39.256Z] 18:21:39 ERROR - gmake[4]: *** [/builds/worker/checkouts/gecko/config/rules.mk:660: Unified_cpp_protocol_http1.o] Error 1
Pushed by rjesup@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0410a9861503 Implement WebTransport SendOrder at the DOM level r=kershaw,necko-reviewers
Comment 8•8 months ago
|
||
bugherder |
Comment 9•7 months ago
|
||
FF119 MDN Docs work for this tracked in https://github.com/mdn/content/issues/29300.
Assignee | ||
Updated•2 months ago
|
Description
•