D11586: libdbusmenu-qt: Remove nonexistant actions directly from the menu

2018-03-23 Thread Fabian Vogt
This revision was automatically updated to reflect the committed changes. Closed by commit R120:8827bb38806e: libdbusmenu-qt: Remove nonexistant actions directly from the menu (authored by fvogt). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE

D11586: libdbusmenu-qt: Remove nonexistant actions directly from the menu

2018-03-22 Thread Fabian Vogt
fvogt added a comment. Looking at the `WAYLAND_DEBUG` output I think I'm closer to understanding the issue now. When the `QMenu::popup` method is called, the chain "opened Event -> LayoutChanged -> GetLayout" gets started. The menu calculates its size and position, and sometime later

D11586: libdbusmenu-qt: Remove nonexistant actions directly from the menu

2018-03-22 Thread Fabian Vogt
fvogt created this revision. fvogt added a reviewer: Plasma. Restricted Application added a project: Plasma. fvogt requested review of this revision. REVISION SUMMARY The getLayout response handler compares the new list of actions with the current actions in the menu and calls deleteLater on