Review Request 127699: Set the xscreensaver plugin as valid only if the dbus interface org.freedesktop.ScreenSaver is present

[Paulo Lieuthier]

> On April 25, 2016, 8:55 a.m., Martin Gräßlin wrote:
> > The change looks wrong to me. The functionality of that poller implementation is not bound to the availability of the dbus interface, but on the XScreenSaver X11 extension. The dbus interface is only used to emit a resumeFromIdle when the screen saver deactivated.

> The dbus interface is only used to emit a resumeFromIdle when the screen saver deactivated.

And is it okay if that doesn't work as expected?


- Paulo


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127699/#review94832
-----------------------------------------------------------


On April 20, 2016, 1:54 p.m., Paulo Lieuthier wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127699/
> -----------------------------------------------------------
> 
> (Updated April 20, 2016, 1:54 p.m.)
> 
> 
> Review request for KDE Frameworks and Martin Gräßlin.
> 
> 
> Repository: kidletime
> 
> 
> Description
> -------
> 
> If the dbus interface org.freedesktop.ScreenSaver is not present, it makes no sense to use the plugin.
> 
> 
> Diffs
> -----
> 
>   src/plugins/xscreensaver/xscreensaverbasedpoller.h 05b9a23 
>   src/plugins/xscreensaver/xscreensaverbasedpoller.cpp f56396c 
>   src/widgetbasedpoller.h 68fc7dc 
> 
> Diff: https://git.reviewboard.kde.org/r/127699/diff/
> 
> 
> Testing
> -------
> 
> Tested minimal application using KIdleTime using the xscreensaver plugin.
> 
> 
> Thanks,
> 
> Paulo Lieuthier
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160425/1b95a3db/attachment.html>