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

Unified Diff: content/shell/test_runner/event_sender.cc

Issue 2782893002: WebMouseEvent coordinates are now fractional & private (Closed)
Patch Set: Fixed compile failures. 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/shell/test_runner/event_sender.cc
diff --git a/content/shell/test_runner/event_sender.cc b/content/shell/test_runner/event_sender.cc
index 28520a0f92664701bdac28da800c7e57824937a0..d358d52c931206d3bac29b04aa600e9951a0cfd7 100644
--- a/content/shell/test_runner/event_sender.cc
+++ b/content/shell/test_runner/event_sender.cc
@@ -217,10 +217,10 @@ void InitMouseEventGeneric(WebMouseEvent::Button b,
int tiltY,
WebMouseEvent* e) {
e->button = b;
- e->x = pos.x;
- e->y = pos.y;
- e->globalX = pos.x;
- e->globalY = pos.y;
+ e->position.x = pos.x;
+ e->position.y = pos.y;
+ e->screenPosition.x = pos.x;
+ e->screenPosition.y = pos.y;
e->pointerType = pointerType;
e->id = pointerId;
e->force = pressure;
@@ -242,10 +242,10 @@ void InitMouseEvent(WebMouseEvent::Button b,
void InitGestureEventFromMouseWheel(const WebMouseWheelEvent& wheel_event,
WebGestureEvent* gesture_event) {
gesture_event->sourceDevice = blink::WebGestureDeviceTouchpad;
- gesture_event->x = wheel_event.x;
- gesture_event->y = wheel_event.y;
- gesture_event->globalX = wheel_event.globalX;
- gesture_event->globalY = wheel_event.globalY;
+ gesture_event->x = wheel_event.position.x;
+ gesture_event->y = wheel_event.position.y;
+ gesture_event->globalX = wheel_event.screenPosition.x;
+ gesture_event->globalY = wheel_event.screenPosition.y;
}
int GetKeyModifier(const std::string& modifier_name) {
@@ -1350,8 +1350,8 @@ void EventSender::DoDragDrop(const WebDragData& drag_data,
widget_event.get() ? static_cast<WebMouseEvent*>(widget_event.get())
: &raw_event;
- WebPoint client_point(event->x, event->y);
- WebPoint screen_point(event->globalX, event->globalY);
+ WebPoint client_point(event->position.x, event->position.y);
+ WebPoint screen_point(event->screenPosition.x, event->screenPosition.y);
current_drag_data_ = drag_data;
current_drag_effects_allowed_ = mask;
current_drag_effect_ = mainFrameWidget()->dragTargetDragEnter(
@@ -2659,8 +2659,8 @@ void EventSender::FinishDragAndDrop(const WebMouseEvent& raw_event,
widget_event.get() ? static_cast<WebMouseEvent*>(widget_event.get())
: &raw_event;
- WebPoint client_point(event->x, event->y);
- WebPoint screen_point(event->globalX, event->globalY);
+ WebPoint client_point(event->position.x, event->position.y);
+ WebPoint screen_point(event->screenPosition.x, event->screenPosition.y);
current_drag_effect_ = drag_effect;
if (current_drag_effect_) {
// Specifically pass any keyboard modifiers to the drop method. This allows
@@ -2691,8 +2691,8 @@ void EventSender::DoDragAfterMouseUp(const WebMouseEvent& raw_event) {
if (current_drag_data_.isNull())
return;
- WebPoint client_point(event->x, event->y);
- WebPoint screen_point(event->globalX, event->globalY);
+ WebPoint client_point(event->position.x, event->position.y);
+ WebPoint screen_point(event->screenPosition.x, event->screenPosition.y);
blink::WebDragOperation drag_effect = mainFrameWidget()->dragTargetDragOver(
client_point, screen_point, current_drag_effects_allowed_,
event->modifiers());
@@ -2717,8 +2717,8 @@ void EventSender::DoDragAfterMouseMove(const WebMouseEvent& raw_event) {
widget_event.get() ? static_cast<WebMouseEvent*>(widget_event.get())
: &raw_event;
- WebPoint client_point(event->x, event->y);
- WebPoint screen_point(event->globalX, event->globalY);
+ WebPoint client_point(event->position.x, event->position.y);
+ WebPoint screen_point(event->screenPosition.x, event->screenPosition.y);
current_drag_effect_ = mainFrameWidget()->dragTargetDragOver(
client_point, screen_point, current_drag_effects_allowed_,
event->modifiers());
@@ -2741,7 +2741,7 @@ void EventSender::ReplaySavedEvents() {
current_pointer_state_[kRawMousePointerId].current_buttons_, e.pos,
click_count_, &event);
current_pointer_state_[kRawMousePointerId].last_pos_ =
- WebPoint(event.x, event.y);
+ WebPoint(event.position.x, event.position.y);
HandleInputEventOnViewOrPopup(event);
DoDragAfterMouseMove(event);
break;

Powered by Google App Engine
This is Rietveld 408576698