summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--src/multimedia/playback/playback.pri10
-rw-r--r--src/multimedia/playback/qlocalmediaplaylistprovider.cpp194
-rw-r--r--src/multimedia/playback/qlocalmediaplaylistprovider_p.h98
-rw-r--r--src/multimedia/playback/qmediaplayer.cpp403
-rw-r--r--src/multimedia/playback/qmediaplayer.h8
-rw-r--r--src/multimedia/playback/qmediaplaylist.cpp6
-rw-r--r--src/multimedia/playback/qmediaplaylist_p.h16
-rw-r--r--tests/auto/unit/qmediaplayer/tst_qmediaplayer.cpp85
-rw-r--r--tests/auto/unit/qmediaplaylistnavigator/tst_qmediaplaylistnavigator.cpp30
9 files changed, 407 insertions, 443 deletions
diff --git a/src/multimedia/playback/playback.pri b/src/multimedia/playback/playback.pri
index 284c9f8e6..327b63b32 100644
--- a/src/multimedia/playback/playback.pri
+++ b/src/multimedia/playback/playback.pri
@@ -11,16 +11,18 @@ PRIVATE_HEADERS += \
playback/qmediaplaylist_p.h \
playback/qmediaplaylistprovider_p.h \
playback/qmediaplaylistioplugin_p.h \
- playback/qlocalmediaplaylistprovider_p.h \
- playback/qmediaplaylistnavigator_p.h
+ playback/qmediaplaylistnavigator_p.h \
+ playback/qmedianetworkplaylistprovider_p.h \
+ playback/playlistfileparser_p.h
SOURCES += \
playback/qaudioendpointselector.cpp \
- playback/qlocalmediaplaylistprovider.cpp \
+ playback/qmedianetworkplaylistprovider.cpp \
playback/qmediacontent.cpp \
playback/qmediaplayer.cpp \
playback/qmediaplaylist.cpp \
playback/qmediaplaylistioplugin.cpp \
playback/qmediaplaylistnavigator.cpp \
playback/qmediaplaylistprovider.cpp \
- playback/qmediaresource.cpp
+ playback/qmediaresource.cpp \
+ playback/playlistfileparser.cpp
diff --git a/src/multimedia/playback/qlocalmediaplaylistprovider.cpp b/src/multimedia/playback/qlocalmediaplaylistprovider.cpp
deleted file mode 100644
index 051f76a9b..000000000
--- a/src/multimedia/playback/qlocalmediaplaylistprovider.cpp
+++ /dev/null
@@ -1,194 +0,0 @@
-/****************************************************************************
-**
-** Copyright (C) 2012 Nokia Corporation and/or its subsidiary(-ies).
-** Contact: http://www.qt-project.org/
-**
-** This file is part of the Qt Toolkit.
-**
-** $QT_BEGIN_LICENSE:LGPL$
-** GNU Lesser General Public License Usage
-** This file may be used under the terms of the GNU Lesser General Public
-** License version 2.1 as published by the Free Software Foundation and
-** appearing in the file LICENSE.LGPL included in the packaging of this
-** file. Please review the following information to ensure the GNU Lesser
-** General Public License version 2.1 requirements will be met:
-** http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html.
-**
-** In addition, as a special exception, Nokia gives you certain additional
-** rights. These rights are described in the Nokia Qt LGPL Exception
-** version 1.1, included in the file LGPL_EXCEPTION.txt in this package.
-**
-** GNU General Public License Usage
-** Alternatively, this file may be used under the terms of the GNU General
-** Public License version 3.0 as published by the Free Software Foundation
-** and appearing in the file LICENSE.GPL included in the packaging of this
-** file. Please review the following information to ensure the GNU General
-** Public License version 3.0 requirements will be met:
-** http://www.gnu.org/copyleft/gpl.html.
-**
-** Other Usage
-** Alternatively, this file may be used in accordance with the terms and
-** conditions contained in a signed written agreement between you and Nokia.
-**
-**
-**
-**
-**
-**
-** $QT_END_LICENSE$
-**
-****************************************************************************/
-
-#include "qlocalmediaplaylistprovider_p.h"
-#include "qmediaplaylistprovider_p.h"
-#include "qmediacontent.h"
-
-QT_BEGIN_NAMESPACE
-
-class QLocalMediaPlaylistProviderPrivate: public QMediaPlaylistProviderPrivate
-{
-public:
- QList<QMediaContent> resources;
-};
-
-QLocalMediaPlaylistProvider::QLocalMediaPlaylistProvider(QObject *parent)
- :QMediaPlaylistProvider(*new QLocalMediaPlaylistProviderPrivate, parent)
-{
-}
-
-QLocalMediaPlaylistProvider::~QLocalMediaPlaylistProvider()
-{
-}
-
-bool QLocalMediaPlaylistProvider::isReadOnly() const
-{
- return false;
-}
-
-int QLocalMediaPlaylistProvider::mediaCount() const
-{
- return d_func()->resources.size();
-}
-
-QMediaContent QLocalMediaPlaylistProvider::media(int pos) const
-{
- return d_func()->resources.value(pos);
-}
-
-bool QLocalMediaPlaylistProvider::addMedia(const QMediaContent &content)
-{
- Q_D(QLocalMediaPlaylistProvider);
-
- int pos = d->resources.count();
-
- emit mediaAboutToBeInserted(pos, pos);
- d->resources.append(content);
- emit mediaInserted(pos, pos);
-
- return true;
-}
-
-bool QLocalMediaPlaylistProvider::addMedia(const QList<QMediaContent> &items)
-{
- Q_D(QLocalMediaPlaylistProvider);
-
- if (items.isEmpty())
- return true;
-
- int pos = d->resources.count();
- int end = pos+items.count()-1;
-
- emit mediaAboutToBeInserted(pos, end);
- d->resources.append(items);
- emit mediaInserted(pos, end);
-
- return true;
-}
-
-
-bool QLocalMediaPlaylistProvider::insertMedia(int pos, const QMediaContent &content)
-{
- Q_D(QLocalMediaPlaylistProvider);
-
- emit mediaAboutToBeInserted(pos, pos);
- d->resources.insert(pos, content);
- emit mediaInserted(pos,pos);
-
- return true;
-}
-
-bool QLocalMediaPlaylistProvider::insertMedia(int pos, const QList<QMediaContent> &items)
-{
- Q_D(QLocalMediaPlaylistProvider);
-
- if (items.isEmpty())
- return true;
-
- const int last = pos+items.count()-1;
-
- emit mediaAboutToBeInserted(pos, last);
- for (int i=0; i<items.count(); i++)
- d->resources.insert(pos+i, items.at(i));
- emit mediaInserted(pos, last);
-
- return true;
-}
-
-bool QLocalMediaPlaylistProvider::removeMedia(int fromPos, int toPos)
-{
- Q_D(QLocalMediaPlaylistProvider);
-
- Q_ASSERT(fromPos >= 0);
- Q_ASSERT(fromPos <= toPos);
- Q_ASSERT(toPos < mediaCount());
-
- emit mediaAboutToBeRemoved(fromPos, toPos);
- d->resources.erase(d->resources.begin()+fromPos, d->resources.begin()+toPos+1);
- emit mediaRemoved(fromPos, toPos);
-
- return true;
-}
-
-bool QLocalMediaPlaylistProvider::removeMedia(int pos)
-{
- Q_D(QLocalMediaPlaylistProvider);
-
- emit mediaAboutToBeRemoved(pos, pos);
- d->resources.removeAt(pos);
- emit mediaRemoved(pos, pos);
-
- return true;
-}
-
-bool QLocalMediaPlaylistProvider::clear()
-{
- Q_D(QLocalMediaPlaylistProvider);
- if (!d->resources.isEmpty()) {
- int lastPos = mediaCount()-1;
- emit mediaAboutToBeRemoved(0, lastPos);
- d->resources.clear();
- emit mediaRemoved(0, lastPos);
- }
-
- return true;
-}
-
-void QLocalMediaPlaylistProvider::shuffle()
-{
- Q_D(QLocalMediaPlaylistProvider);
- if (!d->resources.isEmpty()) {
- QList<QMediaContent> resources;
-
- while (!d->resources.isEmpty()) {
- resources.append(d->resources.takeAt(qrand() % d->resources.size()));
- }
-
- d->resources = resources;
- emit mediaChanged(0, mediaCount()-1);
- }
-
-}
-
-#include "moc_qlocalmediaplaylistprovider_p.cpp"
-QT_END_NAMESPACE
-
diff --git a/src/multimedia/playback/qlocalmediaplaylistprovider_p.h b/src/multimedia/playback/qlocalmediaplaylistprovider_p.h
deleted file mode 100644
index 879e8f715..000000000
--- a/src/multimedia/playback/qlocalmediaplaylistprovider_p.h
+++ /dev/null
@@ -1,98 +0,0 @@
-/****************************************************************************
-**
-** Copyright (C) 2012 Nokia Corporation and/or its subsidiary(-ies).
-** Contact: http://www.qt-project.org/
-**
-** This file is part of the Qt Toolkit.
-**
-** $QT_BEGIN_LICENSE:LGPL$
-** GNU Lesser General Public License Usage
-** This file may be used under the terms of the GNU Lesser General Public
-** License version 2.1 as published by the Free Software Foundation and
-** appearing in the file LICENSE.LGPL included in the packaging of this
-** file. Please review the following information to ensure the GNU Lesser
-** General Public License version 2.1 requirements will be met:
-** http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html.
-**
-** In addition, as a special exception, Nokia gives you certain additional
-** rights. These rights are described in the Nokia Qt LGPL Exception
-** version 1.1, included in the file LGPL_EXCEPTION.txt in this package.
-**
-** GNU General Public License Usage
-** Alternatively, this file may be used under the terms of the GNU General
-** Public License version 3.0 as published by the Free Software Foundation
-** and appearing in the file LICENSE.GPL included in the packaging of this
-** file. Please review the following information to ensure the GNU General
-** Public License version 3.0 requirements will be met:
-** http://www.gnu.org/copyleft/gpl.html.
-**
-** Other Usage
-** Alternatively, this file may be used in accordance with the terms and
-** conditions contained in a signed written agreement between you and Nokia.
-**
-**
-**
-**
-**
-**
-** $QT_END_LICENSE$
-**
-****************************************************************************/
-
-#ifndef QLOCALMEDIAPAYLISTPROVIDER_P_H
-#define QLOCALMEDIAPAYLISTPROVIDER_P_H
-
-//
-// W A R N I N G
-// -------------
-//
-// This file is not part of the Qt API. It exists purely as an
-// implementation detail. This header file may change from version to
-// version without notice, or even be removed.
-//
-// We mean it.
-//
-
-#include "qmediaplaylistprovider_p.h"
-
-QT_BEGIN_HEADER
-
-QT_BEGIN_NAMESPACE
-
-QT_MODULE(Multimedia)
-
-
-class QLocalMediaPlaylistProviderPrivate;
-class Q_MULTIMEDIA_EXPORT QLocalMediaPlaylistProvider : public QMediaPlaylistProvider
-{
- Q_OBJECT
-public:
- QLocalMediaPlaylistProvider(QObject *parent=0);
- virtual ~QLocalMediaPlaylistProvider();
-
- virtual int mediaCount() const;
- virtual QMediaContent media(int pos) const;
-
- virtual bool isReadOnly() const;
-
- virtual bool addMedia(const QMediaContent &content);
- virtual bool addMedia(const QList<QMediaContent> &items);
- virtual bool insertMedia(int pos, const QMediaContent &content);
- virtual bool insertMedia(int pos, const QList<QMediaContent> &items);
- virtual bool removeMedia(int pos);
- virtual bool removeMedia(int start, int end);
- virtual bool clear();
-
-public Q_SLOTS:
- virtual void shuffle();
-
-private:
- Q_DECLARE_PRIVATE(QLocalMediaPlaylistProvider)
-};
-
-QT_END_NAMESPACE
-
-QT_END_HEADER
-
-
-#endif // QLOCALMEDIAPAYLISTSOURCE_P_H
diff --git a/src/multimedia/playback/qmediaplayer.cpp b/src/multimedia/playback/qmediaplayer.cpp
index 0ff341438..a699e281f 100644
--- a/src/multimedia/playback/qmediaplayer.cpp
+++ b/src/multimedia/playback/qmediaplayer.cpp
@@ -49,7 +49,6 @@
#include <qmediaplaylist.h>
#include <qmediaplaylistcontrol_p.h>
#include <qmediaplaylistsourcecontrol_p.h>
-
#include <qmedianetworkaccesscontrol.h>
#include <QtCore/qcoreevent.h>
@@ -97,6 +96,8 @@ public:
} _registerPlayerMetaTypes;
}
+#define MAX_NESTED_PLAYLISTS 16
+
class QMediaPlayerPrivate : public QMediaObjectPrivate
{
Q_DECLARE_NON_CONST_PUBLIC(QMediaPlayer)
@@ -105,47 +106,86 @@ public:
QMediaPlayerPrivate()
: provider(0)
, control(0)
- , playlistSourceControl(0)
, state(QMediaPlayer::StoppedState)
, error(QMediaPlayer::NoError)
- , filterStates(false)
, playlist(0)
, networkAccessControl(0)
+ , nestedPlaylists(0)
{}
QMediaServiceProvider *provider;
QMediaPlayerControl* control;
- QMediaPlaylistSourceControl* playlistSourceControl;
QMediaPlayer::State state;
QMediaPlayer::Error error;
QString errorString;
- bool filterStates;
QPointer<QObject> videoOutput;
QMediaPlaylist *playlist;
QMediaNetworkAccessControl *networkAccessControl;
QVideoSurfaceOutput surfaceOutput;
+ QMediaContent rootMedia;
+ QMediaContent pendingPlaylist;
+ QMediaPlaylist *parentPlaylist(QMediaPlaylist *pls);
+ bool isInChain(QUrl url);
+ int nestedPlaylists;
+
+ void setPlaylist(QMediaPlaylist *playlist);
+ void setPlaylistMedia();
+ void loadPlaylist();
+ void disconnectPlaylist();
+ void connectPlaylist();
+
void _q_stateChanged(QMediaPlayer::State state);
void _q_mediaStatusChanged(QMediaPlayer::MediaStatus status);
void _q_error(int error, const QString &errorString);
void _q_updateMedia(const QMediaContent&);
void _q_playlistDestroyed();
+ void _q_handlePlaylistLoaded();
+ void _q_handlePlaylistLoadFailed();
};
+QMediaPlaylist *QMediaPlayerPrivate::parentPlaylist(QMediaPlaylist *pls)
+{
+ // This function finds a parent playlist for an item in the active chain of playlists.
+ // Every item in the chain comes from currentMedia() of its parent.
+ // We don't need to travers the whole tree of playlists,
+ // but only the subtree of active ones.
+ for (QMediaPlaylist *current = rootMedia.playlist(); current && current != pls; current = current->currentMedia().playlist())
+ if (current->currentMedia().playlist() == pls)
+ return current;
+ return 0;
+}
+
+bool QMediaPlayerPrivate::isInChain(QUrl url)
+{
+ // Check whether a URL is already in the chain of playlists.
+ // Also see a comment in parentPlaylist().
+ for (QMediaPlaylist *current = rootMedia.playlist(); current && current != playlist; current = current->currentMedia().playlist())
+ if (current->currentMedia().canonicalUrl() == url) {
+ return true;
+ }
+ return false;
+}
+
void QMediaPlayerPrivate::_q_stateChanged(QMediaPlayer::State ps)
{
Q_Q(QMediaPlayer);
- if (filterStates)
- return;
-
- if (playlist
- && ps != state && ps == QMediaPlayer::StoppedState
- && (control->mediaStatus() == QMediaPlayer::EndOfMedia ||
- control->mediaStatus() == QMediaPlayer::InvalidMedia)) {
- playlist->next();
- ps = control->state();
+ // Backend switches into stopped state every time new media is about to be loaded.
+ // If media player has a playlist loaded make sure player doesn' stop.
+ if (playlist && playlist->currentIndex() != -1 && ps != state && ps == QMediaPlayer::StoppedState) {
+ if (control->mediaStatus() == QMediaPlayer::EndOfMedia ||
+ control->mediaStatus() == QMediaPlayer::InvalidMedia) {
+ // if media player is not stopped, and
+ // we have finished playback for the current media,
+ // advance to the next item in the playlist
+ Q_ASSERT(state != QMediaPlayer::StoppedState);
+ playlist->next();
+ return;
+ } else if (control->mediaStatus() == QMediaPlayer::LoadingMedia) {
+ return;
+ }
}
if (ps != state) {
@@ -182,10 +222,16 @@ void QMediaPlayerPrivate::_q_error(int error, const QString &errorString)
{
Q_Q(QMediaPlayer);
- this->error = QMediaPlayer::Error(error);
- this->errorString = errorString;
+ if (error == int(QMediaPlayer::MediaIsPlaylist)) {
+ loadPlaylist();
+ } else {
+ this->error = QMediaPlayer::Error(error);
+ this->errorString = errorString;
+ emit q->error(this->error);
- emit q->error(this->error);
+ if (playlist)
+ playlist->next();
+ }
}
void QMediaPlayerPrivate::_q_updateMedia(const QMediaContent &media)
@@ -195,9 +241,45 @@ void QMediaPlayerPrivate::_q_updateMedia(const QMediaContent &media)
if (!control)
return;
+ // check if the current playlist is a top-level playlist
+ Q_ASSERT(playlist);
+ if (media.isNull() && playlist != rootMedia.playlist()) {
+ // switch back to parent playlist
+ QMediaPlaylist *pls = parentPlaylist(playlist);
+ Q_ASSERT(pls);
+ disconnectPlaylist();
+ playlist = pls;
+ connectPlaylist();
+
+ Q_ASSERT(!pendingPlaylist.playlist());
+ nestedPlaylists--;
+ Q_ASSERT(nestedPlaylists >= 0);
+
+ playlist->next();
+ return;
+ }
+
+ if (media.playlist()) {
+ if (nestedPlaylists < MAX_NESTED_PLAYLISTS) {
+ nestedPlaylists++;
+ Q_ASSERT(!pendingPlaylist.playlist());
+
+ // disconnect current playlist
+ disconnectPlaylist();
+ // new playlist signals are connected
+ // in the call to setPlaylist() in _q_handlePlaylistLoaded()
+ playlist = media.playlist();
+ emit q->currentMediaChanged(media);
+ _q_handlePlaylistLoaded();
+ return;
+ } else if (playlist) {
+ playlist->next();
+ }
+ return;
+ }
+
const QMediaPlayer::State currentState = state;
- filterStates = true;
control->setMedia(media, 0);
if (!media.isNull()) {
@@ -212,18 +294,8 @@ void QMediaPlayerPrivate::_q_updateMedia(const QMediaContent &media)
break;
}
}
- filterStates = false;
- state = control->state();
-
- if (state != currentState) {
- if (state == QMediaPlayer::PlayingState)
- q->addPropertyWatch("position");
- else
- q->removePropertyWatch("position");
-
- emit q->stateChanged(state);
- }
+ _q_stateChanged(control->state());
}
void QMediaPlayerPrivate::_q_playlistDestroyed()
@@ -233,12 +305,151 @@ void QMediaPlayerPrivate::_q_playlistDestroyed()
if (!control)
return;
- if (playlistSourceControl)
- playlistSourceControl->setPlaylist(0);
-
control->setMedia(QMediaContent(), 0);
}
+void QMediaPlayerPrivate::setPlaylist(QMediaPlaylist *pls)
+{
+ disconnectPlaylist();
+ playlist = pls;
+
+ setPlaylistMedia();
+}
+
+void QMediaPlayerPrivate::setPlaylistMedia()
+{
+ // This function loads current playlist media into backend.
+ // If current media is a playlist, the function recursively
+ // loads media from the playlist.
+ // It also makes sure the correct playlist signals are connected.
+ Q_Q(QMediaPlayer);
+
+ if (playlist) {
+ connectPlaylist();
+ if (playlist->currentMedia().playlist()) {
+ if (nestedPlaylists < MAX_NESTED_PLAYLISTS) {
+ emit q->currentMediaChanged(playlist->currentMedia());
+ // rewind nested playlist to start
+ playlist->currentMedia().playlist()->setCurrentIndex(0);
+ nestedPlaylists++;
+ setPlaylist(playlist->currentMedia().playlist());
+ } else {
+ playlist->next();
+ }
+ return;
+ } else if (control != 0) {
+ // If we've just switched to a new playlist,
+ // then last emited currentMediaChanged was a playlist.
+ // Make sure we emit currentMediaChanged if new playlist has
+ // the same media as the previous one:
+ // sample.m3u
+ // test.wav -- processed by backend
+ // nested.m3u -- processed by frontend
+ // test.wav -- processed by backend,
+ // media is not changed,
+ // frontend needs to emit currentMediaChanged
+ bool isSameMedia = (control->media() == playlist->currentMedia());
+ control->setMedia(playlist->currentMedia(), 0);
+ if (isSameMedia) {
+ emit q->currentMediaChanged(control->media());
+ }
+ }
+ } else {
+ q->setMedia(QMediaContent(), 0);
+ }
+}
+
+void QMediaPlayerPrivate::loadPlaylist()
+{
+ Q_Q(QMediaPlayer);
+ Q_ASSERT(pendingPlaylist.isNull());
+
+ // Do not load a playlist if there are more than MAX_NESTED_PLAYLISTS in the chain already,
+ // or if the playlist URL is already in the chain, i.e. do not allow recursive playlists and loops.
+ if (nestedPlaylists < MAX_NESTED_PLAYLISTS && !q->currentMedia().canonicalUrl().isEmpty() && !isInChain(q->currentMedia().canonicalUrl())) {
+ pendingPlaylist = QMediaContent(new QMediaPlaylist, q->currentMedia().canonicalUrl(), true);
+ QObject::connect(pendingPlaylist.playlist(), SIGNAL(loaded()), q, SLOT(_q_handlePlaylistLoaded()));
+ QObject::connect(pendingPlaylist.playlist(), SIGNAL(loadFailed()), q, SLOT(_q_handlePlaylistLoadFailed()));
+ pendingPlaylist.playlist()->load(pendingPlaylist.canonicalUrl());
+ } else if (playlist) {
+ playlist->next();
+ }
+}
+
+void QMediaPlayerPrivate::disconnectPlaylist()
+{
+ Q_Q(QMediaPlayer);
+ if (playlist) {
+ QObject::disconnect(playlist, SIGNAL(currentMediaChanged(QMediaContent)),
+ q, SLOT(_q_updateMedia(QMediaContent)));
+ QObject::disconnect(playlist, SIGNAL(destroyed()), q, SLOT(_q_playlistDestroyed()));
+ }
+}
+
+void QMediaPlayerPrivate::connectPlaylist()
+{
+ Q_Q(QMediaPlayer);
+ if (playlist) {
+ QObject::connect(playlist, SIGNAL(currentMediaChanged(QMediaContent)),
+ q, SLOT(_q_updateMedia(QMediaContent)));
+ QObject::connect(playlist, SIGNAL(destroyed()), q, SLOT(_q_playlistDestroyed()));
+ }
+}
+
+void QMediaPlayerPrivate::_q_handlePlaylistLoaded()
+{
+ Q_Q(QMediaPlayer);
+
+ QMediaPlaylist *oldPlaylist = 0;
+ if (pendingPlaylist.playlist()) {
+ Q_ASSERT(!q->currentMedia().playlist());
+ // if there is an active playlist
+ if (playlist) {
+ Q_ASSERT(playlist->currentIndex() >= 0);
+ oldPlaylist = playlist;
+ disconnectPlaylist();
+ playlist->insertMedia(playlist->currentIndex() + 1, pendingPlaylist);
+ playlist->removeMedia(playlist->currentIndex());
+ nestedPlaylists++;
+ } else {
+ Q_ASSERT(!rootMedia.playlist());
+ rootMedia = pendingPlaylist;
+ emit q->mediaChanged(rootMedia);
+ }
+
+ playlist = pendingPlaylist.playlist();
+ emit q->currentMediaChanged(pendingPlaylist);
+ }
+ pendingPlaylist = QMediaContent();
+
+ playlist->next();
+ setPlaylistMedia();
+
+ switch (state) {
+ case QMediaPlayer::PausedState:
+ control->pause();
+ break;
+ case QMediaPlayer::PlayingState:
+ control->play();
+ break;
+ case QMediaPlayer::StoppedState:
+ break;
+ }
+}
+
+void QMediaPlayerPrivate::_q_handlePlaylistLoadFailed()
+{
+ pendingPlaylist = QMediaContent();
+
+ if (!control)
+ return;
+
+ if (playlist)
+ playlist->next();
+ else
+ control->setMedia(QMediaContent(), 0);
+}
+
static QMediaService *playerService(QMediaPlayer::Flags flags)
{
QMediaServiceProvider *provider = QMediaServiceProvider::defaultServiceProvider();
@@ -277,10 +488,9 @@ QMediaPlayer::QMediaPlayer(QObject *parent, QMediaPlayer::Flags flags):
d->error = ServiceMissingError;
} else {
d->control = qobject_cast<QMediaPlayerControl*>(d->service->requestControl(QMediaPlayerControl_iid));
- d->playlistSourceControl = qobject_cast<QMediaPlaylistSourceControl*>(d->service->requestControl(QMediaPlaylistSourceControl_iid));
d->networkAccessControl = qobject_cast<QMediaNetworkAccessControl*>(d->service->requestControl(QMediaNetworkAccessControl_iid));
if (d->control != 0) {
- connect(d->control, SIGNAL(mediaChanged(QMediaContent)), SIGNAL(mediaChanged(QMediaContent)));
+ connect(d->control, SIGNAL(mediaChanged(QMediaContent)), SIGNAL(currentMediaChanged(QMediaContent)));
connect(d->control, SIGNAL(stateChanged(QMediaPlayer::State)), SLOT(_q_stateChanged(QMediaPlayer::State)));
connect(d->control, SIGNAL(mediaStatusChanged(QMediaPlayer::MediaStatus)),
SLOT(_q_mediaStatusChanged(QMediaPlayer::MediaStatus)));
@@ -330,10 +540,7 @@ QMediaContent QMediaPlayer::media() const
{
Q_D(const QMediaPlayer);
- if (d->control != 0)
- return d->control->media();
-
- return QMediaContent();
+ return d->rootMedia;
}
/*!
@@ -356,44 +563,25 @@ const QIODevice *QMediaPlayer::mediaStream() const
QMediaPlaylist *QMediaPlayer::playlist() const
{
- return d_func()->playlistSourceControl ?
- d_func()->playlistSourceControl->playlist() :
- d_func()->playlist;
+ Q_D(const QMediaPlayer);
+
+ return d->rootMedia.playlist();
}
-void QMediaPlayer::setPlaylist(QMediaPlaylist *playlist)
+QMediaContent QMediaPlayer::currentMedia() const
{
- Q_D(QMediaPlayer);
-
- if (d->playlistSourceControl) {
- if (d->playlistSourceControl->playlist())
- disconnect(d->playlist, SIGNAL(destroyed()), this, SLOT(_q_playlistDestroyed()));
-
- d->playlistSourceControl->setPlaylist(playlist);
-
- if (playlist)
- connect(d->playlist, SIGNAL(destroyed()), this, SLOT(_q_playlistDestroyed()));
- } else {
- if (d->playlist) {
- disconnect(d->playlist, SIGNAL(currentMediaChanged(QMediaContent)),
- this, SLOT(_q_updateMedia(QMediaContent)));
- disconnect(d->playlist, SIGNAL(destroyed()), this, SLOT(_q_playlistDestroyed()));
- }
-
- d->playlist = playlist;
+ Q_D(const QMediaPlayer);
- if (d->playlist) {
- connect(d->playlist, SIGNAL(currentMediaChanged(QMediaContent)),
- this, SLOT(_q_updateMedia(QMediaContent)));
- connect(d->playlist, SIGNAL(destroyed()), this, SLOT(_q_playlistDestroyed()));
+ if (d->control != 0)
+ return d->control->media();
- if (d->control != 0)
- d->control->setMedia(playlist->currentMedia(), 0);
- } else {
- setMedia(QMediaContent(), 0);
- }
+ return QMediaContent();
+}
- }
+void QMediaPlayer::setPlaylist(QMediaPlaylist *playlist)
+{
+ QMediaContent m(playlist, QUrl(), false);
+ setMedia(m);
}
/*!
@@ -563,8 +751,18 @@ void QMediaPlayer::play()
}
//if playlist control is available, the service should advance itself
- if (d->playlist && d->playlist->currentIndex() == -1 && !d->playlist->isEmpty())
+ if (d->rootMedia.playlist() && d->rootMedia.playlist()->currentIndex() == -1 && !d->rootMedia.playlist()->isEmpty()) {
+
+ // switch to playing state
+ if (d->state != QMediaPlayer::PlayingState)
+ d->_q_stateChanged(QMediaPlayer::PlayingState);
+
+ if (d->playlist != d->rootMedia.playlist())
+ d->setPlaylist(d->rootMedia.playlist());
+ Q_ASSERT(d->playlist == d->rootMedia.playlist());
+ emit currentMediaChanged(d->rootMedia);
d->playlist->setCurrentIndex(0);
+ }
// Reset error conditions
d->error = NoError;
@@ -595,6 +793,17 @@ void QMediaPlayer::stop()
if (d->control != 0)
d->control->stop();
+
+ // If media player didn't stop in response to control.
+ // This happens if we have an active playlist and control
+ // media status is
+ // QMediaPlayer::LoadingMedia, QMediaPlayer::InvalidMedia, or QMediaPlayer::EndOfMedia
+ // see QMediaPlayerPrivate::_q_stateChanged()
+ if (d->playlist && d->state != QMediaPlayer::StoppedState) {
+ d->state = QMediaPlayer::StoppedState;
+ removePropertyWatch("position");
+ emit stateChanged(QMediaPlayer::StoppedState);
+ }
}
void QMediaPlayer::setPosition(qint64 position)
@@ -654,12 +863,24 @@ void QMediaPlayer::setPlaybackRate(qreal rate)
void QMediaPlayer::setMedia(const QMediaContent &media, QIODevice *stream)
{
Q_D(QMediaPlayer);
-
- if (playlist() && playlist()->currentMedia() != media)
- setPlaylist(0);
-
- if (d->control != 0)
- d_func()->control->setMedia(media, stream);
+ stop();
+
+ QMediaContent oldMedia = d->rootMedia;
+ d->disconnectPlaylist();
+ d->playlist = 0;
+ d->rootMedia = media;
+ d->nestedPlaylists = 0;
+
+ if (oldMedia != media)
+ emit mediaChanged(d->rootMedia);
+
+ if (media.playlist()) {
+ // reset playlist to the 1st item
+ media.playlist()->setCurrentIndex(0);
+ d->setPlaylist(media.playlist());
+ } else if (d->control != 0) {
+ d->control->setMedia(media, stream);
+ }
}
/*!
@@ -796,6 +1017,7 @@ QtMultimedia::AvailabilityError QMediaPlayer::availabilityError() const
return QMediaObject::availabilityError();
}
+
// Enums
/*!
\enum QMediaPlayer::State
@@ -841,6 +1063,7 @@ QtMultimedia::AvailabilityError QMediaPlayer::availabilityError() const
\value NetworkError A network error occurred.
\value AccessDeniedError There are not the appropriate permissions to play a media resource.
\value ServiceMissingError A valid playback service was not found, playback cannot proceed.
+ \omitvalue MediaIsPlaylist
*/
// Signals
@@ -869,9 +1092,17 @@ QtMultimedia::AvailabilityError QMediaPlayer::availabilityError() const
/*!
\fn void QMediaPlayer::mediaChanged(const QMediaContent &media);
- Signals that the current playing content will be obtained from \a media.
+ Signals that the media source has been changed to \a media.
+
+ \sa media(), currentMediaChanged()
+*/
+
+/*!
+ \fn void QMediaPlayer::currentMediaChanged(const QMediaContent &media);
+
+ Signals that the current playing content has been changed to \a media.
- \sa media()
+ \sa currentMedia(), mediaChanged()
*/
/*!
@@ -916,7 +1147,17 @@ QtMultimedia::AvailabilityError QMediaPlayer::availabilityError() const
information relating to the current media source and to cease all I/O operations related
to that media.
- \sa QMediaContent
+ \sa QMediaContent, currentMedia()
+*/
+
+/*!
+ \property QMediaPlayer::currentMedia
+ \brief the current active media content being played by the player object.
+ This value could be different from QMediaPlayer::media property if a playlist is used.
+ In this case currentMedia indicates the current media content being processed
+ by the player, while QMediaPlayer::media property contains the original playlist.
+
+ \sa QMediaContent, media()
*/
/*!
@@ -928,7 +1169,7 @@ QtMultimedia::AvailabilityError QMediaPlayer::availabilityError() const
By default this property is set to null.
- If the media playlist is used as a source, QMediaPlayer::media is updated with
+ If the media playlist is used as a source, QMediaPlayer::currentMedia is updated with
a current playlist item. The current source should be selected with
QMediaPlaylist::setCurrentIndex(int) instead of QMediaPlayer::setMedia(),
otherwise the current playlist will be discarded.
diff --git a/src/multimedia/playback/qmediaplayer.h b/src/multimedia/playback/qmediaplayer.h
index eed09b566..b6d154bc2 100644
--- a/src/multimedia/playback/qmediaplayer.h
+++ b/src/multimedia/playback/qmediaplayer.h
@@ -65,6 +65,7 @@ class Q_MULTIMEDIA_EXPORT QMediaPlayer : public QMediaObject
{
Q_OBJECT
Q_PROPERTY(QMediaContent media READ media WRITE setMedia NOTIFY mediaChanged)
+ Q_PROPERTY(QMediaContent currentMedia READ currentMedia NOTIFY currentMediaChanged)
Q_PROPERTY(QMediaPlaylist * playlist READ playlist WRITE setPlaylist)
Q_PROPERTY(qint64 duration READ duration NOTIFY durationChanged)
Q_PROPERTY(qint64 position READ position WRITE setPosition NOTIFY positionChanged)
@@ -118,7 +119,8 @@ public:
FormatError,
NetworkError,
AccessDeniedError,
- ServiceMissingError
+ ServiceMissingError,
+ MediaIsPlaylist
};
QMediaPlayer(QObject *parent = 0, Flags flags = 0);
@@ -136,6 +138,7 @@ public:
QMediaContent media() const;
const QIODevice *mediaStream() const;
QMediaPlaylist *playlist() const;
+ QMediaContent currentMedia() const;
State state() const;
MediaStatus mediaStatus() const;
@@ -178,6 +181,7 @@ public Q_SLOTS:
Q_SIGNALS:
void mediaChanged(const QMediaContent &media);
+ void currentMediaChanged(const QMediaContent &media);
void stateChanged(QMediaPlayer::State newState);
void mediaStatusChanged(QMediaPlayer::MediaStatus status);
@@ -210,6 +214,8 @@ private:
Q_PRIVATE_SLOT(d_func(), void _q_error(int, const QString &))
Q_PRIVATE_SLOT(d_func(), void _q_updateMedia(const QMediaContent&))
Q_PRIVATE_SLOT(d_func(), void _q_playlistDestroyed())
+ Q_PRIVATE_SLOT(d_func(), void _q_handlePlaylistLoaded())
+ Q_PRIVATE_SLOT(d_func(), void _q_handlePlaylistLoadFailed())
};
QT_END_NAMESPACE
diff --git a/src/multimedia/playback/qmediaplaylist.cpp b/src/multimedia/playback/qmediaplaylist.cpp
index 860fca2cc..8c2cd3112 100644
--- a/src/multimedia/playback/qmediaplaylist.cpp
+++ b/src/multimedia/playback/qmediaplaylist.cpp
@@ -42,8 +42,8 @@
#include "qmediaplaylist.h"
#include "qmediaplaylist_p.h"
#include "qmediaplaylistprovider_p.h"
-#include "qlocalmediaplaylistprovider_p.h"
#include "qmediaplaylistioplugin_p.h"
+#include "qmedianetworkplaylistprovider_p.h"
#include "qmediaservice.h"
#include "qmediaplaylistcontrol_p.h"
#include "qmediaplayercontrol.h"
@@ -129,7 +129,7 @@ QMediaPlaylist::QMediaPlaylist(QObject *parent)
Q_D(QMediaPlaylist);
d->q_ptr = this;
- d->localPlaylistControl = new QLocalMediaPlaylistControl(this);
+ d->networkPlaylistControl = new QMediaNetworkPlaylistControl(this);
setMediaObject(0);
}
@@ -178,7 +178,7 @@ bool QMediaPlaylist::setMediaObject(QMediaObject *mediaObject)
newControl = qobject_cast<QMediaPlaylistControl*>(service->requestControl(QMediaPlaylistControl_iid));
if (!newControl)
- newControl = d->localPlaylistControl;
+ newControl = d->networkPlaylistControl;
if (d->control != newControl) {
int oldSize = 0;
diff --git a/src/multimedia/playback/qmediaplaylist_p.h b/src/multimedia/playback/qmediaplaylist_p.h
index 7d718da2a..9a1555184 100644
--- a/src/multimedia/playback/qmediaplaylist_p.h
+++ b/src/multimedia/playback/qmediaplaylist_p.h
@@ -57,7 +57,7 @@
#include "qmediaplaylistcontrol_p.h"
#include "qmediaplayer.h"
#include "qmediaplayercontrol.h"
-#include "qlocalmediaplaylistprovider_p.h"
+#include "qmedianetworkplaylistprovider_p.h"
#include "qmediaobject_p.h"
#include <QtCore/qdebug.h>
@@ -86,7 +86,7 @@ public:
QMediaPlaylistPrivate()
:mediaObject(0),
control(0),
- localPlaylistControl(0),
+ networkPlaylistControl(0),
error(QMediaPlaylist::NoError)
{
}
@@ -105,7 +105,7 @@ public:
{
Q_Q(QMediaPlaylist);
mediaObject = 0;
- if (control != localPlaylistControl)
+ if (control != networkPlaylistControl)
control = 0;
q->setMediaObject(0);
}
@@ -115,7 +115,7 @@ public:
QMediaPlaylistControl *control;
QMediaPlaylistProvider *playlist() const { return control->playlistProvider(); }
- QMediaPlaylistControl *localPlaylistControl;
+ QMediaPlaylistControl *networkPlaylistControl;
bool readItems(QMediaPlaylistReader *reader);
bool writeItems(QMediaPlaylistWriter *writer);
@@ -127,14 +127,14 @@ public:
};
-class QLocalMediaPlaylistControl : public QMediaPlaylistControl
+class QMediaNetworkPlaylistControl : public QMediaPlaylistControl
{
Q_OBJECT
public:
- QLocalMediaPlaylistControl(QObject *parent)
+ QMediaNetworkPlaylistControl(QObject *parent)
:QMediaPlaylistControl(parent)
{
- QMediaPlaylistProvider *playlist = new QLocalMediaPlaylistProvider(this);
+ QMediaPlaylistProvider *playlist = new QMediaNetworkPlaylistProvider(this);
m_navigator = new QMediaPlaylistNavigator(playlist,this);
m_navigator->setPlaybackMode(QMediaPlaylist::Sequential);
@@ -143,7 +143,7 @@ public:
connect(m_navigator, SIGNAL(playbackModeChanged(QMediaPlaylist::PlaybackMode)), SIGNAL(playbackModeChanged(QMediaPlaylist::PlaybackMode)));
}
- virtual ~QLocalMediaPlaylistControl() {};
+ virtual ~QMediaNetworkPlaylistControl() {};
QMediaPlaylistProvider* playlistProvider() const { return m_navigator->playlist(); }
bool setPlaylistProvider(QMediaPlaylistProvider *mediaPlaylist)
diff --git a/tests/auto/unit/qmediaplayer/tst_qmediaplayer.cpp b/tests/auto/unit/qmediaplayer/tst_qmediaplayer.cpp
index f2bcdc10e..a3aedc197 100644
--- a/tests/auto/unit/qmediaplayer/tst_qmediaplayer.cpp
+++ b/tests/auto/unit/qmediaplayer/tst_qmediaplayer.cpp
@@ -227,11 +227,12 @@ void tst_QMediaPlayer::testNullService()
{
QFETCH_GLOBAL(QMediaContent, mediaContent);
- QSignalSpy spy(&player, SIGNAL(mediaChanged(QMediaContent)));
+ QSignalSpy spy(&player, SIGNAL(currentMediaChanged(QMediaContent)));
QFile file;
player.setMedia(mediaContent, &file);
- QCOMPARE(player.media(), QMediaContent());
+ QCOMPARE(player.currentMedia(), QMediaContent());
+ QCOMPARE(player.media(), mediaContent);
QCOMPARE(player.mediaStream(), nullDevice);
QCOMPARE(spy.count(), 0);
} {
@@ -297,13 +298,15 @@ void tst_QMediaPlayer::testNullService()
playlist.setCurrentIndex(0);
QCOMPARE(playlist.currentIndex(), 0);
- QCOMPARE(player.media(), QMediaContent());
+ QCOMPARE(player.currentMedia(), QMediaContent());
+ QCOMPARE(player.media().playlist(), &playlist);
QCOMPARE(mediaSpy.count(), 0);
QCOMPARE(statusSpy.count(), 0);
playlist.next();
QCOMPARE(playlist.currentIndex(), 1);
- QCOMPARE(player.media(), QMediaContent());
+ QCOMPARE(player.currentMedia(), QMediaContent());
+ QCOMPARE(player.media().playlist(), &playlist);
QCOMPARE(mediaSpy.count(), 0);
QCOMPARE(statusSpy.count(), 0);
}
@@ -324,11 +327,11 @@ void tst_QMediaPlayer::testMedia()
QFETCH_GLOBAL(QMediaContent, mediaContent);
mockService->setMedia(mediaContent);
- QCOMPARE(player->media(), mediaContent);
+ QCOMPARE(player->currentMedia(), mediaContent);
QBuffer stream;
player->setMedia(mediaContent, &stream);
- QCOMPARE(player->media(), mediaContent);
+ QCOMPARE(player->currentMedia(), mediaContent);
QCOMPARE((QBuffer*)player->mediaStream(), &stream);
}
@@ -525,7 +528,7 @@ void tst_QMediaPlayer::testPlay()
mockService->setState(state);
mockService->setMedia(mediaContent);
QVERIFY(player->state() == state);
- QVERIFY(player->media() == mediaContent);
+ QVERIFY(player->currentMedia() == mediaContent);
QSignalSpy spy(player, SIGNAL(stateChanged(QMediaPlayer::State)));
@@ -551,7 +554,7 @@ void tst_QMediaPlayer::testPause()
mockService->setState(state);
mockService->setMedia(mediaContent);
QVERIFY(player->state() == state);
- QVERIFY(player->media() == mediaContent);
+ QVERIFY(player->currentMedia() == mediaContent);
QSignalSpy spy(player, SIGNAL(stateChanged(QMediaPlayer::State)));
@@ -575,7 +578,7 @@ void tst_QMediaPlayer::testStop()
mockService->setState(state);
mockService->setMedia(mediaContent);
QVERIFY(player->state() == state);
- QVERIFY(player->media() == mediaContent);
+ QVERIFY(player->currentMedia() == mediaContent);
QSignalSpy spy(player, SIGNAL(stateChanged(QMediaPlayer::State)));
@@ -686,14 +689,15 @@ void tst_QMediaPlayer::testPlaylist()
QMediaPlaylist *playlist = new QMediaPlaylist;
player->setPlaylist(playlist);
+ QCOMPARE(player->media().playlist(), playlist);
QSignalSpy stateSpy(player, SIGNAL(stateChanged(QMediaPlayer::State)));
- QSignalSpy mediaSpy(player, SIGNAL(mediaChanged(QMediaContent)));
+ QSignalSpy mediaSpy(player, SIGNAL(currentMediaChanged(QMediaContent)));
// Test the player does nothing with an empty playlist attached.
player->play();
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
- QCOMPARE(player->media(), QMediaContent());
+ QCOMPARE(player->currentMedia(), QMediaContent());
QCOMPARE(stateSpy.count(), 0);
QCOMPARE(mediaSpy.count(), 0);
@@ -704,28 +708,28 @@ void tst_QMediaPlayer::testPlaylist()
// Test changing the playlist position, changes the current media, but not the playing state.
playlist->setCurrentIndex(1);
- QCOMPARE(player->media(), content1);
+ QCOMPARE(player->currentMedia(), content1);
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
QCOMPARE(stateSpy.count(), 0);
QCOMPARE(mediaSpy.count(), 1);
// Test playing starts with the current media.
player->play();
- QCOMPARE(player->media(), content1);
+ QCOMPARE(player->currentMedia(), content1);
QCOMPARE(player->state(), QMediaPlayer::PlayingState);
QCOMPARE(stateSpy.count(), 1);
QCOMPARE(mediaSpy.count(), 1);
// Test pausing doesn't change the current media.
player->pause();
- QCOMPARE(player->media(), content1);
+ QCOMPARE(player->currentMedia(), content1);
QCOMPARE(player->state(), QMediaPlayer::PausedState);
QCOMPARE(stateSpy.count(), 2);
QCOMPARE(mediaSpy.count(), 1);
// Test stopping doesn't change the current media.
player->stop();
- QCOMPARE(player->media(), content1);
+ QCOMPARE(player->currentMedia(), content1);
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
QCOMPARE(stateSpy.count(), 3);
QCOMPARE(mediaSpy.count(), 1);
@@ -733,20 +737,20 @@ void tst_QMediaPlayer::testPlaylist()
// Test when the player service reaches the end of the current media, the player moves onto
// the next item without stopping.
player->play();
- QCOMPARE(player->media(), content1);
+ QCOMPARE(player->currentMedia(), content1);
QCOMPARE(player->state(), QMediaPlayer::PlayingState);
QCOMPARE(stateSpy.count(), 4);
QCOMPARE(mediaSpy.count(), 1);
mockService->setState(QMediaPlayer::StoppedState, QMediaPlayer::EndOfMedia);
- QCOMPARE(player->media(), content2);
+ QCOMPARE(player->currentMedia(), content2);
QCOMPARE(player->state(), QMediaPlayer::PlayingState);
QCOMPARE(stateSpy.count(), 4);
QCOMPARE(mediaSpy.count(), 2);
// Test skipping the current media doesn't change the state.
playlist->next();
- QCOMPARE(player->media(), content3);
+ QCOMPARE(player->currentMedia(), content3);
QCOMPARE(player->state(), QMediaPlayer::PlayingState);
QCOMPARE(stateSpy.count(), 4);
QCOMPARE(mediaSpy.count(), 3);
@@ -754,13 +758,13 @@ void tst_QMediaPlayer::testPlaylist()
// Test changing the current media while paused doesn't change the state.
player->pause();
mockService->setMediaStatus(QMediaPlayer::BufferedMedia);
- QCOMPARE(player->media(), content3);
+ QCOMPARE(player->currentMedia(), content3);
QCOMPARE(player->state(), QMediaPlayer::PausedState);
QCOMPARE(stateSpy.count(), 5);
QCOMPARE(mediaSpy.count(), 3);
playlist->previous();
- QCOMPARE(player->media(), content2);
+ QCOMPARE(player->currentMedia(), content2);
QCOMPARE(player->state(), QMediaPlayer::PausedState);
QCOMPARE(stateSpy.count(), 5);
QCOMPARE(mediaSpy.count(), 4);
@@ -768,13 +772,13 @@ void tst_QMediaPlayer::testPlaylist()
// Test changing the current media while stopped doesn't change the state.
player->stop();
mockService->setMediaStatus(QMediaPlayer::LoadedMedia);
- QCOMPARE(player->media(), content2);
+ QCOMPARE(player->currentMedia(), content2);
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
QCOMPARE(stateSpy.count(), 6);
QCOMPARE(mediaSpy.count(), 4);
playlist->next();
- QCOMPARE(player->media(), content3);
+ QCOMPARE(player->currentMedia(), content3);
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
QCOMPARE(stateSpy.count(), 6);
QCOMPARE(mediaSpy.count(), 5);
@@ -782,7 +786,7 @@ void tst_QMediaPlayer::testPlaylist()
// Test the player is stopped and the current media cleared when it reaches the end of the last
// item in the playlist.
player->play();
- QCOMPARE(player->media(), content3);
+ QCOMPARE(player->currentMedia(), content3);
QCOMPARE(player->state(), QMediaPlayer::PlayingState);
QCOMPARE(stateSpy.count(), 7);
QCOMPARE(mediaSpy.count(), 5);
@@ -790,45 +794,46 @@ void tst_QMediaPlayer::testPlaylist()
// Double up the signals to ensure some noise doesn't destabalize things.
mockService->setState(QMediaPlayer::StoppedState, QMediaPlayer::EndOfMedia);
mockService->setState(QMediaPlayer::StoppedState, QMediaPlayer::EndOfMedia);
- QCOMPARE(player->media(), QMediaContent());
+ QCOMPARE(player->currentMedia(), QMediaContent());
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
QCOMPARE(stateSpy.count(), 8);
QCOMPARE(mediaSpy.count(), 6);
// Test starts playing from the start of the playlist if there is no current media selected.
player->play();
- QCOMPARE(player->media(), content0);
+ QCOMPARE(player->currentMedia(), content0);
QCOMPARE(player->state(), QMediaPlayer::PlayingState);
QCOMPARE(stateSpy.count(), 9);
- QCOMPARE(mediaSpy.count(), 7);
+ // one notification is for playlist and another is for the first media in the playlist
+ QCOMPARE(mediaSpy.count(), 8);
// Test deleting the playlist stops the player and clears the media it set.
delete playlist;
- QCOMPARE(player->media(), QMediaContent());
+ QCOMPARE(player->currentMedia(), QMediaContent());
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
QCOMPARE(stateSpy.count(), 10);
- QCOMPARE(mediaSpy.count(), 8);
+ QCOMPARE(mediaSpy.count(), 9);
// Test the player works as normal with the playlist removed.
player->play();
- QCOMPARE(player->media(), QMediaContent());
+ QCOMPARE(player->currentMedia(), QMediaContent());
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
QCOMPARE(stateSpy.count(), 10);
- QCOMPARE(mediaSpy.count(), 8);
+ QCOMPARE(mediaSpy.count(), 9);
player->setMedia(content1);
player->play();
- QCOMPARE(player->media(), content1);
+ QCOMPARE(player->currentMedia(), content1);
QCOMPARE(player->state(), QMediaPlayer::PlayingState);
QCOMPARE(stateSpy.count(), 11);
- QCOMPARE(mediaSpy.count(), 9);
+ QCOMPARE(mediaSpy.count(), 10);
// Test the player can bind to playlist again
playlist = new QMediaPlaylist;
player->setPlaylist(playlist);
- QCOMPARE(player->media(), QMediaContent());
+ QCOMPARE(player->currentMedia(), QMediaContent());
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
playlist->addMedia(content0);
@@ -837,7 +842,7 @@ void tst_QMediaPlayer::testPlaylist()
playlist->addMedia(content3);
playlist->setCurrentIndex(1);
- QCOMPARE(player->media(), content1);
+ QCOMPARE(player->currentMedia(), content1);
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
// Test attaching the new playlist,
@@ -850,11 +855,11 @@ void tst_QMediaPlayer::testPlaylist()
player->play();
player->setPlaylist(playlist2);
- QCOMPARE(player->media(), playlist2->currentMedia());
+ QCOMPARE(player->currentMedia(), playlist2->currentMedia());
QCOMPARE(player->state(), QMediaPlayer::StoppedState);
playlist2->setCurrentIndex(1);
- QCOMPARE(player->media(), playlist2->currentMedia());
+ QCOMPARE(player->currentMedia(), playlist2->currentMedia());
{
QMediaPlaylist playlist;
@@ -863,13 +868,14 @@ void tst_QMediaPlayer::testPlaylist()
playlist.addMedia(content3);
playlist.setCurrentIndex(1);
+ // playlist resets to the first item
player->setPlaylist(&playlist);
QCOMPARE(player->playlist(), &playlist);
- QCOMPARE(player->media(), content2);
+ QCOMPARE(player->currentMedia(), content1);
} //playlist should be detached now
QVERIFY(player->playlist() == 0);
- QCOMPARE(player->media(), QMediaContent());
+ QCOMPARE(player->currentMedia(), QMediaContent());
// Test when the player service encounters an invalid media, the player moves onto
// the next item without stopping
@@ -877,6 +883,7 @@ void tst_QMediaPlayer::testPlaylist()
QSignalSpy ss(player, SIGNAL(stateChanged(QMediaPlayer::State)));
QSignalSpy ms(player, SIGNAL(mediaStatusChanged(QMediaPlayer::MediaStatus)));
+ // playlist index is set to 0 when it is loaded into media player
player->setPlaylist(playlist);
player->play();
QCOMPARE(ss.count(), 1);
@@ -888,7 +895,7 @@ void tst_QMediaPlayer::testPlaylist()
QCOMPARE(ms.count(), 1);
// NOTE: status should begin transitioning through to BufferedMedia.
- QCOMPARE(player->media(), content2);
+ QCOMPARE(player->currentMedia(), content1);
}
delete playlist;
diff --git a/tests/auto/unit/qmediaplaylistnavigator/tst_qmediaplaylistnavigator.cpp b/tests/auto/unit/qmediaplaylistnavigator/tst_qmediaplaylistnavigator.cpp
index 120a2f1e3..6fb1bc930 100644
--- a/tests/auto/unit/qmediaplaylistnavigator/tst_qmediaplaylistnavigator.cpp
+++ b/tests/auto/unit/qmediaplaylistnavigator/tst_qmediaplaylistnavigator.cpp
@@ -43,7 +43,7 @@
#include <QtTest/QtTest>
#include <QDebug>
-#include <private/qlocalmediaplaylistprovider_p.h>
+#include <private/qmedianetworkplaylistprovider_p.h>
#include <private/qmediaplaylistnavigator_p.h>
QT_USE_NAMESPACE
@@ -84,7 +84,7 @@ void tst_QMediaPlaylistNavigator::cleanup()
void tst_QMediaPlaylistNavigator::construction()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QCOMPARE(playlist.mediaCount(), 0);
QMediaPlaylistNavigator navigator(&playlist);
@@ -100,7 +100,7 @@ void tst_QMediaPlaylistNavigator::setPlaylist()
QCOMPARE(navigator.playlist()->media(0), QMediaContent());
QVERIFY(navigator.playlist()->isReadOnly() );
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QCOMPARE(playlist.mediaCount(), 0);
navigator.setPlaylist(&playlist);
@@ -111,7 +111,7 @@ void tst_QMediaPlaylistNavigator::setPlaylist()
void tst_QMediaPlaylistNavigator::linearPlayback()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Sequential);
@@ -165,7 +165,7 @@ void tst_QMediaPlaylistNavigator::linearPlayback()
void tst_QMediaPlaylistNavigator::loopPlayback()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Loop);
@@ -216,7 +216,7 @@ void tst_QMediaPlaylistNavigator::loopPlayback()
void tst_QMediaPlaylistNavigator::currentItemOnce()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::CurrentItemOnce);
@@ -247,7 +247,7 @@ void tst_QMediaPlaylistNavigator::currentItemOnce()
void tst_QMediaPlaylistNavigator::currentItemInLoop()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::CurrentItemInLoop);
@@ -275,7 +275,7 @@ void tst_QMediaPlaylistNavigator::currentItemInLoop()
void tst_QMediaPlaylistNavigator::randomPlayback()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Random);
@@ -326,7 +326,7 @@ void tst_QMediaPlaylistNavigator::randomPlayback()
void tst_QMediaPlaylistNavigator::testItemAt()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Random);
QCOMPARE(navigator.playbackMode(), QMediaPlaylist::Random);
@@ -348,7 +348,7 @@ void tst_QMediaPlaylistNavigator::testItemAt()
void tst_QMediaPlaylistNavigator::testNextIndex()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Random);
QCOMPARE(navigator.playbackMode(), QMediaPlaylist::Random);
@@ -383,7 +383,7 @@ void tst_QMediaPlaylistNavigator::testNextIndex()
void tst_QMediaPlaylistNavigator::testPreviousIndex()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Random);
QCOMPARE(navigator.playbackMode(), QMediaPlaylist::Random);
@@ -413,7 +413,7 @@ void tst_QMediaPlaylistNavigator::testPreviousIndex()
void tst_QMediaPlaylistNavigator::testCurrentIndexChangedSignal()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Random);
QCOMPARE(navigator.playbackMode(), QMediaPlaylist::Random);
@@ -441,7 +441,7 @@ void tst_QMediaPlaylistNavigator::testCurrentIndexChangedSignal()
void tst_QMediaPlaylistNavigator::testPlaybackModeChangedSignal()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Random);
QCOMPARE(navigator.playbackMode(), QMediaPlaylist::Random);
@@ -467,7 +467,7 @@ void tst_QMediaPlaylistNavigator::testPlaybackModeChangedSignal()
void tst_QMediaPlaylistNavigator::testSurroundingItemsChangedSignal()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Random);
QCOMPARE(navigator.playbackMode(), QMediaPlaylist::Random);
@@ -493,7 +493,7 @@ void tst_QMediaPlaylistNavigator::testSurroundingItemsChangedSignal()
void tst_QMediaPlaylistNavigator::testActivatedSignal()
{
- QLocalMediaPlaylistProvider playlist;
+ QMediaNetworkPlaylistProvider playlist;
QMediaPlaylistNavigator navigator(&playlist);
navigator.setPlaybackMode(QMediaPlaylist::Random);
QCOMPARE(navigator.playbackMode(), QMediaPlaylist::Random);