Skip to content

Switch to the new built-in BWE (and remove the old one) - #2010

Draft
ibc wants to merge 15 commits into
v3from
new-bwe-part-30
Draft

ibc wants to merge 15 commits into
v3from
new-bwe-part-30

Conversation

@ibc

@ibc ibc commented Oct 22, 2026

Copy link
Copy Markdown
Member

Details

  • Related to New Bandwidth Estimation stack #1895
  • Remove ms_use_builtin_bwe Meson option and every MS_USE_BUILTIN_BWE usage in the C++.
  • WebRtcTransportOptions and PlainTransportOptions: Add enableBwe, initialAvailableOutgoingBitrate, minOutgoingBitrate, maxOutgoingBitrate and maxIncomingBitrate.
  • Breaking change: WorkerSettings no longer takes libwebrtcFieldTrials.
  • Breaking change: The transport "bwe" trace event now reports desiredBitrate, minBitrate, maxBitrate and availableBitrate.
  • Remove all old BWE related classes.
  • Remove worker/deps/libwebrtc.
  • Remove abseil-cpp Meson subproject (it was only needed by the old libwebrtc dependency).

# Details

- Related to #1895
- Remove `ms_use_builtin_bwe` Meson option and every `MS_USE_BUILTIN_BWE` usage in the C++.
- `WebRtcTransportOptions` and `PlainTransportOptions`: Add `enableBwe`, `initialAvailableOutgoingBitrate`, `minOutgoingBitrate`, `maxOutgoingBitrate` and `maxIncomingBitrate`.
- **Breaking change:** `WorkerSettings` no longer takes `libwebrtcFieldTrials`.
- **Breaking change:** The transport "bwe" trace event now reports `desiredBitrate`, `minBitrate`, `maxBitrate` and `availableBitrate`.
- Remove all old BWE related classes.
- Remove `worker/deps/libwebrtc`.
- Remove `abseil-cpp` Meson subproject (it was only needed by the old `libwebrtc` dependency).
@ibc ibc mentioned this pull request Oct 22, 2026
46 of 56 tasks
@ibc

ibc commented Oct 23, 2026

Copy link
Copy Markdown
Member Author

NOTE: clang-tidy workflow fails due to so many files in the git diff (because worker/deps/libwebrtc/ has been removed). However I reverted them and the workflow passes so we are good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant