New issue
Advanced search Search tips

Issue 825518 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Mar 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

1.9%-17.4% regression in media.desktop at 544324:544617

Project Member Reported by liberato@google.com, Mar 24 2018

Issue description

borderline.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Mar 24 2018

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

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=c8775241e563652864e198e805db20fe2f719e409146b34e6aa6f897f83e0d61


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

chromium-rel-mac11-air
chromium-rel-mac11-pro
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Mar 24 2018

Cc: dcasta...@chromium.org xhw...@chromium.org dalecur...@chromium.org
Owner: dalecur...@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/12e7f43d440000

Switch back to uploading GpuMemoryBuffers in VideoRendererImpl. by dalecurtis@chromium.org
https://chromium.googlesource.com/chromium/src/+/5c175997b4d9c42c7c3ad40468e7b5ab9950ca69

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Status: WontFix (was: Assigned)
This is a revert to the old path, all metrics are the same as prior to the decoder based GMB approach. The interesting one is:

https://chromeperf.appspot.com/report?sid=6365d6948f0fefecfc49c2d31dad358eb6a8bedb1362f0b4d7065c32b165a249&rev=544617

~3W (!) improvement with decoder based approach. I think this is likely because it's using 100mb more of memory due to bursts of outputs which allow the system to sleep more between outputs.

Sign in to add a comment