Review Request 125802: Split generic parts from X11Locker into AbstractLocker

Bhushan Shah bhush94 at gmail.com
Mon Oct 26 08:11:37 UTC 2015


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

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


Review request for Plasma and Martin Gräßlin.


Changes
-------

Fix issues, as well as revert move of lockwindow.{h,cpp} -> x11locker.{h,cpp} for ease in reviewing.


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 (updated)
-----

  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/d8d863c7/attachment.html>


More information about the Plasma-devel mailing list