Changeset 19614 in vbox for trunk/src/VBox/Main

Timestamp:

May 12, 2009 12:27:21 PM (16 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

47192

Message:

Main and Devices: absolute mouse support: do not send a mouse movement event if only the button state has changed

Location:

trunk/src/VBox/Main

Files:

• MouseImpl.cpp (modified) (2 diffs)
• include/MouseImpl.h (modified) (1 diff)

trunk/src/VBox/Main/MouseImpl.cpp

@@ -59 +59 @@
 {
     mpDrv = NULL;
+    mLastAbsX = 0;
+    mLastAbsY = 0;
     return S_OK;
 }
@@ -315 +317 @@
             fButtons |= PDMIMOUSEPORT_BUTTON_MIDDLE;
 
-        vrc = mpDrv->pUpPort->pfnPutEvent(mpDrv->pUpPort, 1, 1, dz,
-                                          fButtons);
+        /* This is a workaround.  In order to alert the Guest Additions to the
+         * fact that the absolute pointer position has changed, we send a
+         * a minute movement event to the PS/2 mouse device.  But in order
+         * to avoid the mouse jiggling every time the use clicks, we check to
+         * see if the position has really changed since the last mouse event.
+         */
+        if ((mLastAbsX == mouseXAbs) && (mLastAbsY == mouseYAbs))
+            vrc = mpDrv->pUpPort->pfnPutEvent(mpDrv->pUpPort, 0, 0, dz,
+                                              fButtons);
+        else
+            vrc = mpDrv->pUpPort->pfnPutEvent(mpDrv->pUpPort, 1, 1, dz,
+                                              fButtons);
+        mLastAbsX = mouseXAbs;
+        mLastAbsY = mouseYAbs;
         if (RT_FAILURE (vrc))
             rc = setError (VBOX_E_IPRT_ERROR,

trunk/src/VBox/Main/include/MouseImpl.h

@@ -106 +106 @@
 
     LONG uHostCaps;
+    uint32_t mLastAbsX;
+    uint32_t mLastAbsY;
 };