New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 594495 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

WebRTC BWE is a bit too sensitive in M50.

Project Member Reported by holmer@chromium.org, Mar 14 2016

Issue description

<b>Version: <Kenneth, what is the frequency?></b>
<b>OS: <please tell me it's not XP></b>

What steps will reproduce the problem?

Run a WebRTC call (apprtc demo) in loopback on a mobile device, such as Galaxy S5. At times it can be seen that the BWE doesn't ramp up as expected.

What is the expected output? What do you see instead?

BWE can get stuck at low bitrates. Instead it should be able to ramp up to 2 Mbps.


Please use labels and text to provide additional information.

 

Comment 1 by holmer@chromium.org, Mar 14 2016

Corresponding webrtc issue: https://bugs.chromium.org/p/webrtc/issues/detail?id=5646

Comment 3 by holmer@chromium.org, Mar 14 2016

Cc: tnakamura@chromium.org mflodman@chromium.org
Cc: srcv@chromium.org cfraasch@chromium.org srnarayanan@chromium.org adwarakanathan@chromium.org
Should the WebRTC fix mentioned in #2 be merged into M50? If so, what kind of sanity checks should we run with Canary Chrome before requesting a merge?

Comment 5 by holmer@chromium.org, Mar 14 2016

Yes, that's the plan.

I haven't been able to reliably reproduce the issue with Chrome, but for sanity I think it would be good to just verify that the BWE seems to be working. We could run a test with netem and bandwidth limited to 1 Mbps and see that it looks OK.

Comment 6 by holmer@chromium.org, Mar 14 2016

Labels: Merge-Request-50

Comment 7 by tin...@google.com, Mar 15 2016

Labels: -Merge-Request-50 Merge-Approved-50 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M50 (branch: 2661)

Comment 8 by holmer@chromium.org, Mar 15 2016

I ran some tests with netem and verified that webrtc still adapted to the bandwidth limitation.

Comment 9 by holmer@chromium.org, Mar 15 2016

Labels: -Merge-Approved-50 Merge-Merged
Status: Fixed (was: Assigned)
Merge done:

https://chromium.googlesource.com/external/webrtc/+/55d15f82f68f8f4daece1000e8641b45898b8b9a
Labels: -Hotlist-Merge-Approved

Sign in to add a comment