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