Commit ee2ad943 by bachinger Committed by Oliver Woodman

annotate nullabilty of CommandReceiver

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