lp:~thomas-voss/media-hub/use-mpris-player-skeleton-and-register-with-indicator

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:media-hub at revision 63

Jim Hodapp (community): Approve (code) on 2014-09-10

PS Jenkins bot: Needs Fixing (continuous-integration) on 2014-09-10

Marcus Tomlinson (community): Approve on 2014-09-10

Nick Dedekind (community): Approve on 2014-09-10

Diff: 3386 lines (+1603/-823)

35 files modified

CMakeLists.txt (+1/-1)
debian/changelog (+6/-0)
debian/control (+4/-4)
debian/libmedia-hub-common2.symbols (+2/-19)
include/core/media/player.h (+3/-3)
src/core/media/CMakeLists.txt (+9/-0)
src/core/media/apparmor.h (+62/-20)
src/core/media/cover_art_resolver.cpp (+27/-0)
src/core/media/cover_art_resolver.h (+51/-0)
src/core/media/engine.cpp (+0/-114)
src/core/media/engine.h (+0/-23)
src/core/media/gstreamer/engine.cpp (+5/-99)
src/core/media/gstreamer/meta_data_extractor.h (+92/-86)
src/core/media/mpris/media_player2.h (+64/-2)
src/core/media/mpris/metadata.h (+55/-0)
src/core/media/mpris/player.h (+181/-53)
src/core/media/mpris/playlists.h (+216/-0)
src/core/media/player.cpp (+7/-1)
src/core/media/player_configuration.h (+9/-0)
src/core/media/player_implementation.cpp (+24/-3)
src/core/media/player_implementation.h (+2/-0)
src/core/media/player_skeleton.cpp (+176/-208)
src/core/media/player_skeleton.h (+21/-11)
src/core/media/player_stub.cpp (+4/-4)
src/core/media/player_stub.h (+3/-3)
src/core/media/service_implementation.cpp (+84/-99)
src/core/media/service_implementation.h (+5/-2)
src/core/media/service_skeleton.cpp (+347/-34)
src/core/media/service_skeleton.h (+29/-2)
src/core/media/the_session_bus.cpp (+1/-1)
src/core/media/the_session_bus.h (+9/-1)
src/core/media/xesam.h (+56/-0)
symbols.map (+16/-0)
tests/unit-tests/CMakeLists.txt (+1/-0)
tests/unit-tests/test-gstreamer-engine.cpp (+31/-30)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1260978: [Greeter] should be able to control music service while greeter locked

High

Fix Released

Link a bug report

Related blueprints

96. By Thomas Voß on 2014-09-10

Address reviewer comments.

95. By Thomas Voß on 2014-09-10

Wire up PlayPause correctly.

94. By Thomas Voß on 2014-09-10

Adjust copyright dates.

93. By Thomas Voß on 2014-09-09

Unset current player if client disconnects.

92. By Thomas Voß on 2014-09-09

Just a criss-cross merge with prerequisite branch.

91. By Thomas Voß on 2014-09-09

[ Jim Hodapp ]
* When power hits the low or very low levels, pause all Players with
  role of multimedia. When the warning notification is cleared from
  the screen, resume playback.
[ Ricardo Salveti de Araujo ]
* playbin: fixing audio-sink gst property name

90. By Thomas Voß on 2014-09-09

Factor out cover art resolver.

89. By Thomas Voß on 2014-09-09

Play/Pause should call play() or pause(), not next().

88. By Thomas Voß on 2014-09-09

Only make player current if it is providing a multi-media stream.

87. By Thomas Voß on 2014-09-09

Move exposure to MPRIS-handlers to the service, and dynamically switch players as they are created and made current.