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

Unified Diff: tools/perf/measurements/smoothness.py

Issue 17757002: Add UMA/Telemetry stats for touch event latency (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 6 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:
View side-by-side diff with in-line comments
Download patch
Index: tools/perf/measurements/smoothness.py
diff --git a/tools/perf/measurements/smoothness.py b/tools/perf/measurements/smoothness.py
index 7ca8d69a96fe7459b2156d3cf25c4480f4479f7a..293544284889ba4969edc2c63375e330bfafd637 100644
--- a/tools/perf/measurements/smoothness.py
+++ b/tools/perf/measurements/smoothness.py
@@ -154,6 +154,52 @@ def CalcLatencyResults(rendering_stats_deltas, results):
results.Add('average_latency', 'ms', averageLatency,
data_type='unimportant')
+ touchSystemCount = rendering_stats_deltas.get(
+ 'touchSystemCount', 0)
+ totalTouchSystemLatencyInSeconds = rendering_stats_deltas.get(
+ 'totalTouchSystemLatency', 0)
+ averageLatency = DivideIfPossibleOrZero(
+ (totalTouchSystemLatencyInSeconds * 1000), touchSystemCount)
+ results.Add('average_touch_system_latency', 'ms', averageLatency,
+ data_type='unimportant')
+
+ touchUICount = rendering_stats_deltas.get(
+ 'touchUICount', 0)
+ totalTouchUILatencyInSeconds = rendering_stats_deltas.get(
+ 'totalTouchUILatency', 0)
+ averageLatency = DivideIfPossibleOrZero(
+ (totalTouchUILatencyInSeconds * 1000), touchUICount)
+ results.Add('average_touch_ui_latency', 'ms', averageLatency,
+ data_type='unimportant')
+
+ touchAckedConsumedCount = rendering_stats_deltas.get(
+ 'touchAckedConsumedCount', 0)
+ totalTouchAckedConsumedLatencyInSeconds = rendering_stats_deltas.get(
+ 'totalTouchAckedConsumedLatency', 0)
+ averageLatency = DivideIfPossibleOrZero(
+ (totalTouchAckedConsumedLatencyInSeconds * 1000), touchAckedConsumedCount)
+ results.Add('average_touch_acked_consumed_latency', 'ms', averageLatency,
+ data_type='unimportant')
+
+ touchAckedNotConsumedCount = rendering_stats_deltas.get(
+ 'touchAckedNotConsumedCount', 0)
+ totalTouchAckedNotConsumedLatencyInSeconds = rendering_stats_deltas.get(
+ 'totalTouchAckedNotConsumedLatency', 0)
+ averageLatency = DivideIfPossibleOrZero(
+ (totalTouchAckedNotConsumedLatencyInSeconds * 1000),
+ touchAckedNotConsumedCount)
+ results.Add('average_touch_acked_not_consumed_latency', 'ms', averageLatency,
+ data_type='unimportant')
+
+ touchAckedNoConsumerCount = rendering_stats_deltas.get(
+ 'touchAckedNoConsumerCount', 0)
+ totalTouchAckedNoConsumerLatencyInSeconds = rendering_stats_deltas.get(
+ 'totalTouchAckedNoConsumerLatency', 0)
+ averageLatency = DivideIfPossibleOrZero(
+ (totalTouchAckedNoConsumerLatencyInSeconds * 1000),
+ touchAckedNoConsumerCount)
+ results.Add('average_touch_acked_no_consumer_latency', 'ms', averageLatency,
Rick Byers 2013/06/26 16:34:55 this is all pretty repetative, perhaps avoid dupli
Yufeng Shen (Slow to review) 2013/06/26 20:44:05 Done.
+ data_type='unimportant')
class Smoothness(page_measurement.PageMeasurement):
def __init__(self):
« content/browser/renderer_host/render_widget_host_impl.cc ('K') | « content/common/view_messages.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698