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

Issue 671646 link

Starred by 1 user
Status: WontFix
Owner:
sergeyu@chromium.org
Closed: Dec 2016
Cc:
asapersson@chromium.org
sergeyu@chromium.org
terelius@chromium.org
█ danilchap@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

21.5%-25.6% improvement in webrtc_perf_tests at 15346:15424

Project Member Reported by terelius@chromium.org, Dec 6 2016 
This is probably a duplicate of some other bug, but I can't tell which one.

Blamelist is here:
https://chromium.googlesource.com/external/webrtc/+log/5c13c33f3073bba420c1dbd5548559d8ec59c406..53e09800f4af829f4e93fba7fc1832d13f20b0d1

Comment 1 by terelius@chromium.org, Dec 6 2016 

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=671646

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgICgl6birAoM,agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgICg54-ZpgoM


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

webrtc-android-tests-nexus9

Comment 2 by terelius@chromium.org, Dec 6 2016

Cc: sergeyu@chromium.org asapersson@chromium.org
Primary candidates are:

e545e5d RtpPacketizer::NextPacket fills RtpPacket instead of just payload. by danilchap
9cef11b Fix exponential probing in ProbeController. by sergeyu
4eb03c7 Turn off error resilience for VP9 if no spatial or temporal layers are configured and NACK is enabled. by asapersson

Comment 3 by danilchap@chromium.org, Dec 6 2016

Cc: danilchap@chromium.org
Owner: ----
Status: Available (was: Assigned)
not ready to take blame for these improvements: screenshare_slides test use VP8 codec which should be unaffected by my CL

Comment 4 by terelius@chromium.org, Dec 6 2016

Owner: sergeyu@chromium.org
Probably unaffected by Åsa's CL too. Assigning to Sergey for now.

Comment 5 by asapersson@chromium.org, Dec 6 2016 

4eb03c7 should only affect VP9.

Comment 6 by sergeyu@chromium.org, Dec 7 2016

Status: WontFix (was: Available)
My change fixed previous regression, see bug 669421.
►

Sign in to add a comment