Commit fc2cdef2 by andrewlewis Committed by Oliver Woodman

Don't return a TrackStream from TrackRenderer.disable.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=125659817
parent adc7ecec
...@@ -878,9 +878,8 @@ import java.util.ArrayList; ...@@ -878,9 +878,8 @@ import java.util.ArrayList;
rendererMediaClockSource = null; rendererMediaClockSource = null;
} }
ensureStopped(renderer); ensureStopped(renderer);
// TODO[playlists]: Consider using TrackStreams from the timeline source instead of renderer.disable();
// returning a TrackStream from TrackRenderer.disable. oldStreams.add(playingSource.trackStreams[i]);
oldStreams.add(renderer.disable());
} }
if (newSelection != null) { if (newSelection != null) {
newSelections.add(newSelection); newSelections.add(newSelection);
......
...@@ -314,14 +314,12 @@ public abstract class TrackRenderer implements ExoPlayerComponent { ...@@ -314,14 +314,12 @@ public abstract class TrackRenderer implements ExoPlayerComponent {
/** /**
* Disable the renderer. * Disable the renderer.
*/ */
/* package */ final TrackStream disable() { /* package */ final void disable() {
Assertions.checkState(state == STATE_ENABLED); Assertions.checkState(state == STATE_ENABLED);
state = STATE_DISABLED; state = STATE_DISABLED;
onDisabled(); onDisabled();
TrackStream trackStream = stream;
stream = null; stream = null;
streamIsFinal = false; streamIsFinal = false;
return trackStream;
} }
/** /**
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment