New issue
Advanced search Search tips

Issue 901853 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 6
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Merge WebRTC CL to M71: JitterEstimator: Remove old LowRate exp and add trial for upper bound

Project Member Reported by sprang@chromium.org, Nov 5

Issue description

This is a merge request for
https://webrtc-review.googlesource.com/q/I51ad279af156aba1b5cc75ae203334a34bce9d48

This change reduces how quickly the jitter estimator reacts to large
frames, which can happen especially with screenshare and sudden content
changes.

In extreme cases a few errant frames could previously cause several
seconds of delay to be added to the render buffer on the receive side.

The change can be considered low risk, especially since it also includes
a field trial flag so we can tweak how outliers are capped by this
change.

The change has been in Canary since 72.0.3599.0 which was released on
November 1st. Everything looking good so far.
 
Project Member

Comment 1 by sheriffbot@chromium.org, Nov 5

Labels: -Merge-Request-71 Hotlist-Merge-Review Merge-Review-71
This bug requires manual review: M71 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: benmason@(Android), kariahda@(iOS), kbleicher@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Review-71 Merge-Approved-71
Approving merge to M71 branch 3578 based on comment #0.Please merge ASAP. Thank you.
Pls merge your change to M71 branch 3578 ASAP so we can pick it up for this week beta release. Thank you.
Status: Fixed (was: Assigned)
This has been merged: https://webrtc-review.googlesource.com/c/src/+/109560
Labels: -Merge-Approved-71 Merge-Merged-71

Sign in to add a comment