Add sub-metrics for Event.Latency.TouchToFirstScrollUpdateSwapBegin

ui/events/latency_info.dot

Index: ui/events/latency_info.dot
diff --git a/ui/events/latency_info.dot b/ui/events/latency_info.dot
index 626384cbd012147295d747f997245734e63404b4..e234cd6ad6bf7d13a4ced82654505e8dd9acd76e 100644
--- a/ui/events/latency_info.dot
+++ b/ui/events/latency_info.dot
@@ -13,24 +13,26 @@ digraph g {
   INPUT_EVENT_LATENCY_ACK_RWH_COMPONENT -> INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_MAIN_COMPONENT;
   INPUT_EVENT_LATENCY_ACK_RWH_COMPONENT -> INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_IMPL_COMPONENT;
 
-  INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_MAIN_COMPONENT -> INPUT_EVENT_LATENCY_RENDERER_SWAP_COMPONENT [label="Event.Latency.ScrollUpdate.HandledToRendererSwap_Main"];
-  INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_IMPL_COMPONENT -> INPUT_EVENT_LATENCY_RENDERER_SWAP_COMPONENT [label="Event.Latency.ScrollUpdate.HandledToRendererSwap_Impl"];
+  INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_MAIN_COMPONENT -> INPUT_EVENT_LATENCY_RENDERER_SWAP_COMPONENT [label="Event.Latency.ScrollBegin/ScrollUpdate.*.HandledToRendererSwap2_Main"];
+  INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_IMPL_COMPONENT -> INPUT_EVENT_LATENCY_RENDERER_SWAP_COMPONENT [label="Event.Latency.ScrollBegin/ScrollUpdate.*.HandledToRendererSwap2_Impl"];
 
-  INPUT_EVENT_LATENCY_RENDERER_SWAP_COMPONENT -> INPUT_EVENT_BROWSER_RECEIVED_RENDERER_SWAP_COMPONENT [label="Event.Latency.ScrollUpdate.RendererSwapToBrowserNotified"];
-  INPUT_EVENT_BROWSER_RECEIVED_RENDERER_SWAP_COMPONENT -> INPUT_EVENT_GPU_SWAP_BUFFER_COMPONENT [label="Event.Latency.ScrollUpdate.BrowserNotifiedToBeforeGpuSwap"];
-  INPUT_EVENT_GPU_SWAP_BUFFER_COMPONENT -> INPUT_EVENT_LATENCY_TERMINATED_FRAME_SWAP_COMPONENT [label="Event.Latency.ScrollUpdate.GPUSwap"];
+  INPUT_EVENT_LATENCY_RENDERER_SWAP_COMPONENT -> INPUT_EVENT_BROWSER_RECEIVED_RENDERER_SWAP_COMPONENT [label="Event.Latency.ScrollBegin/ScrollUpdate.*.RendererSwapToBrowserNotified2"];
+  INPUT_EVENT_BROWSER_RECEIVED_RENDERER_SWAP_COMPONENT -> INPUT_EVENT_GPU_SWAP_BUFFER_COMPONENT [label="Event.Latency.ScrollBegin/ScrollUpdate.*.BrowserNotifiedToBeforeGpuSwap2"];
+  INPUT_EVENT_GPU_SWAP_BUFFER_COMPONENT -> INPUT_EVENT_LATENCY_TERMINATED_FRAME_SWAP_COMPONENT [label="Event.Latency.ScrollBegin/ScrollUpdate.*.GpuSwap2"];
 
   INPUT_EVENT_LATENCY_UI_COMPONENT ->
   INPUT_EVENT_LATENCY_FIRST_SCROLL_UPDATE_ORIGINAL_COMPONENT ->
   INPUT_EVENT_LATENCY_BEGIN_RWH_COMPONENT;
 
   edge[style="dashed"];
-  INPUT_EVENT_LATENCY_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_GPU_SWAP_BUFFER_COMPONENT [label="Event.Latency.TouchToScrollUpdateSwapBegin"];
-  INPUT_EVENT_LATENCY_FIRST_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_GPU_SWAP_BUFFER_COMPONENT [label="Event.Latency.TouchToFirstScrollUpdateSwapBegin"];
+  INPUT_EVENT_LATENCY_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_GPU_SWAP_BUFFER_COMPONENT [label="Event.Latency.ScrollUpdate.*.TimeToScrollUpdateSwapBegin2"];
+  INPUT_EVENT_LATENCY_FIRST_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_GPU_SWAP_BUFFER_COMPONENT [label="Event.Latency.ScrollBegin.*.TimeToScrollUpdateSwapBegin2"];
 
   INPUT_EVENT_LATENCY_UI_COMPONENT -> INPUT_EVENT_LATENCY_BEGIN_RWH_COMPONENT [label="Event.Latency.Browser.*UI"];
-  INPUT_EVENT_LATENCY_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_MAIN_COMPONENT [label="Event.Latency.ScrollUpdate.TouchToHandled_Main"];
-  INPUT_EVENT_LATENCY_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_IMPL_COMPONENT [label="Event.Latency.ScrollUpdate.TouchToHandled_Impl"];
+  INPUT_EVENT_LATENCY_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_MAIN_COMPONENT [label="Event.Latency.ScrollUpdate.*.TimeToHandled2_Main"];
+  INPUT_EVENT_LATENCY_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_IMPL_COMPONENT [label="Event.Latency.ScrollUpdate.*.TimeToHandled2_Impl"];

[tdresser, 2016/11/24 19:16:16]

This is still inconsistent, right? We're using "ScrollBegin/ScrollUpdate" for
some histograms, but not all?

[lanwei, 2016/11/24 19:29:11]

Yes, because the histograms of TimeToScrollUpdateSwapBegin2 and TimeToHandled2
are measured from 
 INPUT_EVENT_LATENCY_FIRST_SCROLL_UPDATE_ORIGINAL_COMPONENT or
INPUT_EVENT_LATENCY_SCROLL_UPDATE_ORIGINAL_COMPONENT, so we will use have
separate ones for both ScrollBegin and ScrollUpdate. But the other histograms
will presents them in just one.

https://drive.google.com/a/google.com/file/d/0B6aINcG6kDLXcmR1NTBnZFVnM2M/vie...

[tdresser, 2016/11/28 15:14:55]

Acknowledged.

 
+  INPUT_EVENT_LATENCY_FIRST_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_MAIN_COMPONENT [label="Event.Latency.ScrollBegin.*.TimeToHandled2_Main"];
+  INPUT_EVENT_LATENCY_FIRST_SCROLL_UPDATE_ORIGINAL_COMPONENT -> INPUT_EVENT_LATENCY_RENDERING_SCHEDULED_IMPL_COMPONENT [label="Event.Latency.ScrollBegin.*.TimeToHandled2_Impl"];
 
 }