New issue
Advanced search Search tips

Issue 704491 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

2.1% regression in webrtc_perf_tests at 17321:17321

Project Member Reported by asapersson@chromium.org, Mar 23 2017

Issue description

See the link to graphs below.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=704491

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDgzJnR4QoM


Bot(s) for this bug's original alert(s):

webrtc-win-large-tests
Owner: nisse@chromium.org

Comment 3 by nisse@chromium.org, Mar 23 2017

Cc: holmer@chromium.org
Stefan, do you see any reason why the congestion controller split would cause this? There might be some slight change in behaviour.

I also note that the graph went down a few weeks ago, see https://bugs.chromium.org/p/chromium/issues/detail?id=697089. 

Comment 4 by holmer@chromium.org, Mar 28 2017

Status: Assigned (was: Untriaged)
My guess would be that it's because we're no longer calling OnRttUpdate on the receive-side cc, but only the send-side:

call.cc:347 call_stats_->RegisterStatsObserver(&send_side_cc_);

Seems like a bug.


Comment 5 by nisse@chromium.org, Mar 28 2017

I'll prepare a cl to address that.

Comment 8 by nisse@chromium.org, Mar 29 2017

The supposed fix is now landed, but it seems to have made no change to these graphs. Not sure what to try next.

Sign in to add a comment