| Index: content/browser/renderer_host/render_widget_host_impl.cc
|
| diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc
|
| index 7cd8d8dd8b15f052b8997656a346a248f1a51e5d..9ff2eaca4f8e28135ea8009bc24353dc14f434c9 100644
|
| --- a/content/browser/renderer_host/render_widget_host_impl.cc
|
| +++ b/content/browser/renderer_host/render_widget_host_impl.cc
|
| @@ -1026,7 +1026,7 @@ void RenderWidgetHostImpl::SimulateTouchGestureWithMouse(
|
|
|
| void RenderWidgetHostImpl::ForwardMouseEvent(const WebMouseEvent& mouse_event) {
|
| ForwardMouseEventWithLatencyInfo(
|
| - MouseEventWithLatencyInfo(mouse_event, NewInputLatencyInfo()));
|
| + MouseEventWithLatencyInfo(mouse_event, NewInputLatencyInfo(NULL)));
|
| }
|
|
|
| void RenderWidgetHostImpl::ForwardMouseEventWithLatencyInfo(
|
| @@ -1060,7 +1060,7 @@ void RenderWidgetHostImpl::OnPointerEventActivate() {
|
|
|
| void RenderWidgetHostImpl::ForwardWheelEvent(
|
| const WebMouseWheelEvent& wheel_event) {
|
| - ForwardWheelEventWithLatencyInfo(wheel_event, NewInputLatencyInfo());
|
| + ForwardWheelEventWithLatencyInfo(wheel_event, NewInputLatencyInfo(NULL));
|
| }
|
|
|
| void RenderWidgetHostImpl::ForwardWheelEventWithLatencyInfo(
|
| @@ -1134,19 +1134,17 @@ void RenderWidgetHostImpl::ForwardGestureEventWithLatencyInfo(
|
| if (ignore_input_events_ || process_->IgnoreInputEvents())
|
| return;
|
|
|
| - ui::LatencyInfo latency_info;
|
| - // In Aura, gesture event will carry its original touch event's
|
| - // INPUT_EVENT_LATENCY_RWH_COMPONENT. For non-aura platform, we add the
|
| - // INPUT_EVENT_LATENCY_RWH_COMPONENT right here.
|
| - if (!ui_latency.FindLatency(ui::INPUT_EVENT_LATENCY_RWH_COMPONENT,
|
| - GetLatencyComponentId(),
|
| - NULL))
|
| - latency_info = NewInputLatencyInfo();
|
| + ui::LatencyInfo latency_info = NewInputLatencyInfo(&ui_latency);
|
|
|
| - latency_info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_INJECTED_RWH_COMPONENT,
|
| - GetLatencyComponentId(),
|
| - ++last_input_number_);
|
| - latency_info.MergeWith(ui_latency);
|
| + WebKit::WebInputEvent::Type type = gesture_event.type;
|
| + if (type == WebKit::WebInputEvent::GestureScrollBegin ||
|
| + type == WebKit::WebInputEvent::GestureScrollUpdate ||
|
| + type == WebKit::WebInputEvent::GestureScrollUpdateWithoutPropagation) {
|
| + latency_info.AddLatencyNumber(
|
| + ui::INPUT_EVENT_LATENCY_GESTURE_SCROLL_RWH_COMPONENT,
|
| + GetLatencyComponentId(),
|
| + ++last_input_number_);
|
| + }
|
|
|
| if (!IsInOverscrollGesture() &&
|
| !gesture_event_filter_->ShouldForward(
|
| @@ -1291,7 +1289,7 @@ void RenderWidgetHostImpl::ForwardKeyboardEvent(
|
|
|
| // Only forward the non-native portions of our event.
|
| ForwardInputEvent(key_event, sizeof(WebKeyboardEvent),
|
| - NewInputLatencyInfo(),
|
| + NewInputLatencyInfo(NULL),
|
| is_keyboard_shortcut);
|
| }
|
| }
|
| @@ -1309,11 +1307,20 @@ void RenderWidgetHostImpl::DisableResizeAckCheckForTesting() {
|
| g_check_for_pending_resize_ack = false;
|
| }
|
|
|
| -ui::LatencyInfo RenderWidgetHostImpl::NewInputLatencyInfo() {
|
| +ui::LatencyInfo RenderWidgetHostImpl::NewInputLatencyInfo(
|
| + const ui::LatencyInfo* original) {
|
| ui::LatencyInfo info;
|
| - info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_RWH_COMPONENT,
|
| + if (original)
|
| + info = *original;
|
| + // In Aura, gesture event will already carry its original touch event's
|
| + // INPUT_EVENT_LATENCY_RWH_COMPONENT.
|
| + if (!info.FindLatency(ui::INPUT_EVENT_LATENCY_RWH_COMPONENT,
|
| GetLatencyComponentId(),
|
| - ++last_input_number_);
|
| + NULL)) {
|
| + info.AddLatencyNumber(ui::INPUT_EVENT_LATENCY_RWH_COMPONENT,
|
| + GetLatencyComponentId(),
|
| + ++last_input_number_);
|
| + }
|
| return info;
|
| }
|
|
|
| @@ -1418,8 +1425,7 @@ void RenderWidgetHostImpl::ForwardInputEvent(
|
| void RenderWidgetHostImpl::ForwardTouchEventWithLatencyInfo(
|
| const WebKit::WebTouchEvent& touch_event,
|
| const ui::LatencyInfo& ui_latency) {
|
| - ui::LatencyInfo latency_info = NewInputLatencyInfo();
|
| - latency_info.MergeWith(ui_latency);
|
| + ui::LatencyInfo latency_info = NewInputLatencyInfo(&ui_latency);
|
| TouchEventWithLatencyInfo touch_with_latency(touch_event, latency_info);
|
| touch_event_queue_->QueueEvent(touch_with_latency);
|
| }
|
| @@ -2654,16 +2660,41 @@ void RenderWidgetHostImpl::ComputeTouchLatency(
|
| }
|
|
|
| void RenderWidgetHostImpl::FrameSwapped(const ui::LatencyInfo& latency_info) {
|
| - ui::LatencyInfo::LatencyMap::const_iterator l =
|
| - latency_info.latency_components.find(std::make_pair(
|
| - ui::INPUT_EVENT_LATENCY_RWH_COMPONENT, GetLatencyComponentId()));
|
| - if (l == latency_info.latency_components.end())
|
| + ui::LatencyInfo::LatencyComponent rwh_component;
|
| + if (!latency_info.FindLatency(ui::INPUT_EVENT_LATENCY_RWH_COMPONENT,
|
| + GetLatencyComponentId(),
|
| + &rwh_component))
|
| return;
|
|
|
| - rendering_stats_.input_event_count += l->second.event_count;
|
| + rendering_stats_.input_event_count += rwh_component.event_count;
|
| rendering_stats_.total_input_latency +=
|
| - l->second.event_count *
|
| - (latency_info.swap_timestamp - l->second.event_time);
|
| + rwh_component.event_count *
|
| + (latency_info.swap_timestamp - rwh_component.event_time);
|
| +
|
| + ui::LatencyInfo::LatencyComponent original_component;
|
| + if (latency_info.FindLatency(ui::INPUT_EVENT_LATENCY_ORIGINAL_COMPONENT,
|
| + 0,
|
| + &original_component) &&
|
| + latency_info.FindLatency(
|
| + ui::INPUT_EVENT_LATENCY_GESTURE_SCROLL_RWH_COMPONENT,
|
| + GetLatencyComponentId(),
|
| + NULL)) {
|
| + // This UMA metric tracks the time from when the original touch event is
|
| + // created (averaged if there are multiple) to when the scroll gesture
|
| + // results in final frame swap.
|
| + base::TimeDelta delta =
|
| + latency_info.swap_timestamp - original_component.event_time;
|
| + UMA_HISTOGRAM_CUSTOM_COUNTS(
|
| + "Event.Latency.Browser.TouchToScroll",
|
| + delta.InMicroseconds(),
|
| + 0,
|
| + 1000000,
|
| + 100);
|
| + rendering_stats_.scroll_event_count += original_component.event_count;
|
| + rendering_stats_.total_scroll_latency +=
|
| + original_component.event_count *
|
| + (latency_info.swap_timestamp - original_component.event_time);
|
| + }
|
|
|
| if (CommandLine::ForCurrentProcess()->HasSwitch(
|
| switches::kEnableGpuBenchmarking))
|
|
|