Commit 89cfa4df by olly Committed by Oliver Woodman

Remove some deprecated Player methods

PiperOrigin-RevId: 372981744
parent d282ad25
......@@ -3,6 +3,9 @@
### dev-v2 (not yet released)
* Remove deprecated symbols:
* Remove `Player.getPlaybackError`. Use `Player.getPlayerError` instead.
* Remove `Player.getCurrentTag`. Use `Player.getCurrentMediaItem` and
`MediaIteam.PlaybackProperties.tag` instead.
* Remove `PlaybackPreparer`. UI components that previously had
`setPlaybackPreparer` methods will now call `Player.prepare` by default.
If this behavior is sufficient, use of `PlaybackPreparer` can be removed
......
......@@ -86,14 +86,6 @@ public abstract class BasePlayer implements Player {
return getAvailableCommands().contains(command);
}
/** @deprecated Use {@link #getPlayerError()} instead. */
@Deprecated
@Override
@Nullable
public final ExoPlaybackException getPlaybackError() {
return getPlayerError();
}
@Override
public final void play() {
setPlayWhenReady(true);
......@@ -180,24 +172,6 @@ public abstract class BasePlayer implements Player {
getCurrentWindowIndex(), getRepeatModeForNavigation(), getShuffleModeEnabled());
}
/**
* @deprecated Use {@link #getCurrentMediaItem()} and {@link MediaItem.PlaybackProperties#tag}
* instead.
*/
@Deprecated
@Override
@Nullable
public final Object getCurrentTag() {
Timeline timeline = getCurrentTimeline();
if (timeline.isEmpty()) {
return null;
}
@Nullable
MediaItem.PlaybackProperties playbackProperties =
timeline.getWindow(getCurrentWindowIndex(), window).mediaItem.playbackProperties;
return playbackProperties != null ? playbackProperties.tag : null;
}
@Override
@Nullable
public final MediaItem getCurrentMediaItem() {
......
......@@ -1337,11 +1337,6 @@ public interface Player {
@Nullable
ExoPlaybackException getPlayerError();
/** @deprecated Use {@link #getPlayerError()} instead. */
@Deprecated
@Nullable
ExoPlaybackException getPlaybackError();
/**
* Resumes playback as soon as {@link #getPlaybackState()} == {@link #STATE_READY}. Equivalent to
* {@code setPlayWhenReady(true)}.
......@@ -1631,14 +1626,6 @@ public interface Player {
int getPreviousWindowIndex();
/**
* @deprecated Use {@link #getCurrentMediaItem()} and {@link MediaItem.PlaybackProperties#tag}
* instead.
*/
@Deprecated
@Nullable
Object getCurrentTag();
/**
* Returns the media item of the current window in the timeline. May be null if the timeline is
* empty.
*
......
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