Issue metadata
Sign in to add a comment
|
Startup.FirstCommitNavigationTime regression on Android M |
||||||||||||||||||||||
Issue description
Measurements done on a local build for Startup.FirstCommitNavigationTime
June 20 refs/heads/master@{#400830} 3500ms
ToT refs/heads/master@{#402381} 12875ms
,
Jun 28 2016
pkotwicz, not sure why you gave this to me. If you need help with a performance regression, contact the perf sheriffs please.
,
Jun 28 2016
lanwei appears to be the current perf sheriff Anecdotally, I did notice that startup is dreadfully slow since yesterday (although I was OOO for a month before that so dont' knwo when the regression started)
,
Jun 28 2016
Thanks for reporting! I think this is a dupe of bug 623956?
,
Jun 28 2016
sullivan@ I am not sure if it is a dupe of bug 623956. I did not notice a significant perf hit on Android L but did on Android M. bug 623956 is about Android L (and Mac)
,
Jun 29 2016
The stats for the 75th percentile of Startup.FirstCommitNavigationTime are really bad and seem to match what I have been seeing locally https://uma.googleplex.com/p/chrome/timeline_v2/?sid=db2f257166bb22a50ba74fcd2379d753
,
Jun 29 2016
+pasko since you're familiar with android startup +perezju we don't have a bot tracking startup on M, right?
,
Jun 30 2016
No, I don't think we have any bots running perf tests on M today.
,
Jul 4 2016
Random thoughts, sorted by importance: 0. Does it visually start longer than 13 seconds from an intent or it's the metric that likely got broken? 1. anything special in logcat for the slow build? 2. Since you have the revision ranges, can you do a local bisect on your local device running M build? (last time I tried bisect script did not work locally - in this case file bugs against speed infra plz) 3. might be related to Issue 622213, it also mentions M
,
Jul 4 2016
I did a manual bisect. The startup regression was fixed by https://codereview.chromium.org/2111233002
,
Jul 4 2016
:( Yeah, that regression should be component-mode only, so hopefully it didn't show on any meaningful telemetry tests. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by pkotw...@chromium.org
, Jun 28 2016Owner: kerz@chromium.org
Status: Assigned (was: Untriaged)