Issue metadata
Sign in to add a comment
|
29.2%-35.9% regression in blink_perf.parser at 554638:554648 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
May 1 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/1484343dc40000
,
May 1 2018
📍 Found significant differences after each of 3 commits. https://pinpoint-dot-chromeperf.appspot.com/job/1484343dc40000 [signin] Add metrics for refresh token revocation. by droger@chromium.org https://chromium.googlesource.com/chromium/src/+/c6bf01d352c240cdda3c0e9c7595db214a6eeb74 Update FileManagerBrowserTestBase to avoid unprocessed messages by noel@chromium.org https://chromium.googlesource.com/chromium/src/+/a59359507e747e61c620610cd4c9fb110ecba5ac Import wpt@cc364d6f98b111e488aa1d88064af21566ec4184 by blink-w3c-test-autoroller@chromium.org https://chromium.googlesource.com/chromium/src/+/ec2d4ae90a08738eb616bb2ab1c0fee5a551e0d6 Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Oct 10
Based on Pinpoint data, it looks like the first and second commits caused the regression. Since the first one regressed more, let's start investigating that.
,
Oct 11
I don't believe my CL could have performance impact: it only adds histograms which are rarely called. These are called on browser signout operations, when a OAuth2 token is revoked. This should not affect the renderer, and is probably not even happening in blink tests at all.
,
Oct 11
Looks like the graphs recovered. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, May 1 2018