New issue
Advanced search Search tips

Issue 842723 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Nov 26
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression

Blocking:
issue 838940



Sign in to add a comment

85.3% regression in system_health.common_desktop at 547881:554637

Project Member Reported by nzolghadr@chromium.org, May 14 2018

Issue description

See the link to graphs below.
 
Project Member

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

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

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


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

win-high-dpi
Project Member

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

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

__init__() got an unexpected keyword argument 'scope'
Cc: lukasza@chromium.org
Owner: lukasza@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/12d7b616a40000

Make --site-per-process the default on ToT via fieldtrial_testing_config by lukasza@chromium.org
https://chromium.googlesource.com/chromium/src/+/fb1ccf02ee8ca79e1404abfd3a3a7d540b7d2dbd
355.5 → 774.5 (+419.1)

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/12f10a05a40000

Make --site-per-process the default on ToT via fieldtrial_testing_config by lukasza@chromium.org
https://chromium.googlesource.com/chromium/src/+/fb1ccf02ee8ca79e1404abfd3a3a7d540b7d2dbd
342.5 → 761.9 (+419.4)

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Cc: taku...@chromium.org pmonette@chromium.org
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/16bc12cea40000

Disable test doubletap-to-jump-forwards-too-short on Win7 by takumif@chromium.org
https://chromium.googlesource.com/chromium/src/+/9e405e24c682904c03fcf620c388fb004b60ec0b
0.4392 → 0.4174 (-0.02181)

Adding fieldtrial config for the IncompatibleApplicationsWarning feature by pmonette@chromium.org
https://chromium.googlesource.com/chromium/src/+/7a86e22e588d7941bde54bdb4a8c58c23e502836
0.4174 → 0.6663 (+0.249)

Make --site-per-process the default on ToT via fieldtrial_testing_config by lukasza@chromium.org
https://chromium.googlesource.com/chromium/src/+/fb1ccf02ee8ca79e1404abfd3a3a7d540b7d2dbd
0.6598 → 0.7313 (+0.07145)

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Cc: -taku...@chromium.org
Components: Speed>Metrics>SystemHealthRegressions
friendly ping from today's perf sheriff
any update?
Blocking: 838940
Components: Internals>Sandbox>SiteIsolation
Ooops.  This felt through the cracks... :-/

That said, I am not sure if there are any actions we can take:

1. The (limited) UMA metrics don't show a regression in the aggregate results.  See http://go/site-isolation-performance#heading=h.u3cs4xppn2j2

2. It is quite possible that site-per-process is causing increased CPU usage in some specific scenarios (especially ones that require quite a bit of communication/coordination between OOPIFs and the browser process).  We've tried to understand some specific benchmarks/scenarios in  issue 835861 , but that didn't go very far.

Please let me know if there are some follow-ups steps that I could do, or that sheriffs could help with.


Ping! This bug has been flagged as stale by Chrome Speed Metrics bug triage, and it has an owner. Any update on this issue?
Status: WontFix (was: Assigned)
Resolving as WontFix per comments in #c13 (and lack of any advice for follow-up steps [asked for also in #c13]).

Sign in to add a comment