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

Unified Diff: ui/events/latency_info.h

Issue 577273003: Add new latency_info for tracking browser composite time. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rm useless comment Created 6 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:
View side-by-side diff with in-line comments
Download patch
Index: ui/events/latency_info.h
diff --git a/ui/events/latency_info.h b/ui/events/latency_info.h
index 92d928d615837d1323f07112b9244c135af4e67d..7915c96c24a056a4b621f6fd77b44e4be30f5e35 100644
--- a/ui/events/latency_info.h
+++ b/ui/events/latency_info.h
@@ -54,6 +54,13 @@ enum LatencyComponentType {
WINDOW_OLD_SNAPSHOT_FRAME_NUMBER_COMPONENT,
// Timestamp when a tab is requested to be shown.
TAB_SHOW_COMPONENT,
+ // Timestamp of when the browser began compositing
+ INPUT_EVENT_BROWSER_COMPOSITE_COMPONENT,
jbauman 2014/09/19 20:40:29 These don't really have anything to do with input
orglofch 2014/09/19 22:40:27 Technically the events are still tied to input in
+ // Timestamp of when an unparented output surface began swap buffers
brianderson 2014/09/18 23:26:15 "an uparented output surface" -> "the Browser proc
orglofch 2014/09/19 22:40:27 Done.
+ INPUT_EVENT_OUTPUT_SURFACE_SWAP_BUFFER_COMPONENT,
brianderson 2014/09/18 23:26:15 INPUT_EVENT_BROWSER_SWAP_BUFFER_COMPONENT?
orglofch 2014/09/19 22:40:27 Done.
+ // Timestamp of when the gpu service began swap buffers, unlike
+ // INPUT_EVENT_LATENCY_TERMINATED_FRAME_SWAP_COMPONENT which measure after
+ INPUT_EVENT_GPU_SWAP_BUFFER_COMPONENT,
// ---------------------------TERMINAL COMPONENT-----------------------------
// TERMINAL COMPONENT is when we show the latency end in chrome://tracing.
// Timestamp when the mouse event is acked from renderer and it does not

Powered by Google App Engine
This is Rietveld 408576698