winex11: Do window search if IDropTarget::DragOver() returns DROPEFFECT_NONE.

Fixes a regression from 1d337249. Prior to the commit, we were solving this case in a different way. If IDropTarget::DragOver() returned DROPEFFECT_NONE, X11DRV_XDND_DROPEFFECTToXdndAction() would convert it to XdndActionCopy. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55179

winex11: Do window search if IDropTarget::DragOver() returns DROPEFFECT_NONE.
c03c1727 · Akihiro Sagawa · Alexandre Julliard · 6ea417d3 · c03c1727
Commit c03c1727 authored Dec 11, 2023 by Akihiro Sagawa Committed by Alexandre Julliard Dec 11, 2023
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

xdnd.c dlls/winex11.drv/xdnd.c +3 -3

No files found.
--- a/dlls/winex11.drv/xdnd.c
+++ b/dlls/winex11.drv/xdnd.c
@@ -233,7 +233,7 @@ NTSTATUS WINAPI x11drv_dnd_position_event( void *arg, ULONG size )
        }
    }

-    if (XDNDAccepted)
+    if (XDNDAccepted && XDNDDropEffect != DROPEFFECT_NONE)
        accept = 1;
    else
    {
@@ -246,7 +246,7 @@ NTSTATUS WINAPI x11drv_dnd_position_event( void *arg, ULONG size )
        }
    }

-    return accept ? effect : 0;
+    return accept ? effect : DROPEFFECT_NONE;
 }

 NTSTATUS x11drv_dnd_drop_event( UINT arg )
@@ -319,7 +319,7 @@ NTSTATUS x11drv_dnd_drop_event( UINT arg )
    TRACE("effectRequested(0x%lx) accept(%d) performed(0x%lx) at x(%ld),y(%ld)\n",
          XDNDDropEffect, accept, effect, XDNDxy.x, XDNDxy.y);

-    return accept ? effect : 0;
+    return accept ? effect : DROPEFFECT_NONE;
 }

 /**************************************************************************