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

Side by Side Diff: content/browser/renderer_host/render_widget_host_input_event_router.cc

Issue 1993383002: Adding Event.Latency.Browser.WheelUI, Passing latency_info to ProcessMouse/MouseWheelEvent functions (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/render_widget_host_input_event_router.h" 5 #include "content/browser/renderer_host/render_widget_host_input_event_router.h"
6 6
7 #include "base/debug/dump_without_crashing.h" 7 #include "base/debug/dump_without_crashing.h"
8 #include "cc/quads/surface_draw_quad.h" 8 #include "cc/quads/surface_draw_quad.h"
9 #include "cc/surfaces/surface_id_allocator.h" 9 #include "cc/surfaces/surface_id_allocator.h"
10 #include "cc/surfaces/surface_manager.h" 10 #include "cc/surfaces/surface_manager.h"
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
135 RenderWidgetHostViewBase* root_view, 135 RenderWidgetHostViewBase* root_view,
136 blink::WebMouseEvent* event) { 136 blink::WebMouseEvent* event) {
137 gfx::Point transformed_point; 137 gfx::Point transformed_point;
138 RenderWidgetHostViewBase* target = FindEventTarget( 138 RenderWidgetHostViewBase* target = FindEventTarget(
139 root_view, gfx::Point(event->x, event->y), &transformed_point); 139 root_view, gfx::Point(event->x, event->y), &transformed_point);
140 if (!target) 140 if (!target)
141 return; 141 return;
142 142
143 event->x = transformed_point.x(); 143 event->x = transformed_point.x();
144 event->y = transformed_point.y(); 144 event->y = transformed_point.y();
145 145 // TODO(crbug.com/613628): Initialize latency info correctly for OOPIFs.
nasko 2016/05/27 22:35:04 nit: Please use TODO(username): Text ... . See htt
sahel 2016/06/06 19:05:34 Done.
146 target->ProcessMouseEvent(*event); 146 ui::LatencyInfo latency_info;
147 target->ProcessMouseEvent(*event, latency_info);
147 } 148 }
148 149
149 void RenderWidgetHostInputEventRouter::RouteMouseWheelEvent( 150 void RenderWidgetHostInputEventRouter::RouteMouseWheelEvent(
150 RenderWidgetHostViewBase* root_view, 151 RenderWidgetHostViewBase* root_view,
151 blink::WebMouseWheelEvent* event) { 152 blink::WebMouseWheelEvent* event) {
152 gfx::Point transformed_point; 153 gfx::Point transformed_point;
153 RenderWidgetHostViewBase* target = FindEventTarget( 154 RenderWidgetHostViewBase* target = FindEventTarget(
154 root_view, gfx::Point(event->x, event->y), &transformed_point); 155 root_view, gfx::Point(event->x, event->y), &transformed_point);
155 if (!target) 156 if (!target)
156 return; 157 return;
157 158
158 event->x = transformed_point.x(); 159 event->x = transformed_point.x();
159 event->y = transformed_point.y(); 160 event->y = transformed_point.y();
160 161 // TODO(crbug.com/613628): Initialize latency info correctly for OOPIFs.
161 target->ProcessMouseWheelEvent(*event); 162 ui::LatencyInfo latency_info;
163 target->ProcessMouseWheelEvent(*event, latency_info);
162 } 164 }
163 165
164 void RenderWidgetHostInputEventRouter::RouteGestureEvent( 166 void RenderWidgetHostInputEventRouter::RouteGestureEvent(
165 RenderWidgetHostViewBase* root_view, 167 RenderWidgetHostViewBase* root_view,
166 blink::WebGestureEvent* event, 168 blink::WebGestureEvent* event,
167 const ui::LatencyInfo& latency) { 169 const ui::LatencyInfo& latency) {
168 // We use GestureTapDown to detect the start of a gesture sequence since there 170 // We use GestureTapDown to detect the start of a gesture sequence since there
169 // is no WebGestureEvent equivalent for ET_GESTURE_BEGIN. Note that this 171 // is no WebGestureEvent equivalent for ET_GESTURE_BEGIN. Note that this
170 // means the GestureFlingCancel that always comes between ET_GESTURE_BEGIN and 172 // means the GestureFlingCancel that always comes between ET_GESTURE_BEGIN and
171 // GestureTapDown is sent to the previous target, in case it is still in a 173 // GestureTapDown is sent to the previous target, in case it is still in a
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
285 if (owner_map_.find(cc::SurfaceIdAllocator::NamespaceForId( 287 if (owner_map_.find(cc::SurfaceIdAllocator::NamespaceForId(
286 params.surface_id)) == owner_map_.end()) { 288 params.surface_id)) == owner_map_.end()) {
287 return; 289 return;
288 } 290 }
289 HittestData data; 291 HittestData data;
290 data.ignored_for_hittest = params.ignored_for_hittest; 292 data.ignored_for_hittest = params.ignored_for_hittest;
291 hittest_data_[params.surface_id] = data; 293 hittest_data_[params.surface_id] = data;
292 } 294 }
293 295
294 } // namespace content 296 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698