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

Unified Diff: content/public/test/browser_test_utils.cc

Issue 2782893002: WebMouseEvent coordinates are now fractional & private (Closed)
Patch Set: Truncated to int on input, git cl format Created 3 years, 9 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: content/public/test/browser_test_utils.cc
diff --git a/content/public/test/browser_test_utils.cc b/content/public/test/browser_test_utils.cc
index 01620c8a7b8f5ae3e0998534f42919a75cd6a900..d65549fc47d6d7d061b2132fc608291cc9c36267 100644
--- a/content/public/test/browser_test_utils.cc
+++ b/content/public/test/browser_test_utils.cc
@@ -531,12 +531,11 @@ void SimulateMouseClickAt(WebContents* web_contents,
blink::WebInputEvent::MouseDown, modifiers,
ui::EventTimeStampToSeconds(ui::EventTimeForNow()));
mouse_event.button = button;
- mouse_event.x = point.x();
- mouse_event.y = point.y();
- // Mac needs globalX/globalY for events to plugins.
+ mouse_event.setPositionInWidget(point.x(), point.y());
+ // Mac needs positionInScreen for events to plugins.
gfx::Rect offset = web_contents->GetContainerBounds();
- mouse_event.globalX = point.x() + offset.x();
- mouse_event.globalY = point.y() + offset.y();
+ mouse_event.setPositionInScreen(point.x() + offset.x(),
+ point.y() + offset.y());
mouse_event.clickCount = 1;
web_contents->GetRenderViewHost()->GetWidget()->ForwardMouseEvent(
mouse_event);
@@ -551,8 +550,7 @@ void SimulateMouseEvent(WebContents* web_contents,
blink::WebMouseEvent mouse_event(
type, blink::WebInputEvent::NoModifiers,
ui::EventTimeStampToSeconds(ui::EventTimeForNow()));
- mouse_event.x = point.x();
- mouse_event.y = point.y();
+ mouse_event.setPositionInWidget(point.x(), point.y());
web_contents->GetRenderViewHost()->GetWidget()->ForwardMouseEvent(
mouse_event);
}
@@ -564,8 +562,7 @@ void SimulateMouseWheelEvent(WebContents* web_contents,
blink::WebInputEvent::MouseWheel, blink::WebInputEvent::NoModifiers,
ui::EventTimeStampToSeconds(ui::EventTimeForNow()));
- wheel_event.x = point.x();
- wheel_event.y = point.y();
+ wheel_event.setPositionInWidget(point.x(), point.y());
wheel_event.deltaX = delta.x();
wheel_event.deltaY = delta.y();
RenderWidgetHostImpl* widget_host = RenderWidgetHostImpl::From(

Powered by Google App Engine
This is Rietveld 408576698