Merge lp:~ahayzen/qtubuntu-media/fix-1531296-random-mode-repeat-for-now into lp:qtubuntu-media/stable

[Andrew Hayzen]

* Until bug 1518157 (RandomAndLoop playbackMode) is fixed set Random to be always looping due to bug 1531296

* Until bug 1518157 (RandomAndLoop playbackMode) is fixed set Random to be always looping due to bug 1531296

[Jim Hodapp (jhodapp)]

Looks good as a work around for making sure music-app doesn't have a regression from the way it works when it implemented playlists. This is a temporary fix until we decide how to solve the lack of this mode being possible with QtMultimedia's API.

review: Approve (code)

[lp:~ahayzen/qtubuntu-media/fix-1531296-random-mode-repeat-for-now]

96. By Andrew Hayzen on 2016-01-12

* Merge of lp:qtubuntu-media

97. By Andrew Hayzen on 2016-01-13

* Merge of lp:qtubuntu-media

Unmerged revisions

97. By Andrew Hayzen on 2016-01-13

* Merge of lp:qtubuntu-media

96. By Andrew Hayzen on 2016-01-12

* Merge of lp:qtubuntu-media

95. By Andrew Hayzen on 2016-01-05

* Until bug 1518157 (RandomAndLoop playbackMode) is fixed set Random to be always looping due to bug 1531296

=== modified file 'debian/changelog'
--- debian/changelog	2016-01-04 14:09:33 +0000
+++ debian/changelog	2016-01-12 14:45:37 +0000
@@ -1,3 +1,10 @@
+qtubuntu-media (0.8.0+16.04.20160105-0ubuntu1) xenial; urgency=medium
+
+  [ CI Train Bot ]
+  * No-change rebuild.
+
+ -- Thomas Voß <[e-mail]>  Tue, 05 Jan 2016 13:07:12 +0000
+
 qtubuntu-media (0.8.0+15.04.20160104-0ubuntu1) vivid; urgency=medium
 
   * Fix removeMedia(start, end) so that it always removes the correct
@@ -29,44 +36,93 @@
 
  -- Jim Hodapp <[e-mail]>  Wed, 18 Nov 2015 16:32:22 +0000
 
-qtubuntu-media (0.7.1+15.04.20151023.3-0ubuntu1) vivid; urgency=medium
-
-  [ CI Train Bot ]
-  * New rebuild forced.
-
-  [ Jim Hodapp ]
-  * Added addTracks support for batch track adding. Improved the
-    robustness of the QMediaPlaylist integration tests.
-
- -- Jim Hodapp <[e-mail]>  Fri, 23 Oct 2015 16:43:38 +0000
-
-qtubuntu-media (0.7.1+15.04.20151014-0ubuntu1) vivid; urgency=medium
-
-  [ Alfonso Sanchez-Beato ]
-  * Make sure the that the current player instance is controlable by
-    MPRIS controls if the type of player is appropriate for playlist
-    control. Also, don't add a track to the tracklist when
-    AalMediaPlayerService::setMedia() is called. (LP: #1500859)
-
-  [ Jim Hodapp ]
-  * Make sure the that the current player instance is controlable by
-    MPRIS controls if the type of player is appropriate for playlist
-    control. Also, don't add a track to the tracklist when
-    AalMediaPlayerService::setMedia() is called. (LP: #1500859)
-
- -- CI Train Bot <[e-mail]>  Wed, 14 Oct 2015 20:58:46 +0000
-
-qtubuntu-media (0.7.1+15.04.20150828-0ubuntu1) vivid; urgency=medium
-
-  [ Jim Hodapp ]
-  * Bump version for vivid sync
-
-  [ CI Train Bot ]
-  * New rebuild forced.
-
- -- CI Train Bot <[e-mail]>  Fri, 28 Aug 2015 13:52:24 +0000
-
-qtubuntu-media (0.7.1+15.04.20150710-0ubuntu1) vivid; urgency=medium
+qtubuntu-media (0.7.1+16.04.20151104.1-0ubuntu1) xenial; urgency=medium
+
+  [ Timo Jyrinki ]
+  * Port to the new Audio Role API (LP: #1493851) (LP: #1493851)
+
+ -- CI Train Bot <[e-mail]>  Wed, 04 Nov 2015 08:31:35 +0000
+
+qtubuntu-media (0.7.1+15.10.20151020.1-0ubuntu1) wily; urgency=medium
+
+  [ Alfonso Sanchez-Beato ]
+  * Make sure the that the current player instance is controlable by
+    MPRIS controls if the type of player is appropriate for playlist
+    control. Also, don't add a track to the tracklist when
+    AalMediaPlayerService::setMedia() is called. (LP: #1500859)
+
+  [ Jim Hodapp ]
+  * Make sure the that the current player instance is controlable by
+    MPRIS controls if the type of player is appropriate for playlist
+    control. Also, don't add a track to the tracklist when
+    AalMediaPlayerService::setMedia() is called. (LP: #1500859)
+
+  [ CI Train Bot ]
+  * No-change rebuild.
+
+ -- Alfonso Sanchez-Beato <[e-mail]>  Tue, 20 Oct 2015 13:10:40 +0000
+
+qtubuntu-media (0.7.1+15.10.20151019.1-0ubuntu1) wily; urgency=medium
+
+  [ CI Train Bot ]
+  * New rebuild forced.
+
+  [ Jim Hodapp ]
+  * Get rid of the unnecessary dependency on libmediascanner. Update
+    mock service interface with changes from media-hub. (LP: #1500859)
+
+ -- Jim Hodapp <[e-mail]>  Mon, 19 Oct 2015 16:21:01 +0000
+
+qtubuntu-media (0.7.1+15.10.20150903.1-0ubuntu1) wily; urgency=medium
+
+  [ Alfonso Sanchez-Beato ]
+  * [ Jim Hodapp ] Added background playlist support connecting
+    qtmultimedia with media-hub. added:
+    src/aal/aalmediaplaylistcontrol.cpp
+    src/aal/aalmediaplaylistcontrol.h
+    src/aal/aalmediaplaylistprovider.cpp
+    src/aal/aalmediaplaylistprovider.h src/aal/aalutility.cpp
+    src/aal/aalutility.h src/aal/private/
+    src/aal/private/qmediaplaylistnavigator_p.h
+    src/aal/private/qmediaplaylistprovider_p.h
+    src/aal/qmediaplaylistcontrol_p.h tests/integration/
+    tests/integration/integration.pro tests/integration/testdata/
+    tests/integration/testdata/testfile.mp4
+    tests/integration/testdata/testfile.ogg
+    tests/integration/tst_mediaplaylist.cpp
+    tests/integration/tst_mediaplaylist.h tests/tests.pro tests/unit/
+    tests/unit/player_mock.cpp tests/unit/player_mock.h
+    tests/unit/tst_mediaplaylistcontrol.cpp
+    tests/unit/tst_mediaplaylistcontrol.h renamed: unittests/ => tests/
+    unittests/player.cpp => tests/unit/player.cpp unittests/player.h =>
+    tests/unit/player.h unittests/service.cpp => tests/unit/service.cpp
+    unittests/service.h => tests/unit/service.h
+    unittests/setup_mediascanner.sh => tests/unit/setup_mediascanner.sh
+    unittests/tst_mediaplayerplugin.cpp =>
+    tests/unit/tst_mediaplayerplugin.cpp
+    unittests/tst_mediaplayerplugin.h =>
+    tests/unit/tst_mediaplayerplugin.h
+    unittests/tst_metadatareadercontrol.cpp =>
+    tests/unit/tst_metadatareadercontrol.cpp
+    unittests/tst_metadatareadercontrol.h =>
+    tests/unit/tst_metadatareadercontrol.h unittests/unittests.pro =>
+    tests/unit/unit.pro
+
+ -- CI Train Bot <[e-mail]>  Thu, 03 Sep 2015 08:25:29 +0000
+
+qtubuntu-media (0.7.1+15.10.20150710-0ubuntu2~gcc5.2) wily; urgency=medium
+
+  * No-change test rebuild for g++5 ABI transition
+
+ -- Matthias Klose <[e-mail]>  Thu, 30 Jul 2015 12:09:11 +0200
+
+qtubuntu-media (0.7.1+15.10.20150710-0ubuntu2~gcc5.1) wily; urgency=medium
+
+  * No-change test rebuild for g++5 ABI transition
+
+ -- Steve Langasek <[e-mail]>  Wed, 22 Jul 2015 23:30:42 +0000
+
+qtubuntu-media (0.7.1+15.10.20150710-0ubuntu1) wily; urgency=medium
 
   [ CI Train Bot ]
   * New rebuild forced.
@@ -77,7 +133,7 @@
 
  -- CI Train Bot <[e-mail]>  Fri, 10 Jul 2015 18:12:45 +0000
 
-qtubuntu-media (0.7.1+15.04.20150602.3-0ubuntu1) vivid; urgency=medium
+qtubuntu-media (0.7.1+15.10.20150602.3-0ubuntu1) wily; urgency=medium
 
   [ CI Train Bot ]
   * New rebuild forced.
@@ -88,7 +144,7 @@
 
  -- CI Train Bot <[e-mail]>  Tue, 02 Jun 2015 19:37:22 +0000
 
-qtubuntu-media (0.7.1+15.04.20150602.1-0ubuntu1) vivid; urgency=medium
+qtubuntu-media (0.7.1+15.10.20150602.1-0ubuntu1) wily; urgency=medium
 
   [ CI Train Bot ]
   * New rebuild forced.
@@ -98,7 +154,7 @@
 
  -- CI Train Bot <[e-mail]>  Tue, 02 Jun 2015 16:55:53 +0000
 
-qtubuntu-media (0.7.1+15.04.20150527.2-0ubuntu1) vivid; urgency=medium
+qtubuntu-media (0.7.1+15.10.20150527.2-0ubuntu1) wily; urgency=medium
 
   [ CI Train Bot ]
   * New rebuild forced.
@@ -110,7 +166,7 @@
 
  -- CI Train Bot <[e-mail]>  Wed, 27 May 2015 18:38:32 +0000
 
-qtubuntu-media (0.7.1+15.04.20150522-0ubuntu1) vivid; urgency=medium
+qtubuntu-media (0.7.1+15.10.20150522-0ubuntu1) wily; urgency=medium
 
   [ Jim Hodapp ]
   * Make sure that stopped and ready (same state for QMediaPlayer) are
@@ -118,6 +174,17 @@
 
  -- CI Train Bot <[e-mail]>  Fri, 22 May 2015 17:36:57 +0000
 
+qtubuntu-media (0.7.1+15.04.20151023.3-0ubuntu1) vivid; urgency=medium
+
+  [ CI Train Bot ]
+  * New rebuild forced.
+
+  [ Jim Hodapp ]
+  * Added addTracks support for batch track adding. Improved the
+    robustness of the QMediaPlaylist integration tests.
+
+ -- Jim Hodapp <[e-mail]>  Fri, 23 Oct 2015 16:43:38 +0000
+
 qtubuntu-media (0.7.1+15.04.20150513.1-0ubuntu1) vivid; urgency=medium
 
   [ CI Train Bot ]

=== modified file 'src/aal/aalmediaplayercontrol.cpp'
--- src/aal/aalmediaplayercontrol.cpp	2015-11-04 17:17:50 +0000
+++ src/aal/aalmediaplayercontrol.cpp	2016-01-12 14:45:37 +0000
@@ -61,12 +61,20 @@
     return m_status;
 }
 
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
 QMediaPlayer::AudioRole AalMediaPlayerControl::audioRole() const
+#else
+QAudio::Role AalMediaPlayerControl::audioRole() const
+#endif
 {
     return m_service->audioRole();
 }
 
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
 void AalMediaPlayerControl::setAudioRole(QMediaPlayer::AudioRole audioRole)
+#else
+void AalMediaPlayerControl::setAudioRole(QAudio::Role audioRole)
+#endif
 {
     qDebug() << __PRETTY_FUNCTION__;
     m_service->setAudioRole(audioRole);

=== modified file 'src/aal/aalmediaplayercontrol.h'
--- src/aal/aalmediaplayercontrol.h	2015-07-24 18:44:13 +0000
+++ src/aal/aalmediaplayercontrol.h	2016-01-12 14:45:37 +0000
@@ -32,8 +32,13 @@
 
     virtual QMediaPlayer::State state() const;
     virtual QMediaPlayer::MediaStatus mediaStatus() const;
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
     virtual QMediaPlayer::AudioRole audioRole() const;
     virtual void setAudioRole(QMediaPlayer::AudioRole audioRole);
+#else
+    virtual QAudio::Role audioRole() const;
+    virtual void setAudioRole(QAudio::Role audioRole);
+#endif
     virtual qint64 duration() const;
     virtual qint64 position() const;
     virtual void setPosition(qint64);

=== modified file 'src/aal/aalmediaplayerservice.cpp'
--- src/aal/aalmediaplayerservice.cpp	2015-11-02 14:26:44 +0000
+++ src/aal/aalmediaplayerservice.cpp	2016-01-12 14:45:37 +0000
@@ -228,21 +228,41 @@
         m_videoOutput->playbackComplete();
 }
 
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
 QMediaPlayer::AudioRole AalMediaPlayerService::audioRole() const
+#else
+QAudio::Role AalMediaPlayerService::audioRole() const
+#endif
 {
     if (m_hubPlayerSession == NULL)
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
         return QMediaPlayer::MultimediaRole;
+#else
+        return QAudio::VideoRole;
+#endif
 
     try {
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
         return static_cast<QMediaPlayer::AudioRole>(m_hubPlayerSession->audio_stream_role().get());
+#else
+        return static_cast<QAudio::Role>(m_hubPlayerSession->audio_stream_role().get());
+#endif
     }
     catch (const std::runtime_error &e) {
         qWarning() << "Failed to get audio stream role: " << e.what();
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
         return QMediaPlayer::MultimediaRole;
+#else
+        return QAudio::VideoRole;
+#endif
     }
 }
 
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
 void AalMediaPlayerService::setAudioRole(QMediaPlayer::AudioRole audioRole)
+#else
+void AalMediaPlayerService::setAudioRole(QAudio::Role audioRole)
+#endif
 {
     if (m_hubPlayerSession == NULL)
         return;
@@ -756,7 +776,11 @@
 {
         // If this player is a multimedia audioRole, then it should possible to
         // use it for MPRIS control
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
     if (audioRole() == QMediaPlayer::MultimediaRole)
+#else
+    if (audioRole() == QAudio::VideoRole || audioRole() == QAudio::MusicRole)
+#endif
     {
         qDebug() << "Setting player as current player";
         try {

=== modified file 'src/aal/aalmediaplayerservice.h'
--- src/aal/aalmediaplayerservice.h	2015-10-09 20:06:40 +0000
+++ src/aal/aalmediaplayerservice.h	2016-01-12 14:45:37 +0000
@@ -74,8 +74,13 @@
     // Call this before attempting to play the same video a second time (after EOS)
     void resetVideoSink();
 
+#if QT_VERSION < QT_VERSION_CHECK(5, 5, 0)
     QMediaPlayer::AudioRole audioRole() const;
     void setAudioRole(QMediaPlayer::AudioRole audioRole);
+#else
+    QAudio::Role audioRole() const;
+    void setAudioRole(QAudio::Role audioRole);
+#endif
 
     void setMedia(const QUrl &url);
     void setMediaPlaylist(const QMediaPlaylist& playlist);

=== modified file 'src/aal/aalmediaplaylistcontrol.cpp'
--- src/aal/aalmediaplaylistcontrol.cpp	2016-01-04 14:07:05 +0000
+++ src/aal/aalmediaplaylistcontrol.cpp	2016-01-12 14:45:37 +0000
@@ -232,6 +232,9 @@
         case QMediaPlaylist::Random:
             qDebug() << "PlaybackMode: Random";
             m_hubPlayerSession->shuffle() = true;
+            // FIXME: Until pad.lv/1518157 (RandomAndLoop playbackMode) is
+            // fixed set Random to be always looping due to pad.lv/1531296
+            m_hubPlayerSession->loop_status() = media::Player::LoopStatus::playlist;
             break;
         default:
             qWarning() << "Unknown playback mode: " << mode;

=== modified file 'tests/unit/service.h'
--- tests/unit/service.h	2015-09-08 19:31:18 +0000
+++ tests/unit/service.h	2016-01-12 14:45:37 +0000
@@ -50,7 +50,7 @@
     virtual void destroy_session(const std::string& uuid, const Player::Configuration&);
     virtual std::shared_ptr<Player> create_fixed_session(const std::string& name, const Player::Configuration&);
     virtual std::shared_ptr<Player> resume_session(Player::PlayerKey);
-    virtual void set_current_player(Player::PlayerKey key);
+    virtual void set_current_player(Player::PlayerKey);
     virtual void pause_other_sessions(Player::PlayerKey);
 };
 }