Browse Source

Merge pull request #10181 from nextcloud/videoFullscreenPause

Correctly obey play/pause state
Tobias Kaminsky 3 years ago
parent
commit
8d4ed0a862

+ 4 - 2
app/src/main/java/com/owncloud/android/ui/preview/PreviewMediaFragment.java

@@ -506,7 +506,10 @@ public class PreviewMediaFragment extends FileFragment implements OnTouchListene
         if (savedPlaybackPosition >= 0) {
             exoPlayer.seekTo(savedPlaybackPosition);
         }
-        exoPlayer.play();
+
+        if (autoplay) {
+            exoPlayer.play();
+        }
     }
 
     @Override
@@ -575,7 +578,6 @@ public class PreviewMediaFragment extends FileFragment implements OnTouchListene
     @Override
     public void onResume() {
         super.onResume();
-        autoplay = false;
         if(getActivity() instanceof FileDisplayActivity){
             ((FileDisplayActivity) getActivity()).configureToolbarForMediaPreview(getFile());
         }

+ 7 - 1
app/src/main/java/com/owncloud/android/ui/preview/PreviewVideoActivity.kt

@@ -114,6 +114,8 @@ class PreviewVideoActivity :
         if (mSavedPlaybackPosition >= 0) {
             exoPlayer?.seekTo(mSavedPlaybackPosition)
         }
+
+        onIsPlayingChanged(exoPlayer!!.isPlaying)
     }
 
     override fun onIsPlayingChanged(isPlaying: Boolean) {
@@ -200,7 +202,10 @@ class PreviewVideoActivity :
     private fun play(item: MediaItem) {
         exoPlayer?.addMediaItem(item)
         exoPlayer?.prepare()
-        exoPlayer?.play()
+
+        if (mAutoplay) {
+            exoPlayer?.play()
+        }
     }
 
     override fun onStart() {
@@ -220,6 +225,7 @@ class PreviewVideoActivity :
                     else -> MediaItem.fromUri(mStreamUri!!)
                 }
                 if (exoPlayer != null) {
+                    setupPlayerView()
                     play(mediaItem)
                 } else {
                     val context = this