kmaterka created this revision. kmaterka added reviewers: Frameworks, davidedmundson, broulik, nicolasfella. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. kmaterka requested review of this revision.
REVISION SUMMARY It is not possible to handle ownership of QMenu the Qt way, because QMenu takes QWidget as a parent, but KSNI is a QObject. In some situations (Qt QPA plugin) we don't want QMenu to be destroyed. CCBUG: 365105 REPOSITORY R289 KNotifications BRANCH master REVISION DETAIL https://phabricator.kde.org/D24755 AFFECTED FILES src/kstatusnotifieritem.cpp src/kstatusnotifieritem.h src/kstatusnotifieritemprivate_p.h To: kmaterka, #frameworks, davidedmundson, broulik, nicolasfella Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns