Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(736)

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 921473003: Break down end-to-end scroll update latency UMA metrics (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: pretty-printing histograms.xml Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
« no previous file with comments | « content/browser/renderer_host/render_widget_host_latency_tracker.cc ('k') | ui/events/latency_info.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 934a2374887bb31335ed0ba73572fdbac2c9ea88..625bac61b28b2c53bd067472a255767c883b2e23 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -7351,6 +7351,52 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="Event.Latency.ScrollUpdate.BrowserNotifiedToBeforeGpuSwap"
+ units="microseconds">
+ <owner>rbyers@chromium.org</owner>
+ <summary>
+ Time between the browser receives the notification of a ScrollUpdate gesture
+ event induced renderer swap and GPU starts to swap.
+ </summary>
+</histogram>
+
+<histogram name="Event.Latency.ScrollUpdate.GpuSwap" units="microseconds">
+ <owner>rbyers@chromium.org</owner>
+ <summary>
+ Time between gpu starts to swap a ScrollUpdate gesture event induced frame
+ and the swap finishes.
+ </summary>
+</histogram>
+
+<histogram name="Event.Latency.ScrollUpdate.HandledToRendererSwap"
+ units="microseconds">
+ <owner>rbyers@chromium.org</owner>
+ <summary>
+ Time between the ScrollUpdate gesture event is handled on main/impl thread
+ (specified by suffix) and before renderer starts to swap.
+ </summary>
+</histogram>
+
+<histogram name="Event.Latency.ScrollUpdate.RendererSwapToBrowserNotified"
+ units="microseconds">
+ <owner>rbyers@chromium.org</owner>
+ <summary>
+ Time between the renderer starts to swap a frame induced by ScrollUpdate
+ gesture event and browser receives the swap notification.
+ </summary>
+</histogram>
+
+<histogram name="Event.Latency.ScrollUpdate.TouchToHandled"
+ units="microseconds">
+ <owner>rbyers@chromium.org</owner>
+ <summary>
+ Time between initial creation of a touch event and the generated
+ ScrollUpdate gesture event is handled on main/impl thread (specified by
+ suffix). If no swap was induced by the ScrollUpdate gesture event, no
+ recording is made.
+ </summary>
+</histogram>
+
<histogram name="Event.Latency.TouchToFirstScrollUpdateSwap"
units="microseconds">
<owner>rbyers@chromium.org</owner>
@@ -63548,6 +63594,13 @@ To add a new entry, add it with any value and run test to compute valid value.
<affected-histogram name="SB2.MalwareInterstitialTimeTakeMeBack"/>
</histogram_suffixes>
+<histogram_suffixes name="ScrollUpdateHandledThread">
+ <affected-histogram name="Event.Latency.ScrollUpdate.HandledToRendererSwap"/>
+ <suffix name="Main" label="ScrollUpdate handled on main thread"/>
+ <suffix name="Impl" label="ScrollUpdate handled on impl thread"/>
+ <affected-histogram name="Event.Latency.ScrollUpdate.TouchToHandled"/>
+</histogram_suffixes>
+
<histogram_suffixes name="SecurityInterstitialType" separator="."
ordering="prefix">
<suffix name="bad_clock"/>
« no previous file with comments | « content/browser/renderer_host/render_widget_host_latency_tracker.cc ('k') | ui/events/latency_info.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698