D24862: Port one of session management connections state to a custom API

[David Edmundson]

davidedmundson created this revision.
davidedmundson added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
davidedmundson requested review of this revision.

REVISION SUMMARY
  Currently kwin opens a second ICE connection to ksmserver in order to
  tell the state of kwin's whether we're logging out and saving clients or
  not.
  
  This requires that kwin launches after ksmserver to have the connection
  which is a dependency I want to break.
  
  Practically this code is already ksmserver specific as it relies on some
  custom code that sends the first saveState request to kwin first.
  
  Instead we can replace it with a bespoke IPC over DBus and siplify the
  code both end. This will allow several other future enhancements that we
  want with regards to handling the session state, as well as make an
  effort platform agnostic session management, as well as cleaning up some
  complex code.
  
  Ksmserver calls into kwin, rather than having kwin watch ksmserver state
  to allow us make sure it's race free.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D24862

AFFECTED FILES
  CMakeLists.txt
  activation.cpp
  libkwineffects/kwinglobals.h
  main_x11.cpp
  org.kde.KWin.Session.xml
  sm.cpp
  sm.h
  workspace.cpp
  workspace.h

To: davidedmundson, #kwin
Cc: kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20191022/d689a006/attachment.html>