Changeset 52989 in vbox for trunk/src/VBox/Frontends/VirtualBox/src/runtime/UIMachine.cpp

Timestamp:

Oct 8, 2014 11:29:07 AM (10 years ago)

Author:

vboxsync

Message:

FE/Qt: Runtime UI rework/cleanup for 7115 (part #4): Better encapsulation for UIMachine (part #1).

File:

• trunk/src/VBox/Frontends/VirtualBox/src/runtime/UIMachine.cpp (modified) (7 diffs)

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

@@ -20 +20 @@
 #else  /* !VBOX_WITH_PRECOMPILED_HEADERS */
 
-/* Local includes: */
+/* GUI includes: */
 # include "VBoxGlobal.h"
 # include "UIExtraDataManager.h"
@@ -142 +142 @@
 }
 
+UIMachineLogic* UIMachine::machineLogic() const
+{
+    if (m_pVisualState && m_pVisualState->machineLogic())
+        return m_pVisualState->machineLogic();
+    return 0;
+}
+
+QWidget* UIMachine::activeWindow() const
+{
+    if (machineLogic() &&  machineLogic()->activeMachineWindow())
+        return machineLogic()->activeMachineWindow();
+    return 0;
+}
+
+void UIMachine::asyncChangeVisualState(UIVisualStateType visualStateType)
+{
+    emit sigRequestAsyncVisualStateChange(visualStateType);
+}
+
+void UIMachine::sltChangeVisualState(UIVisualStateType newVisualStateType)
+{
+    /* Create new state: */
+    UIVisualState *pNewVisualState = new UIVisualState(this, m_pSession, newVisualStateType);
+
+    /* First we have to check if the selected mode is available at all.
+     * Only then we delete the old mode and switch to the new mode. */
+    if (pNewVisualState->prepareChange())
+    {
+        /* Delete previous state: */
+        delete m_pVisualState;
+
+        /* Set the new mode as current mode: */
+        m_pVisualState = pNewVisualState;
+        m_pVisualState->change();
+    }
+    else
+    {
+        /* Discard the temporary created new state: */
+        delete pNewVisualState;
+
+        /* If there is no state currently created => we have to exit: */
+        if (!m_pVisualState)
+            deleteLater();
+    }
+}
+
 UIMachine::UIMachine()
     : QObject(0)
-    , initialStateType(UIVisualStateType_Normal)
     , m_pSession(0)
+    , m_allowedVisualStates(UIVisualStateType_Invalid)
+    , m_initialStateType(UIVisualStateType_Normal)
     , m_pVisualState(0)
-    , m_allowedVisualStates(UIVisualStateType_Invalid)
 {
     /* Make sure VBoxGlobal is aware of VM creation: */
     vboxGlobal().setVirtualMachine(this);
+}
+
+UIMachine::~UIMachine()
+{
+    /* Make sure VBoxGlobal is aware of VM destruction: */
+    vboxGlobal().setVirtualMachine(0);
+
+    /* Cleanup machine UI: */
+    cleanup();
 }
 
@@ -192 +247 @@
     vboxGlobal().startMediumEnumeration(false /* force start */);
 
-    /* Load machine settings: */
-    loadMachineSettings();
+    /* Load settings: */
+    loadSettings();
 
     /* Prepare async visual-state change handler: */
@@ -217 +272 @@
 }
 
-UIMachine::~UIMachine()
-{
-    /* Save machine settings: */
-    saveMachineSettings();
-
-    /* Delete visual state: */
-    delete m_pVisualState;
-    m_pVisualState = 0;
-
-    /* Delete UI session: */
-    delete m_pSession;
-    m_pSession = 0;
-
-    /* Free session finally: */
-    m_session.UnlockMachine();
-    m_session.detach();
-
-    /* Make sure VBoxGlobal is aware of VM destruction: */
-    vboxGlobal().setVirtualMachine(0);
-
-    /* Quit application: */
-    QApplication::quit();
-}
-
-QWidget* UIMachine::activeWindow() const
-{
-    if (machineLogic() &&  machineLogic()->activeMachineWindow())
-        return machineLogic()->activeMachineWindow();
-    return 0;
-}
-
-void UIMachine::asyncChangeVisualState(UIVisualStateType visualStateType)
-{
-    emit sigRequestAsyncVisualStateChange(visualStateType);
-}
-
-void UIMachine::sltChangeVisualState(UIVisualStateType newVisualStateType)
-{
-    /* Create new state: */
-    UIVisualState *pNewVisualState = new UIVisualState(this, m_pSession, newVisualStateType);
-
-    /* First we have to check if the selected mode is available at all.
-     * Only then we delete the old mode and switch to the new mode. */
-    if (pNewVisualState->prepareChange())
-    {
-        /* Delete previous state: */
-        delete m_pVisualState;
-
-        /* Set the new mode as current mode: */
-        m_pVisualState = pNewVisualState;
-        m_pVisualState->change();
-    }
-    else
-    {
-        /* Discard the temporary created new state: */
-        delete pNewVisualState;
-
-        /* If there is no state currently created => we have to exit: */
-        if (!m_pVisualState)
-            deleteLater();
-    }
-}
-
-void UIMachine::enterInitialVisualState()
-{
-    sltChangeVisualState(initialStateType);
-}
-
-UIMachineLogic* UIMachine::machineLogic() const
-{
-    if (m_pVisualState && m_pVisualState->machineLogic())
-        return m_pVisualState->machineLogic();
-    return 0;
-}
-
-void UIMachine::loadMachineSettings()
+void UIMachine::loadSettings()
 {
     /* Load 'visual state' option: */
@@ -309 +289 @@
             {
                 /* Direct transition to scale/fullscreen mode allowed: */
-                case UIVisualStateType_Scale: initialStateType = UIVisualStateType_Scale; break;
-                case UIVisualStateType_Fullscreen: initialStateType = UIVisualStateType_Fullscreen; break;
+                case UIVisualStateType_Scale: m_initialStateType = UIVisualStateType_Scale; break;
+                case UIVisualStateType_Fullscreen: m_initialStateType = UIVisualStateType_Fullscreen; break;
                 /* While to seamless is not, so we have to request transition on GA capability-change event: */
                 case UIVisualStateType_Seamless: uisession()->setRequestedVisualState(UIVisualStateType_Seamless); break;
@@ -319 +299 @@
 }
 
-void UIMachine::saveMachineSettings()
+void UIMachine::saveSettings()
 {
     /* Save 'visual state' option: */
@@ -338 +318 @@
 }
 
+void UIMachine::cleanup()
+{
+    /* Save settings: */
+    saveSettings();
+
+    /* Delete visual state: */
+    delete m_pVisualState;
+    m_pVisualState = 0;
+
+    /* Delete UI session: */
+    delete m_pSession;
+    m_pSession = 0;
+
+    /* Free session finally: */
+    m_session.UnlockMachine();
+    m_session.detach();
+
+    /* Make sure VBoxGlobal is aware of VM destruction: */
+    vboxGlobal().setVirtualMachine(0);
+
+    /* Quit application: */
+    QApplication::quit();
+}
+
+void UIMachine::enterInitialVisualState()
+{
+    sltChangeVisualState(m_initialStateType);
+}
+
 #include "UIMachine.moc"