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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 2268163002: Add sub-metrics for Event.Latency.TouchToFirstScrollUpdateSwapBegin (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add flag in the macro Created 4 years, 3 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
« content/browser/renderer_host/input/render_widget_host_latency_tracker.cc ('K') | « content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc ('k') | ui/events/latency_info.dot » ('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 a4d804ab9ae6221fa573f1e7833404082810d010..4b2ce825f7bfbcdb18a3fea599b6c5230dfabb0c 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -13176,6 +13176,58 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram
+ name="Event.Latency.FirstScrollUpdate.BrowserNotifiedToBeforeGpuSwap"
+ units="microseconds">
+ <owner>tdresser@chromium.org</owner>
+ <summary>
+ Time between the browser receives the notification of a ScrollUpdate gesture
+ event induced renderer swap and GPU starts to swap if that ScrollUpdate is
+ the first such in a given scroll gesture event sequence.
+ </summary>
+</histogram>
+
+<histogram name="Event.Latency.FirstScrollUpdate.GpuSwap" units="microseconds">
+ <owner>tdresser@chromium.org</owner>
+ <summary>
+ Time between gpu starts to swap a ScrollUpdate gesture event induced frame
+ and the swap finishes if that ScrollUpdate is the first such in a given
+ scroll gesture event sequence.
+ </summary>
+</histogram>
+
+<histogram name="Event.Latency.FirstScrollUpdate.HandledToRendererSwap"
+ units="microseconds">
+ <owner>tdresser@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 if that
+ ScrollUpdate is the first such in a given scroll gesture event sequence.
+ </summary>
+</histogram>
+
+<histogram name="Event.Latency.FirstScrollUpdate.RendererSwapToBrowserNotified"
+ units="microseconds">
+ <owner>tdresser@chromium.org</owner>
+ <summary>
+ Time between the renderer starts to swap a frame induced by ScrollUpdate
+ gesture event and browser receives the swap notification if that
+ ScrollUpdate is the first such in a given scroll gesture event sequence.
+ </summary>
+</histogram>
+
+<histogram name="Event.Latency.FirstScrollUpdate.TouchToHandled"
+ units="microseconds">
+ <owner>tdresser@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 that ScrollUpdate is the first such in a given scroll gesture
+ event sequence. If no swap was induced by the ScrollUpdate gesture event, no
+ recording is made.
+ </summary>
+</histogram>
+
<histogram name="Event.Latency.HitTest" units="microseconds">
<owner>dtapuska@chromium.org</owner>
<summary>
@@ -13312,7 +13364,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.Latency.ScrollUpdate.BrowserNotifiedToBeforeGpuSwap"
units="microseconds">
- <owner>rbyers@chromium.org</owner>
+ <owner>tdresser@chromium.org</owner>
<summary>
Time between the browser receives the notification of a ScrollUpdate gesture
event induced renderer swap and GPU starts to swap.
@@ -13320,7 +13372,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</histogram>
<histogram name="Event.Latency.ScrollUpdate.GpuSwap" units="microseconds">
- <owner>rbyers@chromium.org</owner>
+ <owner>tdresser@chromium.org</owner>
<summary>
Time between gpu starts to swap a ScrollUpdate gesture event induced frame
and the swap finishes.
@@ -13329,7 +13381,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.Latency.ScrollUpdate.HandledToRendererSwap"
units="microseconds">
- <owner>rbyers@chromium.org</owner>
+ <owner>tdresser@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.
@@ -13338,7 +13390,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.Latency.ScrollUpdate.RendererSwapToBrowserNotified"
units="microseconds">
- <owner>rbyers@chromium.org</owner>
+ <owner>tdresser@chromium.org</owner>
<summary>
Time between the renderer starts to swap a frame induced by ScrollUpdate
gesture event and browser receives the swap notification.
@@ -13347,7 +13399,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.Latency.ScrollUpdate.TouchToHandled"
units="microseconds">
- <owner>rbyers@chromium.org</owner>
+ <owner>tdresser@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
@@ -13358,7 +13410,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.Latency.TouchToFirstScrollUpdateSwap"
units="microseconds">
- <owner>rbyers@chromium.org</owner>
+ <owner>tdresser@chromium.org</owner>
<obsolete>
Replaced by Event.Latency.TouchToFirstScrollUpdateSwapBegin.
crbug.com/478845
@@ -13373,7 +13425,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.Latency.TouchToFirstScrollUpdateSwapBegin"
units="microseconds">
- <owner>rbyers@chromium.org</owner>
+ <owner>tdresser@chromium.org</owner>
<summary>
Time between initial creation of a touch event and the start of the frame
swap on the GPU service caused by the generated ScrollUpdate gesture event
@@ -13383,7 +13435,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</histogram>
<histogram name="Event.Latency.TouchToScrollUpdateSwap" units="microseconds">
- <owner>rbyers@chromium.org</owner>
+ <owner>tdresser@chromium.org</owner>
<obsolete>
Replaced by Event.Latency.TouchToScrollUpdateSwapBegin. crbug.com/478845
</obsolete>
@@ -13396,7 +13448,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
<histogram name="Event.Latency.TouchToScrollUpdateSwapBegin"
units="microseconds">
- <owner>rbyers@chromium.org</owner>
+ <owner>tdresser@chromium.org</owner>
<summary>
Time between initial creation of a touch event and start of the frame swap
on the GPU service caused by the generated ScrollUpdate gesture event. If no
« content/browser/renderer_host/input/render_widget_host_latency_tracker.cc ('K') | « content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc ('k') | ui/events/latency_info.dot » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698