[Differential] [Commented On] D2033: Clear idle timeouts when session becomes inactive

[oliverhenshaw (Oliver Henshaw)]

oliverhenshaw added inline comments.

INLINE COMMENTS

> powerdevilcore.cpp:151
> +            // to an idle session by removing all idle timeouts
> +            KIdleTime::instance()->removeAllIdleTimeouts();
> +            m_registeredActionTimeouts.clear();

Like you said, other kidletime users might not care whether the session is active or not. This will remove all timeouts set by other kded services, right?

Just off the top of my head, Is ActionPool::unloadAllActiveActions() better here, or is that not appropriate? It's too late in the day here for me to look into that today.

REPOSITORY
  rPOWERDEVIL Powerdevil

REVISION DETAIL
  https://phabricator.kde.org/D2033

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma
Cc: oliverhenshaw, graesslin, plasma-devel, jensreuterberg, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160628/b56bbac8/attachment-0001.html>