Chromium Code Reviews| Index: content/renderer/input/render_widget_input_handler.cc |
| diff --git a/content/renderer/input/render_widget_input_handler.cc b/content/renderer/input/render_widget_input_handler.cc |
| index 75d8f46974b897e701baf2971508eefdd2fead98..a2d6e657f103c21b092dab4d0e04f1602ff677cc 100644 |
| --- a/content/renderer/input/render_widget_input_handler.cc |
| +++ b/content/renderer/input/render_widget_input_handler.cc |
| @@ -329,9 +329,27 @@ void RenderWidgetInputHandler::HandleInputEvent( |
| if (input_event.type == WebInputEvent::TouchStart || |
| input_event.type == WebInputEvent::TouchMove || |
| input_event.type == WebInputEvent::TouchEnd) { |
| - LogPassiveEventListenersUma( |
| - processed, static_cast<const WebTouchEvent&>(input_event).dispatchType, |
| - input_event.timeStampSeconds, latency_info); |
| + WebTouchEvent touch = static_cast<const WebTouchEvent&>(input_event); |
|
tdresser
2016/05/13 13:32:59
Make this a const ref.
lanwei
2016/05/17 22:51:37
Done.
|
| + |
| + LogPassiveEventListenersUma(processed, touch.dispatchType, |
| + input_event.timeStampSeconds, latency_info); |
| + |
| + if (input_event.type == WebInputEvent::TouchStart && |
| + touch.dispatchType == WebInputEvent::Blocking && |
| + base::TimeTicks::IsHighResolution()) { |
| + base::TimeTicks now = base::TimeTicks::Now(); |
| + if (touch.dispatchedDuringFling) { |
| + UMA_HISTOGRAM_CUSTOM_COUNTS( |
| + "Event.Touch.TouchStartDuringFlingLatency", |
| + GetEventLatencyMicros(input_event.timeStampSeconds, now), 1, |
| + 10000000, 100); |
|
Ilya Sherman
2016/05/17 01:27:29
Would 50 buckets suffice here?
lanwei
2016/05/17 22:51:37
Yes.
|
| + } else { |
| + UMA_HISTOGRAM_CUSTOM_COUNTS( |
| + "Event.Touch.TouchStartOutsideFlingLatency", |
| + GetEventLatencyMicros(input_event.timeStampSeconds, now), 1, |
| + 10000000, 100); |
|
Ilya Sherman
2016/05/17 01:27:29
Ditto
|
| + } |
| + } |
| } else if (input_event.type == WebInputEvent::MouseWheel) { |
| LogPassiveEventListenersUma( |
| processed, |