Make QCursor::pos() actually follow the cursor pos

We need to map based on the screen where the cursor actually is.
Drag and drop works on xcb now. Note that this change means that
QMouseEvent::globalPos() is no longer necessarily the same as
QCursor::pos().

Change-Id: I0a4aa0806a3251881890ac6daa09ac7c6c2fd74c
Reviewed-by: Morten Johan Sørvig <morten.sorvig@theqtcompany.com>

1 files changed, 8 insertions, 3 deletions

diff --git a/src/gui/kernel/qcursor.cpp b/src/gui/kernel/qcursor.cpp
index 236b3b0565..d52ecb7a68 100644
--- a/src/gui/kernel/qcursor.cpp
+++ b/src/gui/kernel/qcursor.cpp
@@ -178,9 +178,14 @@ QT_BEGIN_NAMESPACE
 */
 QPoint QCursor::pos(const QScreen *screen)
 {
-    if (screen)
-        if (const QPlatformCursor *cursor = screen->handle()->cursor())
-            return QHighDpi::fromNativePixels(cursor->pos(), screen);
+    if (screen) {
+        if (const QPlatformCursor *cursor = screen->handle()->cursor()) {
+            QPlatformScreen *ps = screen->handle();
+            QPoint nativePos = cursor->pos();
+            ps = ps->screenForPosition(nativePos);
+            return QHighDpi::fromNativePixels(nativePos, ps->screen());
+        }
+    }
     return QGuiApplicationPrivate::lastCursorPosition.toPoint();
 }