-----------------------------------------------------------
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

_______________________________________________
Kde-hardware-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-hardware-devel

Reply via email to