Review Request: Screen Locker daemon

[Aaron J. Seigo]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103105/#review8095
-----------------------------------------------------------


it looks like we're pretty much right back to the situation we had prior to the screenlocking being moved to kwin, except that now we have yet another daemon which also links to kdeui just so it can be a unique app.

if we are going to go this route, i highly recommend that the daemon becomes a kded plugin.

other than that -> how does kwin handle a locked desktop with this new system? e.g. turning effects and other window painting off ...

- Aaron J. Seigo


On Nov. 10, 2011, 12:16 p.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103105/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2011, 12:16 p.m.)
> 
> 
> Review request for kwin, Plasma and Oswald Buddenhagen.
> 
> 
> Description
> -------
> 
> Yes I know it's late in the cycle :-) and yes not everything is implemented yet, but I am confident that I get these things finished today or at least till Beta tagging.
> 
> This is the new screenlocker work as discussed on kcd some time ago.
> 
> The screen locker is split into two parts:
> 1. A daemon (ksld) to just lock the screen and grab input
> 2. An unlock dialog (kscreenunlocker) which is executed as a separate process.
>     
> In case the unlocker fails/crashes the screen is still locked by
> the lock daemon.
>     
> In case kscreenunlocker crashes or does not succeed, it gets
> automatically restarted by the daemon.
> 
> Things I still need to do:
> * D-Bus integration
> * Grace time
> * integration of existing screen savers into the QML
> * cleanup KRunner
> * several more things
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 9fa4c10 
>   screenlocker/CMakeLists.txt PRE-CREATION 
>   screenlocker/kcfg/kscreensaversettings.kcfg PRE-CREATION 
>   screenlocker/kcfg/kscreensaversettings.kcfgc PRE-CREATION 
>   screenlocker/ksld.desktop PRE-CREATION 
>   screenlocker/ksldapp.h PRE-CREATION 
>   screenlocker/ksldapp.cpp PRE-CREATION 
>   screenlocker/lockwindow.h PRE-CREATION 
>   screenlocker/lockwindow.cpp PRE-CREATION 
>   screenlocker/main.cpp PRE-CREATION 
>   screenlocker/unlocker/CMakeLists.txt PRE-CREATION 
>   screenlocker/unlocker/main.cpp PRE-CREATION 
>   screenlocker/unlocker/qml/lockscreen.qml PRE-CREATION 
>   screenlocker/unlocker/unlockapp.h PRE-CREATION 
>   screenlocker/unlocker/unlockapp.cpp PRE-CREATION 
>   screenlocker/unlocker/unlocker.h PRE-CREATION 
>   screenlocker/unlocker/unlocker.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/103105/diff/diff
> 
> 
> Testing
> -------
> 
> * Screen locks
> * Screen stays locked if unlocker crasher
> * unlocker gets restarted
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111110/2bc23910/attachment.html>