Review Request: Move the "Lock Session" shortcut to kwin from krunner

[Alex Merry]

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

Review request for kwin, Plasma, Aaron J. Seigo, Martin Gräßlin, and Farhad Hedayati Fard.


Summary
-------

In the farhad_hf/lockscreen branch (that moves screen locking functionality from krunner to kwin), the shortcut for locking the screen (Ctrl+Alt+L by default) remains in KRunner, which will not necessarily be running (when plasma is in "netbook" mode, for example).  This moves the shortcut registration to kwin (including a necessary hack for unregistering krunner's claim on it).

(See also https://git.reviewboard.kde.org/r/101943/ ).

I can also merge latest master into the branch and/or add in a KWIN_BUILD_SCREENSAVER cmake option (as in the original review request) if you (Martin/Farhad) want.


Diffs
-----

  krunner/krunnerapp.cpp 55ac5ca 
  kwin/screenlocker/screensaver/saverengine.h 3384d4a 
  kwin/screenlocker/screensaver/saverengine.cpp 6c15be6 
  kwin/useractions.cpp 44685b9 

Diff: http://git.reviewboard.kde.org/r/102261/diff


Testing
-------


Thanks,

Alex

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20110809/5ba70266/attachment.html>