From fb9a5bfd0b8ea7551ac7eb01a5ed6ffcb1c86abb Mon Sep 17 00:00:00 2001 From: Assam Boudjelthia Date: Wed, 15 Sep 2021 19:45:50 +0300 Subject: Implement playback progress watcher Android's MediaPlayer doesn't seem to provide a listener for continuously reporting the current playback position, thus such a mechanism need to be implemented by us, otherwise progress is not being reported back to QMediaPlayer. This also overrides QPlatformMediaPlayer::isSeekable(). Fixes: QTBUG-96079 Pick-to: 6.2 6.2.0 Change-Id: I47ae0e2b9fe18b650fbbd0fb0266dc8a50dc3c55 Reviewed-by: Lars Knoll Reviewed-by: Samuel Mira --- .../android/multimedia/QtAndroidMediaPlayer.java | 36 ++++++++++++++++++++-- .../android/mediaplayer/qandroidmediaplayer.cpp | 5 +++ .../android/mediaplayer/qandroidmediaplayer_p.h | 2 ++ 3 files changed, 41 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/android/jar/src/org/qtproject/qt/android/multimedia/QtAndroidMediaPlayer.java b/src/android/jar/src/org/qtproject/qt/android/multimedia/QtAndroidMediaPlayer.java index f332c2db7..1ec250d47 100644 --- a/src/android/jar/src/org/qtproject/qt/android/multimedia/QtAndroidMediaPlayer.java +++ b/src/android/jar/src/org/qtproject/qt/android/multimedia/QtAndroidMediaPlayer.java @@ -57,6 +57,11 @@ import android.content.res.AssetFileDescriptor; import android.content.res.AssetManager; import android.view.SurfaceHolder; +import java.util.concurrent.Executors; +import java.util.concurrent.ScheduledExecutorService; +import java.util.concurrent.ScheduledFuture; +import java.util.concurrent.TimeUnit; + public class QtAndroidMediaPlayer { // Native callback functions for MediaPlayer @@ -81,6 +86,8 @@ public class QtAndroidMediaPlayer private int mVolume = 100; private static final String TAG = "Qt MediaPlayer"; private SurfaceHolder mSurfaceHolder = null; + private ScheduledExecutorService mProgressScheduler = null; + private ScheduledFuture mProgressWatcherHandle = null; private class State { final static int Uninitialized = 0x1 /* End */; @@ -280,6 +287,29 @@ public class QtAndroidMediaPlayer mMediaPlayer.setOnErrorListener(new MediaPlayerErrorListener()); mMediaPlayer.setOnPreparedListener(new MediaPlayerPreparedListener()); mMediaPlayer.setOnTimedTextListener(new MediaPlayerTimedTextListener()); + // Report playback position since there is no default listner for that in MediaPlayer + mProgressScheduler = Executors.newScheduledThreadPool(1); + } + + public void startProgressWatcher() + { + // if it was shutdown, request new thread + if (mProgressScheduler.isTerminated() || mProgressScheduler == null) + mProgressScheduler = Executors.newScheduledThreadPool(1); + + mProgressWatcherHandle = mProgressScheduler.scheduleAtFixedRate(new Runnable() { + @Override + public void run() { + if (isPlaying()) + onProgressUpdateNative(getCurrentPosition(), mID); + } + }, 10, 100, TimeUnit.MILLISECONDS); + } + + public void cancelProgressWatcher() + { + if (mProgressScheduler != null) + mProgressScheduler.shutdown(); } public void start() @@ -294,12 +324,12 @@ public class QtAndroidMediaPlayer try { mMediaPlayer.start(); setState(State.Started); + startProgressWatcher(); } catch (final IllegalStateException exception) { Log.w(TAG, exception); } } - public void pause() { if ((mState & (State.Started | State.Paused | State.PlaybackCompleted)) == 0) @@ -327,6 +357,7 @@ public class QtAndroidMediaPlayer try { mMediaPlayer.stop(); setState(State.Stopped); + cancelProgressWatcher(); } catch (final IllegalStateException exception) { Log.w(TAG, exception); } @@ -679,7 +710,6 @@ public class QtAndroidMediaPlayer return mMuted; } - public void reset() { if (mState == State.Uninitialized) { @@ -688,6 +718,7 @@ public class QtAndroidMediaPlayer mMediaPlayer.reset(); setState(State.Idle); + cancelProgressWatcher(); } public void release() @@ -699,6 +730,7 @@ public class QtAndroidMediaPlayer } setState(State.Uninitialized); + cancelProgressWatcher(); } public void setAudioAttributes(int type, int usage) diff --git a/src/multimedia/platform/android/mediaplayer/qandroidmediaplayer.cpp b/src/multimedia/platform/android/mediaplayer/qandroidmediaplayer.cpp index 373aa41ee..49ed8fbec 100644 --- a/src/multimedia/platform/android/mediaplayer/qandroidmediaplayer.cpp +++ b/src/multimedia/platform/android/mediaplayer/qandroidmediaplayer.cpp @@ -434,6 +434,11 @@ void QAndroidMediaPlayer::stop() mMediaPlayer->stop(); } +bool QAndroidMediaPlayer::isSeekable() const +{ + return true; +} + void QAndroidMediaPlayer::onInfo(qint32 what, qint32 extra) { StateChangeNotifier notifier(this); diff --git a/src/multimedia/platform/android/mediaplayer/qandroidmediaplayer_p.h b/src/multimedia/platform/android/mediaplayer/qandroidmediaplayer_p.h index f48f682d8..b8e187a08 100644 --- a/src/multimedia/platform/android/mediaplayer/qandroidmediaplayer_p.h +++ b/src/multimedia/platform/android/mediaplayer/qandroidmediaplayer_p.h @@ -96,6 +96,8 @@ public: void pause() override; void stop() override; + bool isSeekable() const override; + int trackCount(TrackType trackType) override; QMediaMetaData trackMetaData(TrackType trackType, int streamNumber) override; int activeTrack(TrackType trackType) override; -- cgit v1.2.3