D6100: Only enable MPRIS global shortcuts on demand

[David Edmundson]

davidedmundson created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  The lockscreen uses the MPRIS dataengine. This leads to two applications
  registring the service using the same component name which leads to the
  shortcuts not working when the lock screen closes.
  
  We want to have the applet register the shortcuts, but not every
  dataengine user.
  
  Ideally this shouldn't be in the dataengine either because DataEngines
  are read only. This moves the logic to the service which is writable and
  registers the shortcuts for the multiplexer there when requested from
  the applet.
  
  BUG: 380526
  FIXED-IN: 5.10.1

TEST PLAN
  Opened media player. 
  Used play/pause shortcut. (the only one my keyboard has)

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

AFFECTED FILES
  applets/mediacontroller/contents/ui/main.qml
  dataengines/mpris2/multiplexedservice.cpp
  dataengines/mpris2/multiplexedservice.h
  dataengines/mpris2/multiplexer.cpp

To: davidedmundson, #plasma
Cc: 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/07ddf24b/attachment.html>