New issue
Advanced search Search tips

Issue 823942 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Oct 10
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression

Blocked on:
issue 824934



Sign in to add a comment

11.7% regression in blink_perf.bindings at 543893:543928

Project Member Reported by m...@chromium.org, Mar 20 2018

Issue description

See the link to graphs below.
 
Project Member

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

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

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


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

android-webview-nexus6
Project Member

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

😿 Pinpoint job stopped with an error.
https://pinpoint-dot-chromeperf.appspot.com/job/13178059440000

Comment 5 by m...@chromium.org, Mar 22 2018

Components: Speed>Bisection
Cannot triage further without a working bisect.

Comment 6 by m...@chromium.org, Mar 22 2018

Blockedon: 824934

Comment 7 by m...@chromium.org, Mar 22 2018

Components: -Speed>Bisection
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Mar 23 2018

😿 Pinpoint job stopped with an error.
https://pinpoint-dot-chromeperf.appspot.com/job/1708b079440000
Cc: thomasanderson@chromium.org aee@chromium.org michae...@chromium.org dpa...@chromium.org
Owner: thomasanderson@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/154a93a7440000

WebUI cr-toast: clear timeout before showing by aee@chromium.org
https://chromium.googlesource.com/chromium/src/+/0c1d42b7de12ced90144d045742412b4017ea890

Disable app shell installer when use_sysroot=false by thomasanderson@chromium.org
https://chromium.googlesource.com/chromium/src/+/01ab4cc587e13c2f2cf944e100ca101191b8c1ae

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Cc: -thomasanderson@chromium.org
Owner: ----
Status: Available (was: Assigned)
"Disable app shell installer when use_sysroot=false" should be unrelated to this regression.
😿 Pinpoint job stopped with an error.
https://pinpoint-dot-chromeperf.appspot.com/job/15cf404bc40000

'ReadHistogramsJsonValue' object has no attribute '_results_filename'
Cc: -michae...@chromium.org

Comment 15 by aee@chromium.org, May 14 2018

Cc: -aee@chromium.org

Comment 16 by m...@chromium.org, Jun 25 2018

Owner: aee@chromium.org
Status: Assigned (was: Available)
aee: PTAL. Looks like this should have been assigned to you (back in April!), but it fell off of everyone's radar. Looking at the graph, it seems the perf regression caused by the following CL still exists:

WebUI cr-toast: clear timeout before showing by aee@chromium.org
https://chromium.googlesource.com/chromium/src/+/0c1d42b7de12ced90144d045742412b4017ea890
@miu: The CL updates a UI component using by chrome:// pages. How can this be regressing blink performance? Are the blink_perf.bindings tests operate on chrome:// pages? If not, I think the bisection is not correct.

Also note that this particular code does not run on Android at all, which is referenced by comment #1.

Comment 18 by aee@chromium.org, Jun 29 2018

Cc: aee@chromium.org
Owner: ----
Status: Available (was: Assigned)
I'm removing myself as owner. I removed myself from cc back in May since this seemed unrelated to the cr-toast CL.
Status: WontFix (was: Available)
Long-term, seems things are noisy and mostly recovered.

Sign in to add a comment