New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 603517 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

first_gesture_scroll_update_latency telemetry metric is too noisy

Project Member Reported by tdres...@chromium.org, Apr 14 2016

Issue description

We only get a single sample per page in this metric currently, because we only perform a single scroll.

scrolling_tough_ad_cases already does this.

https://chromeperf.appspot.com/report?sid=202ed61b2d91658f93e2c8bf688dc53b851a48834e0afb4ff9f4a279ae5cd453

We might also want to think about how we approach loading jank. We might want to wait until load is finished to reduce noise.


 
See also  crbug.com/537446  - we might want a blocking relationship here.
Sorry, to clarify, scrolling_tough_ad_cases performs multiple scrolls per test.
We should consider doing this.

Comment 3 by nduca@chromium.org, Apr 14 2016

It seems like there're a few things to talk about here:

1) how to get stable values in face of noise better. right now we track absolute latency, in walltime. what if we could track the slices that were involved in the scroll, e.g. the critical path? We know that cpu time is lower noise. And, if we could track cpu time for input scrolls, we could be getting better data to help with, eg. advising android team on the impact of descheds on input latency.

2) whether to change when we scroll. if we do that then we're just avoiding the problem, and not realistically simulating what users do... maybe thats good because we would catch other regressions, but lets be clear about our intent and consequences of changing our definitions. :)
Yeah, there are a bunch of non-obvious decisions here.

The most obvious change is to get more samples per page load, which should be relatively straightforward.

1. Tracking the critical path might be doable, but it would make our telemetry benchmarks deviate from our UMA metrics, and they're currently the same. Ideally we'd preserve this property.

2. I agree that this would be avoiding the problem, but I'm not sure it's possible to include load without adding a huge amount of noise. It's worth experimenting with at least.
Components: Test>Telemetry
Components: -Tests>Telemetry

Sign in to add a comment