Review Request: Fix battery applet screen power management

[Oliver Henshaw]

> On Oct. 10, 2012, 3:43 p.m., Lukáš Tinkl wrote:
> > Good catch, is this code only in master or also in 4.9 branch?

Yep, it's needed in 4.9 branch and master.


- Oliver


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


On Oct. 10, 2012, 3:13 p.m., Oliver Henshaw wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106784/
> -----------------------------------------------------------
> 
> (Updated Oct. 10, 2012, 3:13 p.m.)
> 
> 
> Review request for Plasma and Solid.
> 
> 
> Description
> -------
> 
> Fix battery applet screen power management
> 
> Connect to the correct job when beginning/stopping
> SuppressingScreenPowerManagement. The pasto in the "stop" case was
> probably harmless; in the "begin" case we need to store the correct
> cookie in order to later stop the suppression.
> 
> 
> Diffs
> -----
> 
>   plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml 72bb8ec8f9339f2d639267bbc435c1f0f76ecac7 
> 
> Diff: http://git.reviewboard.kde.org/r/106784/diff/
> 
> 
> Testing
> -------
> 
> Tested timed screen & session power management before, during and after inhibit with battery applet.
> 
> 
> Thanks,
> 
> Oliver Henshaw
> 
>

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