Merge lp://staging/~phablet-team/qtubuntu-media/background-playlist-plugin into lp://staging/qtubuntu-media

Proposed by Jim Hodapp
Status: Merged
Merged at revision: 84
Proposed branch: lp://staging/~phablet-team/qtubuntu-media/background-playlist-plugin
Merge into: lp://staging/qtubuntu-media
Diff against target: 3109 lines (+2477/-141)
29 files modified
aalMediaPlayer.pro (+1/-1)
debian/control (+5/-0)
src/aal/aal.pro (+8/-4)
src/aal/aalmediaplayercontrol.cpp (+16/-60)
src/aal/aalmediaplayercontrol.h (+0/-2)
src/aal/aalmediaplayerservice.cpp (+142/-52)
src/aal/aalmediaplayerservice.h (+11/-4)
src/aal/aalmediaplayerserviceplugin.cpp (+0/-1)
src/aal/aalmediaplaylistcontrol.cpp (+299/-0)
src/aal/aalmediaplaylistcontrol.h (+82/-0)
src/aal/aalmediaplaylistprovider.cpp (+350/-0)
src/aal/aalmediaplaylistprovider.h (+83/-0)
src/aal/aalutility.cpp (+39/-0)
src/aal/aalutility.h (+35/-0)
src/aal/private/qmediaplaylistnavigator_p.h (+112/-0)
src/aal/private/qmediaplaylistprovider_p.h (+115/-0)
src/aal/qmediaplaylistcontrol_p.h (+96/-0)
tests/integration/integration.pro (+23/-0)
tests/integration/tst_mediaplaylist.cpp (+396/-0)
tests/integration/tst_mediaplaylist.h (+75/-0)
tests/tests.pro (+9/-0)
tests/unit/player.cpp (+6/-0)
tests/unit/player.h (+1/-0)
tests/unit/player_mock.cpp (+313/-0)
tests/unit/player_mock.h (+104/-0)
tests/unit/tst_mediaplayerplugin.cpp (+10/-2)
tests/unit/tst_mediaplaylistcontrol.cpp (+66/-0)
tests/unit/tst_mediaplaylistcontrol.h (+59/-0)
tests/unit/unit.pro (+21/-15)
To merge this branch: bzr merge lp://staging/~phablet-team/qtubuntu-media/background-playlist-plugin
Reviewer Review Type Date Requested Status
Ubuntu Phablet Team Pending
Review via email: mp+263647@code.staging.launchpad.net

Commit message

Added background playlist support connecting qtmultimedia with media-hub.

Description of the change

Added background playlist support connecting qtmultimedia with media-hub.

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

Adds nextIndex() and PlaybackMode

85. By Jim Hodapp

Added tests for verifying sequential and same track in a loop playback modes. Also changed on_track_added and on_track_changed handlers to used a Qt::QueuedConnection to separate the main thread.

86. By Jim Hodapp

Implemented test for tracklist previous.

87. By Jim Hodapp

Enable all integration tests for playlists.

88. By Jim Hodapp

Merged with Simon's branch.

89. By Jim Hodapp

Don't let the promise be fulfilled twice for verifyPlaybackModeSequential() and verifyPlaybackModeCurrentItemInLoop.

90. By Jim Hodapp

Including the necessary private Qt5 headers for qmediaplaylistcontrol and qmediaplaylistprovider.

91. By Jim Hodapp

Disable use of aalmetadatareadercontrol.

92. By Jim Hodapp

The verifyCurrentIndex() test passes every time now after making sure to wait for the currentMediaChanged signal. Also, added a test to construct/delete a player/playlist combo many times.

93. By Jim Hodapp

Disable all integration tests for the playlist stuff right now until we can figure out how to only have them run on a phone target since media-hub can't run on the desktop yet without crashing.

94. By Jim Hodapp

previousIndex() almost works 100%, just need to fix the case where steps > tracklistSize

95. By Jim Hodapp

Merged with Simon's branch to conditionally run integration tests if on a phone target only.

96. By Jim Hodapp

previousIndex() is working but isn't 100% ideal.

97. By Jim Hodapp

Merged with Simon's branch to disable unit tests for now.

98. By Jim Hodapp

Don't run the integration tests right now as they can't run during the package build process.

99. By Jim Hodapp

Remove the older code for setting a playlist.

100. By Jim Hodapp

Make sure to handle and signal removal to the client app of a track correctly.

101. By Jim Hodapp

Make sure toggling random playback on/off works.

102. By Jim Hodapp

Detach and Reattach the media-hub player session when the application goes to the background or foreground respectively.

103. By Jim Hodapp

Get rid of old unused playlist pushing code.

104. By Jim Hodapp

Fix issue where playing a single track wouldn't work.

105. By Jim Hodapp

Make sure non-playlist based playback is still possible and make sure video playback works with dettach/reattach media-hub player session

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: