WebRTC's send-side bandwidth estimator incorrectly includes RTP headers in estimate |
|||||||
Issue descriptionSend-side BWE incorrectly includes RTP headers in its estimate, while the recv-side BWE only includes the payload, which causes the estimate to be higher than it should be. This in turn causes us to encode at a too high bitrate, which in the end causes us to send more than the link can handle.
,
Aug 23 2016
The fix for this just landed in webrtc, so it hasn't baked in Canary for a day yet. Still, I'm requesting a merge now since I'll be on vacation the following days. Philip will help out as needed.
,
Aug 23 2016
,
Aug 23 2016
Thank you holmer@. philipel@, please update the bug with canary result. Once it is well baked/verified and safe merge to M53, I will approve the merge. Thank you.
,
Aug 24 2016
[Automated comment] Less than 2 weeks to go before stable on M53, manual review required.
,
Aug 24 2016
How is the canary data looking?
,
Aug 25 2016
Tried it out just now and everything looks good.
,
Aug 25 2016
Approving merge to M53 branch 2785 based on comment #7. Please merge ASAP or latest by 5:00 PM PT, Friday (08/26) in order to make into the desktop Stable final build cut.
,
Aug 26 2016
CL for this merge: https://codereview.webrtc.org/2285733002/
,
Aug 26 2016
,
Aug 26 2016
Per comment #10, this is already merged to M53, so removing "Merge-Approved-53" label and applying "merge-merged-53" label.
,
Aug 29 2016
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by holmer@chromium.org
, Aug 23 2016