[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/#review23645
-----------------------------------------------------------


This review has been submitted with commit fa6d2e86451350730f2bc4c7ba8c79717865b75d by Oliver Henshaw to branch master.

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