Review Request 125802: Split generic parts from X11Locker into AbstractLocker

[Bhushan Shah]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125802/
-----------------------------------------------------------

(Updated Oct. 26, 2015, 1:36 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma, David Edmundson and Martin Gräßlin.


Changes
-------

Submitted with commit 5a0ab38613763ab50a0e1315615163a48fdb753b by Bhushan Shah to branch master.


Repository: plasma-workspace


Description
-------

Advantage of this split is some parts such as showLockWindow and hideLockWindow needs platform specific code. So instead of doing complex if/else branching we can have seperate X11Locker and WaylandLocker.


Diffs
-----

  ksmserver/screenlocker/CMakeLists.txt 24a89f6 
  ksmserver/screenlocker/abstractlocker.h PRE-CREATION 
  ksmserver/screenlocker/abstractlocker.cpp PRE-CREATION 
  ksmserver/screenlocker/autotests/CMakeLists.txt ab901ca 
  ksmserver/screenlocker/autotests/lockwindowtest.cpp 444c44f 
  ksmserver/screenlocker/autotests/x11lockertest.cpp PRE-CREATION 
  ksmserver/screenlocker/ksldapp.h b281ab6 
  ksmserver/screenlocker/ksldapp.cpp 68ebfc2 
  ksmserver/screenlocker/lockwindow.h c4013be 
  ksmserver/screenlocker/lockwindow.cpp e46733e 

Diff: https://git.reviewboard.kde.org/r/125802/diff/


Testing
-------

builds, installs, autotests and kscreenlocker_test passes, normal lockscreen also works.


Thanks,

Bhushan Shah

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151026/07ed368d/attachment-0001.html>