D6100: Only enable MPRIS global shortcuts on demand

[Kai Uwe Broulik]

broulik accepted this revision.
broulik added a comment.
This revision is now accepted and ready to land.


  Good idea. I'm somewhat concerned by the fact that media controller applet only loads when you have an mpris service running and then we have applications like Amarok that also try to register them which might do so before we do this here? Certainly better approach than my original patch.

INLINE COMMENTS

> multiplexedservice.h:43
> +public Q_SLOTS:
> +    void enableGlobalShortcuts();
> +

Doesn't this need to be registered in some .services file? Or is that just needed for "startOperationCall"?

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

To: davidedmundson, #plasma, broulik
Cc: broulik, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170605/07ffe684/attachment-0001.html>