Changeset 61094 in vbox for trunk

Timestamp:

May 20, 2016 11:50:31 AM (9 years ago)

Author:

vboxsync

Message:

FE/Qt: Runtime UI: Qt5 migration consequences: There is an issue with duplicated Windows native events causing GUI misbehaviors here and there (for example Host+Del was not working unless keyboard is captured and so on with other Host+[Shortcut]).

File:

• trunk/src/VBox/Frontends/VirtualBox/src/runtime/UIMachineView.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/runtime/UIMachineView.cpp

@@ -1875 +1875 @@
         case WM_SYSKEYUP:
         {
+            // WORKAROUND:
+            // There is an issue in the Windows Qt5 event processing sequence
+            // causing QAbstractNativeEventFilter to receive Windows native events
+            // coming not just to the top-level window but to actual target as well.
+            // They are calling one - "global event" and another one - "context event".
+            // That way native events are always duplicated with almost no possibility
+            // to distinguish copies except the fact that synthetic event always have
+            // time set to 0 (actually that field was not initialized at all, we had
+            // fixed that in our private Qt tool). We should skip such events instantly.
+            if (pEvent->time == 0)
+                return false;
+
             /* Delegate key-event handling to the keyboard-handler: */
             return machineLogic()->keyboardHandler()->nativeEventPostprocessor(pMessage, screenId());