D16277: [ksmserver] Split xsession logout and shutdown into separate classes

[David Edmundson]

davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidedmundson requested review of this revision.

REVISION SUMMARY
  This commit splits ksmserver's xsession shutdown logic from performing
  the actual shutdown and running shutdown scripts and implement proposed
  org.kde.Shutdown interface.
  
  Intended longer term target is to move this to a separate executable.
  
  KSMServer's existing logout dbus method still exists for compatibility
  forwarding to the new interface.
  
  There are 2 minor behavioural changes.
  
  The shutdownMode property (which doesn't seem to do anything and is not
  exposed in our UI) is not kept.
  
  If you shutdown /whilst/ starting up somehow, previously we delayed
  showing the logout prompt, we now delay performing the actual logout.

TEST PLAN
  Logged out / shut down using the old API
  Logged out / shut down using the new DBus API

REPOSITORY
  R120 Plasma Workspace

BRANCH
  ksmserver_split

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

AFFECTED FILES
  ksmserver/CMakeLists.txt
  ksmserver/logout.cpp
  ksmserver/main.cpp
  ksmserver/org.kde.Shutdown.xml
  ksmserver/server.cpp
  ksmserver/server.h
  ksmserver/shutdown.cpp
  ksmserver/shutdown.h

To: davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181017/6612e97e/attachment.html>