Changeset 85633 in vbox for trunk

Timestamp:

Aug 6, 2020 10:35:19 AM (4 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:9609: A bit of cleanup for manager action-pool stuff: UIActionSimpleManagerCommonPerformDiscard doesn't need to inherit UIActionPolymorphicMenu, since state management present in base class.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/globals/UIActionPoolManager.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/globals/UIActionPoolManager.cpp

@@ -1113 +1113 @@
 };
 
-/** Polymorphic menu action extension, used as 'Perform Discard' action class. */
-class UIActionSimpleManagerCommonPerformDiscard : public UIActionPolymorphicMenu
+/** Simple menu action extension, used as 'Perform Discard' action class. */
+class UIActionSimpleManagerCommonPerformDiscard : public UIActionSimple
 {
     Q_OBJECT;
@@ -1122 +1122 @@
     /** Constructs action passing @a pParent to the base-class. */
     UIActionSimpleManagerCommonPerformDiscard(UIActionPool *pParent)
-        : UIActionPolymorphicMenu(pParent,
-                                  ":/vm_discard_32px.png", ":/vm_discard_16px.png",
-                                  ":/vm_discard_disabled_32px.png", ":/vm_discard_disabled_16px.png")
-    {
-        /* Hide menu always: */
-        hideMenu();
-    }
+        : UIActionSimple(pParent,
+                         ":/vm_discard_32px.png", ":/vm_discard_16px.png",
+                         ":/vm_discard_disabled_32px.png", ":/vm_discard_disabled_16px.png")
+    {}
 
 protected: