New issue
Advanced search Search tips

Issue 811334 link

Starred by 1 user
Status: WontFix
Owner:
yhirano@chromium.org
Closed: May 2018
Cc:
yhirano@chromium.org
sullivan@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

13.3%-633.8% regression in system_health.common_desktop at 535028:535336

Project Member Reported by sullivan@chromium.org, Feb 12 2018 
See the link to graphs below.
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Feb 12 2018 

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

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


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

chromium-rel-mac11-pro
chromium-rel-win10
chromium-rel-win7-gpu-nvidia
win-high-dpi
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Feb 12 2018 

📍 Couldn't reproduce a difference.
https://pinpoint-dot-chromeperf.appspot.com/job/14fe883d840000
Project Member

Comment 7 by 42576172...@developer.gserviceaccount.com, Feb 12 2018 

📍 Couldn't reproduce a difference.
https://pinpoint-dot-chromeperf.appspot.com/job/12a8652d840000
Project Member

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

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

Update RendererSideResourceScheduler testing config (Enabled_2_1024) by yhirano@chromium.org
https://chromium.googlesource.com/chromium/src/+/1f25bbc692f06964b5e10c7312f9f81377cdea32

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

Comment 9 by yhirano@chromium.org, Feb 13 2018

Cc: -jwd@chromium.org
Project Member

Comment 10 by 42576172...@developer.gserviceaccount.com, Feb 13 2018

Cc: jwd@chromium.org
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/12976b2d840000

Update RendererSideResourceScheduler testing config (Enabled_2_1024) by yhirano@chromium.org
https://chromium.googlesource.com/chromium/src/+/1f25bbc692f06964b5e10c7312f9f81377cdea32

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

Comment 11 by yhirano@chromium.org, Feb 13 2018

Cc: -jwd@chromium.org
Project Member

Comment 13 by 42576172...@developer.gserviceaccount.com, Feb 13 2018 

📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/16b9ebf5840000

ResourceLoadScheduler: Count the number of throttlable requests by yhirano@chromium.org
https://chromium-review.googlesource.com/c/chromium/src/+/915882/4

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

Comment 17 by 42576172...@developer.gserviceaccount.com, Feb 15 2018 

📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/11d3f97d840000

[DO NOT COMMIT] 2_1024 => disabled by yhirano@chromium.org
https://chromium-review.googlesource.com/c/chromium/src/+/920963/1

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

Comment 18 by 42576172...@developer.gserviceaccount.com, Feb 15 2018 

📍 Couldn't reproduce a difference.
https://pinpoint-dot-chromeperf.appspot.com/job/16d3f97d840000
Project Member

Comment 19 by 42576172...@developer.gserviceaccount.com, Feb 15 2018 

📍 Couldn't reproduce a difference.
https://pinpoint-dot-chromeperf.appspot.com/job/1491a793840000

Comment 20 by yhirano@chromium.org, May 14 2018 

 Issue 811336  has been merged into this issue.

Comment 21 by yhirano@chromium.org, May 14 2018

Status: WontFix (was: Assigned)
I'd like to close this issue as WONTFIX because the change is net neutral for performance, according to the finch result. See https://docs.google.com/document/d/1oreWB98RmEhqGIM9vQgOs9s7llBLxHsxubhNyN8Cmrg/edit for details.
►

Sign in to add a comment