Fix movementX and movementY properties of mouse events

Calculate movementX/movementY at every turn, not only when
the mouse is in locked state.

Task-number: QTBUG-59127
Change-Id: I362e055e6b5d6f831671604a093434373cedf19a
Reviewed-by: Peter Varga <pvarga@inf.u-szeged.hu>
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>

1 files changed, 8 insertions, 6 deletions

diff --git a/src/core/render_widget_host_view_qt.cpp b/src/core/render_widget_host_view_qt.cpp
index f1c69fca3..f96329139 100644
--- a/src/core/render_widget_host_view_qt.cpp
+++ b/src/core/render_widget_host_view_qt.cpp
@@ -422,7 +422,7 @@ void RenderWidgetHostViewQt::SetBackgroundColor(SkColor color)
 bool RenderWidgetHostViewQt::LockMouse()
 {
     mouse_locked_ = true;
-    m_lockedMousePosition = QCursor::pos();
+    m_previousMousePosition = QCursor::pos();
     m_delegate->lockMouse();
     qApp->setOverrideCursor(Qt::BlankCursor);
     return true;
@@ -938,11 +938,13 @@ void RenderWidgetHostViewQt::handleMouseEvent(QMouseEvent* event)
         m_clickHelper.lastPressPosition = QPointF(event->pos()).toPoint();
     }
 
-    if (IsMouseLocked()) {
-        webEvent.movementX = -(m_lockedMousePosition.x() - event->globalX());
-        webEvent.movementY = -(m_lockedMousePosition.y() - event->globalY());
-        QCursor::setPos(m_lockedMousePosition);
-    }
+    webEvent.movementX = event->globalX() - m_previousMousePosition.x();
+    webEvent.movementY = event->globalY() - m_previousMousePosition.y();
+
+    if (IsMouseLocked())
+        QCursor::setPos(m_previousMousePosition);
+    else
+        m_previousMousePosition = event->globalPos();
 
     if (m_imeInProgress && event->type() == QMouseEvent::MouseButtonPress) {
         m_imeInProgress = false;