New issue
Advanced search Search tips

Issue 649754 link

Starred by 1 user

Issue metadata

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

Blocking:
issue 679493



Sign in to add a comment

Deprecate scroll update latency histograms

Project Member Reported by sahel@chromium.org, Sep 23 2016

Issue description

Old Event.Latancy.. scrolUpdate metrics report the latency only for touch scrolls. They use the average event time of start and end components. 

Deprecate them with new latency metrics that report latency for both wheel and touch scrolls. The new metrics are calculated based on the difference between the first and last event time of the start and end components,
respectively. 
 
Owner: sahel@chromium.org
Status: Assigned (was: Untriaged)
Probably makes sense for you to own cleaning this up in M56.
Labels: Hotlist-Input-Dev

Comment 3 by sahel@chromium.org, Jan 9 2017

Blocking: 679493
Components: Blink>Scroll

Comment 5 by sahel@chromium.org, Feb 14 2017

Cc: -tdres...@chromium.org sahel@chromium.org
Owner: tdres...@chromium.org
As discussed, @tdresser will do the clean up.
Cc: tdres...@chromium.org
Owner: nzolghadr@chromium.org
Navid is driving this now.
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 19 2017

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

commit 2bdaf0af5e6ab06707931b0c1f8a79635219c024
Author: Navid Zolghadr <nzolghadr@chromium.org>
Date: Thu Oct 19 21:18:33 2017

Remove old input metrics that were replaced

Remove old touch scrolling metrics that were
replaced back in M56.

Bug:  649754 
Change-Id: If47392ac6902bd9cf4e8a001752943e0f615ba70
Reviewed-on: https://chromium-review.googlesource.com/728714
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Commit-Queue: Navid Zolghadr <nzolghadr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#510218}
[modify] https://crrev.com/2bdaf0af5e6ab06707931b0c1f8a79635219c024/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
[modify] https://crrev.com/2bdaf0af5e6ab06707931b0c1f8a79635219c024/tools/metrics/histograms/histograms.xml
[modify] https://crrev.com/2bdaf0af5e6ab06707931b0c1f8a79635219c024/ui/latency/latency_tracker.cc

Status: Fixed (was: Assigned)

Sign in to add a comment