Title: [213835] releases/WebKitGTK/webkit-2.16/Source/WebKit2
Revision
213835
Author
[email protected]
Date
2017-03-13 05:09:11 -0700 (Mon, 13 Mar 2017)

Log Message

Merge r213638 - Correctly process the return value of gdk_drag_context_get_selected_action()

Reviewed by Carlos Garcia Campos.

It returns the action itself and not the bitmask.

* UIProcess/gtk/DragAndDropHandler.cpp:
(WebKit::DragAndDropHandler::drop):

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.16/Source/WebKit2/ChangeLog (213834 => 213835)


--- releases/WebKitGTK/webkit-2.16/Source/WebKit2/ChangeLog	2017-03-13 12:06:06 UTC (rev 213834)
+++ releases/WebKitGTK/webkit-2.16/Source/WebKit2/ChangeLog	2017-03-13 12:09:11 UTC (rev 213835)
@@ -1,3 +1,14 @@
+2017-03-09  Tomas Popela  <[email protected]>
+
+        Correctly process the return value of gdk_drag_context_get_selected_action()
+
+        Reviewed by Carlos Garcia Campos.
+
+        It returns the action itself and not the bitmask.
+
+        * UIProcess/gtk/DragAndDropHandler.cpp:
+        (WebKit::DragAndDropHandler::drop):
+
 2017-03-04  Joseph Pecoraro  <[email protected]>
 
         Remove duplicate initialization guard in WebKit2 logging initialization

Modified: releases/WebKitGTK/webkit-2.16/Source/WebKit2/UIProcess/gtk/DragAndDropHandler.cpp (213834 => 213835)


--- releases/WebKitGTK/webkit-2.16/Source/WebKit2/UIProcess/gtk/DragAndDropHandler.cpp	2017-03-13 12:06:06 UTC (rev 213834)
+++ releases/WebKitGTK/webkit-2.16/Source/WebKit2/UIProcess/gtk/DragAndDropHandler.cpp	2017-03-13 12:09:11 UTC (rev 213835)
@@ -290,7 +290,7 @@
     droppingContext->dropHappened = true;
 
     uint32_t flags = 0;
-    if ((gdk_drag_context_get_selected_action(context) & GDK_ACTION_COPY))
+    if (gdk_drag_context_get_selected_action(context) == GDK_ACTION_COPY)
         flags |= WebCore::DragApplicationIsCopyKeyDown;
     DragData dragData(droppingContext->selectionData.ptr(), position, convertWidgetPointToScreenPoint(m_page.viewWidget(), position), gdkDragActionToDragOperation(gdk_drag_context_get_actions(context)), static_cast<WebCore::DragApplicationFlags>(flags));
     SandboxExtension::Handle handle;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to