Skip to content

Commit 9a3c247

Browse files
toniheicopybara-github
authored andcommitted
Only set one of customLayout or mediaButtonPreferences
The custom layout is automatically derived from the button preferences (if non-empty), so there is no need to set and potentially transfer both. PiperOrigin-RevId: 828027599
1 parent 4cb577e commit 9a3c247

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

libraries/session/src/main/java/androidx/media3/session/MediaSessionLegacyStub.java

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -312,12 +312,16 @@ public void setAvailableCommands(
312312
* the platform session.
313313
*/
314314
public MediaSession.ConnectionResult getPlatformConnectionResult(MediaSession mediaSession) {
315-
return new MediaSession.ConnectionResult.AcceptedResultBuilder(mediaSession)
316-
.setAvailableSessionCommands(availableSessionCommands)
317-
.setAvailablePlayerCommands(availablePlayerCommands)
318-
.setCustomLayout(customLayout)
319-
.setMediaButtonPreferences(mediaButtonPreferences)
320-
.build();
315+
MediaSession.ConnectionResult.AcceptedResultBuilder result =
316+
new MediaSession.ConnectionResult.AcceptedResultBuilder(mediaSession)
317+
.setAvailableSessionCommands(availableSessionCommands)
318+
.setAvailablePlayerCommands(availablePlayerCommands);
319+
if (!mediaButtonPreferences.isEmpty()) {
320+
result.setMediaButtonPreferences(mediaButtonPreferences);
321+
} else {
322+
result.setCustomLayout(customLayout);
323+
}
324+
return result.build();
321325
}
322326

323327
/**

0 commit comments

Comments
 (0)