Merge lp://qastaging/~phablet-team/media-hub/fix-1506953 into lp://qastaging/media-hub

Proposed by Jim Hodapp
Status: Merged
Approved by: Konrad Zapałowicz
Approved revision: 204
Merged at revision: 202
Proposed branch: lp://qastaging/~phablet-team/media-hub/fix-1506953
Merge into: lp://qastaging/media-hub
Diff against target: 346 lines (+119/-23)
11 files modified
CMakeLists.txt (+2/-2)
debian/changelog (+7/-0)
include/core/media/service.h (+5/-0)
include/core/media/track.h (+0/-1)
src/core/media/player_stub.cpp (+2/-0)
src/core/media/service_implementation.cpp (+15/-0)
src/core/media/service_implementation.h (+3/-0)
src/core/media/service_skeleton.cpp (+14/-0)
src/core/media/service_skeleton.h (+3/-0)
src/core/media/service_stub.cpp (+52/-18)
src/core/media/service_stub.h (+16/-2)
To merge this branch: bzr merge lp://qastaging/~phablet-team/media-hub/fix-1506953
Reviewer Review Type Date Requested Status
Konrad Zapałowicz (community) code Approve
Review via email: mp+300531@code.qastaging.launchpad.net

Commit message

Emit a signal on the media-hub-client side when the dbus media-hub-server to client connection unregisters and re-registers.

Description of the change

Emit a signal on the media-hub-client side when the dbus media-hub-server to client connection unregisters and re-registers.

To post a comment you must log in.
202. By Jim Hodapp

Merge with upstream

203. By Jim Hodapp

Version bump for API change

204. By Jim Hodapp

Get rid of unnecessary debug statements

Revision history for this message
Konrad Zapałowicz (kzapalowicz) wrote :

LGTM

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: