Commit 2adec8d9 by bachinger Committed by Oliver Woodman

annotate nullabilty of CommandReceiver

Issue: #6938
PiperOrigin-RevId: 292415853
parent 52fb5f38
......@@ -178,8 +178,8 @@ public final class MediaSessionConnector {
Player player,
ControlDispatcher controlDispatcher,
String command,
Bundle extras,
ResultReceiver cb);
@Nullable Bundle extras,
@Nullable ResultReceiver cb);
}
/** Interface to which playback preparation and play actions are delegated. */
......@@ -394,6 +394,7 @@ public final class MediaSessionConnector {
* @param player The player connected to the media session.
* @return The custom action to be included in the session playback state or {@code null}.
*/
@Nullable
PlaybackStateCompat.CustomAction getCustomAction(Player player);
}
......@@ -1291,7 +1292,7 @@ public final class MediaSessionConnector {
}
@Override
public void onCommand(String command, Bundle extras, ResultReceiver cb) {
public void onCommand(String command, @Nullable Bundle extras, @Nullable ResultReceiver cb) {
if (player != null) {
for (int i = 0; i < commandReceivers.size(); i++) {
if (commandReceivers.get(i).onCommand(player, controlDispatcher, command, extras, cb)) {
......
......@@ -191,9 +191,9 @@ public final class TimelineQueueEditor
Player player,
ControlDispatcher controlDispatcher,
String command,
Bundle extras,
ResultReceiver cb) {
if (!COMMAND_MOVE_QUEUE_ITEM.equals(command)) {
@Nullable Bundle extras,
@Nullable ResultReceiver cb) {
if (!COMMAND_MOVE_QUEUE_ITEM.equals(command) || extras == null) {
return false;
}
int from = extras.getInt(EXTRA_FROM_INDEX, C.INDEX_UNSET);
......
......@@ -186,8 +186,8 @@ public abstract class TimelineQueueNavigator implements MediaSessionConnector.Qu
Player player,
ControlDispatcher controlDispatcher,
String command,
Bundle extras,
ResultReceiver cb) {
@Nullable Bundle extras,
@Nullable ResultReceiver cb) {
return false;
}
......
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