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

Unified Diff: content/common/browser_rendering_stats.cc

Issue 17757002: Add UMA/Telemetry stats for touch event latency (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: address various issues 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: content/common/browser_rendering_stats.cc
diff --git a/content/common/browser_rendering_stats.cc b/content/common/browser_rendering_stats.cc
index 87e38575e8a7b8ce6f21477f961b3749c8ba3e30..8cea4378f5c01f7c647d878bcc11c94cac9c736f 100644
--- a/content/common/browser_rendering_stats.cc
+++ b/content/common/browser_rendering_stats.cc
@@ -6,7 +6,13 @@
namespace content {
-BrowserRenderingStats::BrowserRenderingStats() : input_event_count(0) {}
+BrowserRenderingStats::BrowserRenderingStats() :
+ input_event_count(0),
+ touch_ui_count(0),
+ touch_acked_consumed_count(0),
+ touch_acked_not_consumed_count(0),
+ touch_acked_no_consumer_count(0) {
+}
BrowserRenderingStats::~BrowserRenderingStats() {}
@@ -14,6 +20,21 @@ void BrowserRenderingStats::EnumerateFields(
cc::RenderingStats::Enumerator* enumerator) const {
enumerator->AddInt("inputEventCount", input_event_count);
enumerator->AddTimeDeltaInSecondsF("totalInputLatency", total_input_latency);
+
+ enumerator->AddInt("touchUICount", touch_ui_count);
+ enumerator->AddTimeDeltaInSecondsF("totalTouchUILatency",
+ total_touch_ui_latency);
+ enumerator->AddInt("touchAckedConsumedCount", touch_acked_consumed_count);
+ enumerator->AddTimeDeltaInSecondsF("totalTouchAckedConsumedLatency",
+ total_touch_acked_consumed_latency);
+ enumerator->AddInt("touchAckedNotConsumedCount",
+ touch_acked_not_consumed_count);
+ enumerator->AddTimeDeltaInSecondsF("totalTouchAckedNotConsumedLatency",
+ total_touch_acked_not_consumed_latency);
+ enumerator->AddInt("touchAckedNoConsumerCount",
+ touch_acked_no_consumer_count);
+ enumerator->AddTimeDeltaInSecondsF("totalTouchAckedNoConsumerLatency",
+ total_touch_acked_no_consumer_latency);
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698