New issue
Advanced search Search tips

Issue 616482 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug

Blocked on:
issue 622827



Sign in to add a comment

Add sub-metrics for Event.Latency.TouchToFirstScrollUpdateSwapBegin

Project Member Reported by lanwei@chromium.org, Jun 1 2016

Issue description

We defined a set of sub-metrics to measure the latencies inside Event.Latency.TouchToScrollUpdateSwapBegin to help us better understand the time spent on each stage during scroll. Now we will add the same set of sub-metrics to  Event.Latency.TouchToFirstScrollUpdateSwapBegin.

The details are in this doc:
https://docs.google.com/document/d/1EPKT652e2PcO1ysA7LQosGyyr1xmxbtiiiivNU3lnpQ/edit#heading=h.r6af7ldx7kcw.



 

Comment 1 by lanwei@chromium.org, Aug 18 2016

Status: Started (was: Assigned)
Blockedon: 622827
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/eb43b09f06395377e66c684522d76ec002563fb0

commit eb43b09f06395377e66c684522d76ec002563fb0
Author: lanwei <lanwei@chromium.org>
Date: Mon Nov 28 17:49:27 2016

Add sub-metrics for Event.Latency.TouchToFirstScrollUpdateSwapBegin.

We defined a set of sub-metrics to measure the latencies inside
Event.Latency.TouchToScrollUpdateSwapBegin to help us better understand
the time spent on each stage during scroll. Now we will add the same set
of sub-metrics to Event.Latency.TouchToFirstScrollUpdateSwapBegin.

The details are in this doc:
https://docs.google.com/document/d/1EPKT652e2PcO1ysA7LQosGyyr1xmxbtiiiivNU3lnpQ/edit#heading=h.r6af7ldx7kcw.

BUG= 616482 

Review-Url: https://codereview.chromium.org/2268163002
Cr-Commit-Position: refs/heads/master@{#434682}

[modify] https://crrev.com/eb43b09f06395377e66c684522d76ec002563fb0/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc
[modify] https://crrev.com/eb43b09f06395377e66c684522d76ec002563fb0/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
[modify] https://crrev.com/eb43b09f06395377e66c684522d76ec002563fb0/tools/metrics/histograms/histograms.xml
[modify] https://crrev.com/eb43b09f06395377e66c684522d76ec002563fb0/ui/events/latency_info.dot

Comment 4 by lanwei@chromium.org, Nov 28 2016

Status: Fixed (was: Started)

Sign in to add a comment