Remove deprecated Player#hasPrevious() method.

Use Player#hasPreviousMediaItem() instead.

#cherrypick

PiperOrigin-RevId: 647336042
This commit is contained in:
ktrajkovski 2024-06-27 09:01:28 -07:00 committed by Copybara-Service
parent 3da63eeaa7
commit 47b1ca18ed
7 changed files with 2 additions and 55 deletions

View File

@ -57,6 +57,8 @@
* Use `HttpEngineDataSource` as the `HttpDataSource` when supported by the * Use `HttpEngineDataSource` as the `HttpDataSource` when supported by the
device. device.
* Remove deprecated symbols: * Remove deprecated symbols:
* Remove deprecated `Player#hasPrevious` method. Use
`Player#hasPreviousMediaItem()` instead.
## 1.4 ## 1.4

View File

@ -147,15 +147,6 @@ public abstract class BasePlayer implements Player {
seekToOffset(getSeekForwardIncrement(), Player.COMMAND_SEEK_FORWARD); seekToOffset(getSeekForwardIncrement(), Player.COMMAND_SEEK_FORWARD);
} }
/**
* @deprecated Use {@link #hasPreviousMediaItem()} instead.
*/
@Deprecated
@Override
public final boolean hasPrevious() {
return hasPreviousMediaItem();
}
/** /**
* @deprecated Use {@link #hasPreviousMediaItem()} instead. * @deprecated Use {@link #hasPreviousMediaItem()} instead.
*/ */

View File

@ -327,18 +327,6 @@ public class ForwardingPlayer implements Player {
player.seekForward(); player.seekForward();
} }
/**
* Calls {@link Player#hasPrevious()} on the delegate and returns the result.
*
* @deprecated Use {@link #hasPreviousMediaItem()} instead.
*/
@SuppressWarnings("deprecation") // Forwarding to deprecated method
@Deprecated
@Override
public boolean hasPrevious() {
return player.hasPrevious();
}
/** /**
* Calls {@link Player#hasPreviousWindow()} on the delegate and returns the result. * Calls {@link Player#hasPreviousWindow()} on the delegate and returns the result.
* *

View File

@ -2635,13 +2635,6 @@ public interface Player {
*/ */
void seekForward(); void seekForward();
/**
* @deprecated Use {@link #hasPreviousMediaItem()} instead.
*/
@UnstableApi
@Deprecated
boolean hasPrevious();
/** /**
* @deprecated Use {@link #hasPreviousMediaItem()} instead. * @deprecated Use {@link #hasPreviousMediaItem()} instead.
*/ */

View File

@ -1427,16 +1427,6 @@ public class MediaController implements Player {
return isConnected() ? impl.getNextMediaItemIndex() : C.INDEX_UNSET; return isConnected() ? impl.getNextMediaItemIndex() : C.INDEX_UNSET;
} }
/**
* @deprecated Use {@link #hasPreviousMediaItem()} instead.
*/
@UnstableApi
@Deprecated
@Override
public final boolean hasPrevious() {
return hasPreviousMediaItem();
}
/** /**
* @deprecated Use {@link #hasNextMediaItem()} instead. * @deprecated Use {@link #hasNextMediaItem()} instead.
*/ */

View File

@ -557,14 +557,6 @@ import java.util.List;
super.replaceMediaItems(fromIndex, toIndex, mediaItems); super.replaceMediaItems(fromIndex, toIndex, mediaItems);
} }
@SuppressWarnings("deprecation") // Forwarding deprecated call
@Deprecated
@Override
public boolean hasPrevious() {
verifyApplicationThread();
return super.hasPrevious();
}
@SuppressWarnings("deprecation") // Forwarding deprecated call @SuppressWarnings("deprecation") // Forwarding deprecated call
@Deprecated @Deprecated
@Override @Override

View File

@ -1080,15 +1080,6 @@ public class MockPlayer implements Player {
checkNotNull(conditionVariables.get(METHOD_REPLACE_MEDIA_ITEMS)).open(); checkNotNull(conditionVariables.get(METHOD_REPLACE_MEDIA_ITEMS)).open();
} }
/**
* @deprecated Use {@link #hasPreviousMediaItem()} instead.
*/
@Deprecated
@Override
public boolean hasPrevious() {
throw new UnsupportedOperationException();
}
/** /**
* @deprecated Use {@link #hasNextMediaItem()} instead. * @deprecated Use {@link #hasNextMediaItem()} instead.
*/ */