Changeset 59912 in vbox for trunk

Timestamp:

Mar 3, 2016 6:04:04 PM (9 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:8200: Runtime UI: Stability fix for PowerUp procedure, replacing r105792: Manually re-caching actual machine state when necessary.

File:

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

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

@@ -213 +213 @@
             return false;
 
-    /* Check if we missed a really quick termination after successful powerUp().
-     * We should take into account the fact of async machine-state arrival.
-     * And we don't even want to loop at this point to handle posted
-     * machine-state events, so we are relying onto CMachine getter
-     * to return the fresh state of the thing we want. */
-    const KMachineState ms = machine().GetState();
-    switch (ms)
-    {
-        case KMachineState_PoweredOff:
-        case KMachineState_Saved:
-        case KMachineState_Teleported:
-        case KMachineState_Aborted:
-        {
-            LogRel(("GUI: Aborting startup due to invalid machine state detected: %d\n", ms));
-            return false;
-        }
-        default:
-            break;
+    /* Make sure all the pending Console events converted to signals
+     * during the powerUp() progress above reached their destinations.
+     * That is necessary to make sure all the pending machine state change events processed.
+     * We can't just use the machine state directly acquired from IMachine because there
+     * will be few places which are using stale machine state, not just this one. */
+    QApplication::sendPostedEvents(0, QEvent::MetaCall);
+
+    /* Check if we missed a really quick termination after successful startup: */
+    if (isTurnedOff())
+    {
+        LogRel(("GUI: Aborting startup due to invalid machine state detected: %d\n", machineState()));
+        return false;
     }