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

Issue 845243 link

Starred by 3 users
Status: Fixed
Owner:
mlamouri@chromium.org
Closed: Jun 2018
Cc:
█ fsam...@chromium.org
sande...@chromium.org
liber...@chromium.org
pfeldman@chromium.org
mlamouri@chromium.org
lethalantidote@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

35.7% regression in media.desktop at 559172:559310

Project Member Reported by sandersd@google.com, May 21 2018 
See the link to graphs below.
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, May 21 2018 

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

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


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

chromium-rel-win7-dual
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, May 21 2018

Cc: lethalantidote@chromium.org
Owner: lethalantidote@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/128e4e9c240000

Allows cc::Surfaces for video to work with software compositing. by lethalantidote@chromium.org
https://chromium.googlesource.com/chromium/src/+/298ef248dd85dc2f747982ad1ac3589124873c92

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 4 by mlamouri@chromium.org, May 22 2018

Cc: liber...@chromium.org

Comment 5 by mlamouri@chromium.org, May 25 2018

Cc: fsam...@chromium.org

Comment 7 by mlamouri@chromium.org, Jun 11 2018

Owner: mlamouri@chromium.org
Status: Started (was: Assigned)
It sounds that https://chromium.googlesource.com/chromium/src.git/+/74be4bc85264530ca32901400e3a50f9bf60dae3 may have fixed this. Will see if I can get confirmation.
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Jun 12 2018

Cc: pfeldman@chromium.org mlamouri@chromium.org
📍 Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/16b802e9240000

DevTools: allow network interceptor to access the cookie header. by pfeldman@chromium.org
https://chromium.googlesource.com/chromium/src/+/78044c8325af946ace4dd560856572a76af07386
3.2e+07 → 3.218e+07 (+1.845e+05)

VideoSurfaceLayer: do not send a new frame until an ACK is received. by mlamouri@chromium.org
https://chromium.googlesource.com/chromium/src/+/74be4bc85264530ca32901400e3a50f9bf60dae3
3.218e+07 → 2.792e+07 (-4.263e+06)

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 9 by mlamouri@chromium.org, Jun 12 2018

Status: Fixed (was: Started)
Closing as my CL actually solved this regression.
►

Sign in to add a comment