New issue
Advanced search Search tips

Issue 862389 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Implement all input delay UMA histogram and UKM

Project Member Reported by charliea@chromium.org, Jul 10

Issue description

Splitting this off from bug 822412 as my starter project on the Chrome Speed Metrics team. dproy@ recently implemented an UMA histogram and UKM for longest input delay. I should do the same for all input delays.
 
Components: -Internals>Metrics>UMA -Internals>Metrics>UKM Speed>Metrics
The Internals components are for metric infrastructure, not metric definition.

We should probably update the component descriptions to better reflect this...
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 17

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

commit eda9e70a1979f4d413469c84984c2fd3c7b3e7d2
Author: Charlie Andrews <charliea@chromium.org>
Date: Tue Jul 17 21:12:31 2018

Implement UMA histograms for all input delays and timestamps


dproy@ added histograms for longest input delays and timestamps in
http://crrev.com/c/1084058: this CL does the same for general input delays.

R=tdresser@chromium.org,rkaplow@chromium.org
CC=​​dproy@chromium.org

Bug: 862389
Change-Id: I4453b0b7338101098ca4521100abe3293add45c0
Reviewed-on: https://chromium-review.googlesource.com/1132108
Reviewed-by: Steve Kobes <skobes@chromium.org>
Reviewed-by: Robert Kaplow (slow) <rkaplow@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#575774}
[modify] https://crrev.com/eda9e70a1979f4d413469c84984c2fd3c7b3e7d2/third_party/blink/renderer/core/exported/web_frame_test.cc
[modify] https://crrev.com/eda9e70a1979f4d413469c84984c2fd3c7b3e7d2/third_party/blink/renderer/core/exported/web_view_test.cc
[modify] https://crrev.com/eda9e70a1979f4d413469c84984c2fd3c7b3e7d2/third_party/blink/renderer/core/loader/document_loader.cc
[modify] https://crrev.com/eda9e70a1979f4d413469c84984c2fd3c7b3e7d2/third_party/blink/renderer/core/loader/interactive_detector.cc
[modify] https://crrev.com/eda9e70a1979f4d413469c84984c2fd3c7b3e7d2/tools/metrics/histograms/histograms.xml

100 day ping! Charlie: did this get finished?
Yikes - the fact that this was 100 days ago scares me. No, it didn't: it got put on the back burner for other things and remains there, sadly. Hoping to revisit after EventTiming work wraps up.

Sign in to add a comment