Issue metadata
Sign in to add a comment
|
164.1% regression in system_health.common_desktop at 546064:546105 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Mar 29 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/11bd877b440000
,
Mar 29 2018
📍 Couldn't reproduce a difference. https://pinpoint-dot-chromeperf.appspot.com/job/11bd877b440000
,
Apr 5 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/1295b8d8c40000
,
Apr 5 2018
Expanding the range since the value bumps up over multiple data points.
,
Apr 5 2018
📍 Found significant differences after each of 2 commits. https://pinpoint-dot-chromeperf.appspot.com/job/1295b8d8c40000 Add components_perftests to chromium.perf.json by csharrison@chromium.org https://chromium.googlesource.com/chromium/src/+/a33ba78b7502b9bfad38cfd48fa5f8ff698bac79 Fix startup tracing in renderers by chiniforooshan@chromium.org https://chromium.googlesource.com/chromium/src/+/f192e8a25e1937336d9cc16f9273b1d13b99530a Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Apr 17 2018
I believe this is not a regression. It is a logging issue that my CL fixed. - My CL fixed startup tracing, which means now we see more early trace events. - The way timeToFirstPaint is calculated is that for every firstPaint event of the main frame we find the corresponding navigationStart event before it and compute the duration from navigationStart to firstPaint. - Before my CL, the first navigationStart event was not logged and so one large timeToFirstPaint was not reported. After my CL, we see all navigationStart events and so report more timeToFirstPaint values. +dproy@ for verification.
,
Apr 17 2018
,
Apr 17 2018
Issue 827149 has been merged into this issue.
,
Apr 17 2018
Issue 827150 has been merged into this issue.
,
Apr 17 2018
Verified c#7. Although it looks like we now have two firstPaint samples, and the the URL for the new first paint is "https://myaccount.google.com/?pli=1" (so this is the login page), while the URL for the other first paint is "mail.google.com/mail". Someone who has more context into this benchmark should verify this was the original intention.
,
Apr 18 2018
+Ned and Kouhei about the multiple urls in #11.
,
Apr 24 2018
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Mar 29 2018