Fix playback of short duration content

Issue #1837 ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=133944552
google · Sep 28, 2016 · 57a2749 · 57a2749
1 parent e1ab1e3
commit 57a2749
Showing 1 changed file with 13 additions and 11 deletions.
diff --git a/library/src/main/java/com/google/android/exoplayer2/ExoPlayerImplInternal.java b/library/src/main/java/com/google/android/exoplayer2/ExoPlayerImplInternal.java
@@ -458,12 +458,11 @@ private void doSomeWork() throws ExoPlaybackException, IOException {
           startRenderers();
         }
       }
-    } else if (state == ExoPlayer.STATE_READY) {
-      if (enabledRenderers.length > 0 ? !allRenderersReadyOrEnded : !isTimelineReady) {
-        rebuffering = playWhenReady;
-        setState(ExoPlayer.STATE_BUFFERING);
-        stopRenderers();
-      }
+    } else if (state == ExoPlayer.STATE_READY
+        && (enabledRenderers.length > 0 ? !allRenderersReadyOrEnded : !isTimelineReady)) {
+      rebuffering = playWhenReady;
+      setState(ExoPlayer.STATE_BUFFERING);
+      stopRenderers();
     }
 
     if (state == ExoPlayer.STATE_BUFFERING) {
@@ -995,10 +994,18 @@ private void updatePeriods() throws ExoPlaybackException, IOException {
       eventHandler.obtainMessage(MSG_POSITION_DISCONTINUITY, playbackInfo).sendToTarget();
     }
     updateTimelineState();
+
+    if (readingPeriodHolder != null && readingPeriodHolder.isLast) {
+      readingPeriodHolder = null;
+      for (Renderer renderer : enabledRenderers) {
+        renderer.setCurrentStreamIsFinal();
+      }
+    }
     if (readingPeriodHolder == null) {
       // The renderers have their final SampleStreams.
       return;
     }
+
     for (Renderer renderer : enabledRenderers) {
       if (!renderer.hasReadStreamToEnd()) {
         return;
@@ -1029,11 +1036,6 @@ private void updatePeriods() throws ExoPlaybackException, IOException {
           }
         }
       }
-    } else if (readingPeriodHolder.isLast) {
-      readingPeriodHolder = null;
-      for (Renderer renderer : enabledRenderers) {
-        renderer.setCurrentStreamIsFinal();
-      }
     }
   }