Bug 39465 – [GTK] webkit_web_view_drag_end should call EventHandler::dragSourceEndedAt

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Martin Robinson]

This is required so that the DOM can process dragEnd events.

[Martin Robinson]

Created attachment 56656 [details]
Patch for this issue

I've attached a patch for this issue.

[Xan Lopez]

Comment on attachment 56656 [details]
Patch for this issue

>+        [GTK] webkit_web_view_drag_end should call EventHandler::dragSourceEndedAt
>+        https://bugs.webkit.org/show_bug.cgi?id=39465

Why should it call it? What happens if you don't? Just because of the names it makes sense, but we should explain it in the ChangeLog.

>+    event->button.window = static_cast<GdkWindow*>(g_object_ref(
>+        gdk_display_get_window_at_pointer(display, &x, &y)));

I think the style is to let lines be as long as needed.

>+    event->button.x = x;
>+    event->button.y = y;
>+    event->button.x_root = xRoot;
>+    event->button.y_root = yRoot;
>+    event->button.state = modifiers;
>+
>+    PlatformMouseEvent platformEvent(&event->button);
>+    frame->eventHandler()->dragSourceEndedAt(platformEvent,
>+        gdkDragActionToDragOperation(context->action));

Same.

r- for missing rationale and because I'm feeling evil.

[Xan Lopez]

Comment on attachment 56656 [details]
Patch for this issue

Bah, sorry, the explanation actually was in the ChangeLog, just inlined in the function mention. My apologies. r=me

[Martin Robinson]

Created attachment 56784 [details]
Patch

[Martin Robinson]

(In reply to comment #3)
> (From update of attachment 56656 [details])
> Bah, sorry, the explanation actually was in the ChangeLog, just inlined in the function mention. My apologies. r=me

I think you're right in that it could be expanded a bit. I've done that above
and removed the newlines as you've asked. I'll commit the patch that I just
posted. Thanks for the review. :)

[Martin Robinson]

Committed r60010: <http://trac.webkit.org/changeset/60010>

[Martin Robinson]

Comment on attachment 56784 [details]
Patch

Clearing flags, as this has landed.