Cocoa: 'Quit' in menu bar does not work

If you have two window, each with its own menu bar that
has a 'Quit' action, we reuse the quit menu item when
switching between the windows. Now, it we deleteLater one of
the menu bars, the new menubar will update the 'Quit' item
just before deleteLater will come along and remote the update
again. This patch will fix this.

Task-number: QTBUG-4684
Reviewed-by: Prasanth

1 files changed, 5 insertions, 1 deletions

diff --git a/src/gui/widgets/qmenu_mac.mm b/src/gui/widgets/qmenu_mac.mm
index 9510cc6..8b371e5 100644
--- a/src/gui/widgets/qmenu_mac.mm
+++ b/src/gui/widgets/qmenu_mac.mm
@@ -774,8 +774,12 @@ QMacMenuAction::~QMacMenuAction()
                    && menuItem != [getMenuLoader() quitMenuItem]) {
             [menuItem setHidden:YES];
         }
+        // Only clear the tag if it points to this action. Since items like
+        // 'Quit' is being reused, we sometimes end up redirecting the item
+        // before deleting the assosiated action.
+        if (action.data() == reinterpret_cast<QAction *>([menuItem tag]))
+            [menuItem setTag:nil];
     }
-    [menuItem setTag:nil];
     [menuItem release];
 #endif
 }