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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Side-by-side diff isn't available for this file because of its large size.
Issue 2416173003: Rename Event.Latency.* to Event.Latency.ScrollUpdate.* (Closed)
Patch Set: Created 4 years, 2 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/input/render_widget_host_latency_tracker_unittest.cc ('k') | no next file » | 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 9d00901a8d18cb0f51b705b2daf771e47e753501..2f9bb19ae5b8a0db2251694e3f666df37977383b 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -13909,6 +13909,18 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram
+ name="Event.Latency.ScrollUpdate.Touch.TimeToFirstScrollUpdateSwapBegin2"
+ units="microseconds">
+ <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
+ if that ScrollUpdate is the first such event in a given scroll gesture event
+ sequence. If no swap was induced by the event, no recording is made.
+ </summary>
+</histogram>
+
<histogram name="Event.Latency.ScrollUpdate.Touch.TimeToHandled2"
units="microseconds">
<owner>tdresser@chromium.org</owner>
@@ -13920,6 +13932,16 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
+<histogram name="Event.Latency.ScrollUpdate.Touch.TimeToScrollUpdateSwapBegin2"
+ units="microseconds">
+ <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
+ swap was induced by the event, no recording is made.
+ </summary>
+</histogram>
+
<histogram name="Event.Latency.ScrollUpdate.TouchToHandled"
units="microseconds">
<owner>rbyers@chromium.org</owner>
@@ -13973,35 +13995,26 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
</summary>
</histogram>
-<histogram name="Event.Latency.ScrollUpdate.Wheel.TimeToHandled2"
- units="microseconds">
- <owner>tdresser@chromium.org</owner>
- <summary>
- Time between initial creation of a wheel 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.Touch.TimeToFirstScrollUpdateSwapBegin2"
Steven Holte 2016/10/14 18:02:49 Mark obsolete instead of removing?
tdresser 2016/10/14 18:38:19 Yeah, sadly we have to mark these obsolete...
sahel 2016/10/14 20:13:34 Done.
+<histogram
+ name="Event.Latency.ScrollUpdate.Wheel.TimeToFirstScrollUpdateSwapBegin2"
units="microseconds">
<owner>tdresser@chromium.org</owner>
<summary>
- Time between initial creation of a touch event and the start of the frame
+ Time between initial creation of a wheel event and the start of the frame
swap on the GPU service caused by the generated ScrollUpdate gesture event
if that ScrollUpdate is the first such event in a given scroll gesture event
sequence. If no swap was induced by the event, no recording is made.
</summary>
</histogram>
-<histogram name="Event.Latency.Touch.TimeToScrollUpdateSwapBegin2"
+<histogram name="Event.Latency.ScrollUpdate.Wheel.TimeToHandled2"
units="microseconds">
<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
- swap was induced by the event, no recording is made.
+ Time between initial creation of a wheel 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>
@@ -14024,8 +14037,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
units="microseconds">
<owner>rbyers@chromium.org</owner>
<obsolete>
- To be replaced by Event.Latency.Touch.TimeToFirstScrollUpdateSwapBegin2 in
- M56. https://crbug.com/649754
+ To be replaced by
+ Event.Latency.ScrollUpdate.Touch.TimeToFirstScrollUpdateSwapBegin2 in M56.
+ https://crbug.com/649754
</obsolete>
<summary>
Time between initial creation of a touch event and the start of the frame
@@ -14051,7 +14065,8 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
units="microseconds">
<owner>rbyers@chromium.org</owner>
<obsolete>
- To be replaced by Event.Latency.Touch.TimeToScrollUpdateSwapBegin2 in M56.
+ To be replaced by
+ Event.Latency.ScrollUpdate.Touch.TimeToScrollUpdateSwapBegin2 in M56.
https://crbug.com/649754
</obsolete>
<summary>
« no previous file with comments | « content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698