[Kde-hardware-devel] Review Request: Pass inhibition on to powerdevil PolicyAgent

[Commit Hook]

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


This review has been submitted with commit f4b89c2abaead52d770c65108f30684f1df529c1 by Oliver Henshaw to branch KDE/4.10.

- Commit Hook


On Dec. 13, 2012, 2:05 p.m., Oliver Henshaw wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107701/
> -----------------------------------------------------------
> 
> (Updated Dec. 13, 2012, 2:05 p.m.)
> 
> 
> Review request for Plasma and Solid.
> 
> 
> Description
> -------
> 
> Pass inhibition on to powerdevil PolicyAgent
> 
> So that ScreenSaver inhibit causes ChangeScreenSettings (and thus
> InterruptSession) inhibit.
> 
> 
> Diffs
> -----
> 
>   ksmserver/screenlocker/CMakeLists.txt 0ea6efaa41d5e5a50932af20e7b5f9ef891a41fb 
>   ksmserver/screenlocker/interface.h 689cd4f0edb5275ebce1484a299a8e50e1b28171 
>   ksmserver/screenlocker/interface.cpp c7678f261055ea3ef5844e77e3502ee360451216 
> 
> Diff: http://git.reviewboard.kde.org/r/107701/diff/
> 
> 
> Testing
> -------
> 
> Tested dbus inhibition via python script and Solid::PowerManagement API use via battery applet. Tested that nothing breaks when powerdevil kded module is not started or when the kded module is stopped (the PolicyAgent dbus service remains available in the latter case).
> 
> 
> Thanks,
> 
> Oliver Henshaw
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20121217/a7501995/attachment.html>