Closed Bug 2035707 Opened 19 days ago Closed 9 days ago

[wpt-sync] Sync PR 59463 - WebTransport: Add congestionControl hint (W3C spec compliance)

Categories

(Core :: Networking: HTTP, task, P4)

task

Tracking

()

RESOLVED FIXED
152 Branch
Tracking Status
firefox152 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Attachments

(1 file)

Sync web-platform-tests PR 59463 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/59463
Details from upstream follow.

Himanshu Panwar <himpanwar@microsoft.com> wrote:

WebTransport: Add congestionControl hint (W3C spec compliance)

Implement the WebTransportCongestionControl enum, constructor option,
and readonly attribute per the W3C WebTransport spec.

The spec defines congestionControl as a hint for the preferred
congestion control algorithm — an enum {"default", "throughput",
"low-latency"} passed via WebTransportOptions and readable as a readonly
attribute on WebTransport.

Plumbing through all layers:

  • Blink: New enum IDL, option, getter, Blink→Mojo conversion
  • Mojo IPC: New enum + parameter on Connect() and CreateWebTransport()
  • Browser: Thread through throttle/security callback chain
  • Network service: Convert Mojo enum → net::CongestionControlHint
  • net/QUIC: AdjustSendAlgorithm() maps "throughput" → BBR v2,
    "low-latency" → BBR v1, "default" → no change (Cubic)

A global kWebTransportCongestionControl feature flag overrides
per-connection hints for Chrome-wide experiments.

Bug: 501268547
Change-Id: I6d0d7163fdb777058b940d9b9f699ee65c631dfe
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7743543
Reviewed-by: Daniel Cheng \<dcheng@chromium.org>
Reviewed-by: Adam Rice \<ricea@chromium.org>
Commit-Queue: Himanshu Panwar \<himpanwar@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#1620659}

Component: web-platform-tests → Networking: HTTP
Product: Testing → Core
Attached file Notable wpt changes

CI Results

Ran 11 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 69 tests and 10 subtests

Status Summary

Firefox

OK : 4[GitHub] 56[Gecko-windows11-64-25h2-opt] 63[Gecko-linux2404-64-debug, Gecko-windows11-32-25h2-debug, Gecko-windows11-64-25h2-debug] 64[Gecko-linux2404-64-opt, Gecko-windows11-32-25h2-opt]
PASS : 1[Gecko-linux2204-64-wayland-debug, Gecko-linux2204-64-wayland-opt] 16[GitHub] 506[Gecko-windows11-64-25h2-opt] 560[Gecko-windows11-32-25h2-debug, Gecko-windows11-64-25h2-debug] 570[Gecko-linux2404-64-debug, Gecko-linux2404-64-opt, Gecko-windows11-32-25h2-opt]
FAIL : 308[Gecko-windows11-64-25h2-opt] 316[Gecko-linux2404-64-debug, Gecko-linux2404-64-opt, Gecko-windows11-32-25h2-debug, Gecko-windows11-32-25h2-opt, Gecko-windows11-64-25h2-debug]
PRECONDITION_FAILED: 3
ERROR : 4

Chrome

OK : 4
FAIL : 16

Safari

OK : 4
PASS : 16

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

(See attachment for full changes)

Pushed by wptsync@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/620f1c114886 https://hg.mozilla.org/integration/autoland/rev/6b169090bab1 [wpt PR 59463] - WebTransport: Add congestionControl hint (W3C spec compliance), a=testonly
Status: NEW → RESOLVED
Closed: 9 days ago
Resolution: --- → FIXED
Target Milestone: --- → 152 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: