1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

QCocoaMenu: Explicitly remove NSMenu from supermenu on destruction

Browse Source 
The d'tor of QCocoaMenu calls dismiss() to remove the current menu.
dismiss() calls cancelTracking, which doesn't always remove the
menu from its parent.

Remove the menu from its parent in addition.

Fixes: QTBUG-135634
Pick-to: 6.9 6.8 6.5
Change-Id: Ic8e9b4172c3b2035b7e9ad1b576725923987abf2
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
(cherry picked from commit 36ebec92e81fb8ee7d965ad1a84221d6c779f562)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
This commit is contained in:
Axel Spoerl 2025-06-03 15:39:03 +02:00 committed by Qt Cherry-pick Bot
parent 3afa63c30c
commit 641422686d
1 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/plugins/platforms/cocoa/qcocoamenu.mm
View File

@ -44,6 +44,16 @@ QCocoaMenu::~QCocoaMenu()
if (isOpen())
dismiss();
if (NSMenu *superMenu = m_nativeMenu.supermenu) {
for (NSMenuItem *item in superMenu.itemArray) {
if (item.submenu == m_nativeMenu) {
[superMenu removeItem:item];
break;
}
}
}
[m_nativeMenu release];
}