Track input UMA metrics in telemetry |
||||
Issue descriptionThe critical UMA metrics the input team monitors (e.g. TimeToScrollUpdateSwapBegin2 etc.) should ideally be track in telemetry tests too, so that regressions in the pipeline can be detected and addressed earlier.
,
Aug 28
Another issue is that some of the regressions happen on 99th percentile only and there is no guarantee for a written perf test to reproduce these regressions.
,
Aug 28
Even for the regression at 99th %ile, I think reproing in test devices isn't impossible (e.g. https://bugs.chromium.org/p/chromium/issues/detail?id=867581#c9). So having some coverage in telemetry would be useful (would also allow bisecting using pinpoint). For issue 867581 , it doesn't look like the telemetry metric changed much around the time of the offending CL [1]. Is there anything we could do so regressions like this are easier to detect from chromeperf? [1] https://chromeperf.appspot.com/report?sid=6475328454d3250a35ea46a102af0c45d8d81417709d8169e6c1186bf0316619&start_rev=556352&end_rev=571551
,
Aug 28
>Even for the regression at 99th %ile, I think reproing in test devices isn't impossible (e.g. https://bugs.chromium.org/p/chromium/issues/detail?id=867581#c9). So having some coverage in telemetry would be useful (would also allow bisecting using pinpoint). The repo in that case was possible since we checked the UKM data to find a device and a URL that was showing the regression and then tried to come up with a repo. We already have telemetry tests for scroll latency (https://cs.chromium.org/chromium/src/third_party/catapult/telemetry/telemetry/web_perf/metrics/smoothness.py?type=cs&sq=package:chromium&q=ComputeFirstGestureScrollUpdateLatencies&g=0&l=208) which gets executed over some popular websites, but there is no guarantee that this test (or any similar tests) catch the 99th percentile regressions.
,
Aug 28
That particular metric seems to report only the first scroll-update. If we reported all scroll-updates, would that better represent the performance?
,
Aug 28
As far as I have noticed the scroll regression that we see are either only related to the first update (regression on ScrollBegin metrics) or related to updates in general (regressions both on ScrollBegin and ScrollUpdate metrics), either way we would see the regression on ScrollBegin metrics and first-update telemetry should be enough. nzolghadr@ or tdresser@ might know better though: Is it possible to have regressions that affect only non-first scroll updates? (e.i. regression on ScrollUpdate metrics that we don't see on ScrollBegin metrics.)
,
Sep 26
I'm not aware of any regressions which would only effect updates. However, a large proportional regression in updates is likely to be small relative to the absolute latency for a scroll begin, which means there are regressions for updates (and scroll begins) which get lost in the noise for scroll begins.
,
Jan 15
Navid, do you have any update here?
,
Jan 15
This will be done in this quarter.
,
Jan 15
|
||||
►
Sign in to add a comment |
||||
Comment 1 by nzolghadr@chromium.org
, Aug 28