davidedmundson created this revision. davidedmundson added a reviewer: KWin. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. davidedmundson requested review of this revision.
REVISION SUMMARY Kwin had to have two ICE connections in order to track state indepdendently of it's session saving. This replaces that with a more direct DBus protocol allowing for both simplification on the kwin side as well as comunicating the logout state better for effects. Whilst this code temporarily complicates things, now we have this interface the next step is drop all the isWM() stuff and do kwin specific session management also over this interface. See T11882 <https://phabricator.kde.org/T11882> TEST PLAN Added qdebug into kwin started logging out with an unsaved file, cancelled shutdown started logging out with, discarded file REPOSITORY R120 Plasma Workspace BRANCH origin-master (branched from master) REVISION DETAIL https://phabricator.kde.org/D24945 AFFECTED FILES ksmserver/CMakeLists.txt ksmserver/logout.cpp ksmserver/server.cpp ksmserver/server.h To: davidedmundson, #kwin Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
