From 629edc2b9549e2151c204c09abe9e8b989fbcddf Mon Sep 17 00:00:00 2001 From: tonihei Date: Mon, 12 Jun 2017 04:13:01 -0700 Subject: [PATCH] Remove needsContinueLoading from ExoPlayerImplInternal The same effect can be achieved by checking the isLoading variable of ExoPlayerImplInternal because this variable is in almost all cases set simultaneously with loadingMediaPeriodHolder.needsContinueLoading. ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=158697948 --- .../google/android/exoplayer2/ExoPlayerImplInternal.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/library/core/src/main/java/com/google/android/exoplayer2/ExoPlayerImplInternal.java b/library/core/src/main/java/com/google/android/exoplayer2/ExoPlayerImplInternal.java index fd30b673be..b93b31bdaa 100644 --- a/library/core/src/main/java/com/google/android/exoplayer2/ExoPlayerImplInternal.java +++ b/library/core/src/main/java/com/google/android/exoplayer2/ExoPlayerImplInternal.java @@ -1198,7 +1198,7 @@ import java.io.IOException; maybeUpdateLoadingPeriod(); if (loadingPeriodHolder == null || loadingPeriodHolder.isFullyBuffered()) { setIsLoading(false); - } else if (loadingPeriodHolder != null && loadingPeriodHolder.needsContinueLoading) { + } else if (loadingPeriodHolder != null && !isLoading) { maybeContinueLoading(); } @@ -1394,10 +1394,7 @@ import java.io.IOException; boolean continueLoading = loadControl.shouldContinueLoading(bufferedDurationUs); setIsLoading(continueLoading); if (continueLoading) { - loadingPeriodHolder.needsContinueLoading = false; loadingPeriodHolder.mediaPeriod.continueLoading(loadingPeriodPositionUs); - } else { - loadingPeriodHolder.needsContinueLoading = true; } } } @@ -1506,7 +1503,6 @@ import java.io.IOException; public boolean prepared; public boolean hasEnabledTracks; public MediaPeriodHolder next; - public boolean needsContinueLoading; public TrackSelectorResult trackSelectorResult; private final Renderer[] renderers;