[Kde-hardware-devel] Review Request: Make ChangeScreenSettings imply InterruptSession

[Oliver Henshaw]

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

Review request for Solid.


Description
-------

Make ChangeScreenSettings imply InterruptSession

As e.g. suspending the computer will interrupt video playing. There do
not seem to be any use cases that require keeping the screen active
right up to the moment that the system idles out and suspends itself.
Indeed, PolicyAgent users that ask for ChangeScreenSettings inhibit do
not expect the screen contents to disappear, no matter the cause.


Diffs
-----

  powerdevil/daemon/powerdevilpolicyagent.cpp d688512f0eba1c9d201c32bf5312edbe87af3c78 

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


Testing
-------

Verified org.kde.Solid.PowerManagement.PolicyAgent, org.freedesktop.ScreenSaver and org.freedesktop.PowerManagement.Inhibit dbus inhibitions work as expected.


Thanks,

Oliver Henshaw

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