D24945: [ksmserver] Signal session management state to kwin directly
David Edmundson
noreply at phabricator.kde.org
Fri Oct 25 15:29:41 BST 2019
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191025/8c1ab058/attachment.html>
More information about the Plasma-devel
mailing list