Pitel updated this revision to Diff 30963. Pitel added a comment.
Convert `changeVolume` helper into a Job... Right now it does not check wheter the DBus call is successful - I tried to wire it in but ended with segfault. I will investigate it further. Except that I hope it is ok. (I also started thinking how to change the job system into something easier to work with. Were there any attempts in this direction?) REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11021?vs=28729&id=30963 BRANCH mediacontroller REVISION DETAIL https://phabricator.kde.org/D11021 AFFECTED FILES applets/mediacontroller/contents/ui/main.qml dataengines/mpris2/mpris2.operations dataengines/mpris2/multiplexedservice.cpp dataengines/mpris2/playeractionjob.cpp dataengines/mpris2/playercontrol.cpp dataengines/mpris2/playercontrol.h To: Pitel, #plasma, broulik Cc: broulik, nicolasfella, plasma-devel, ragreen, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart