session: fix IllegalStateException when calling setSessionExtras()

nift4 · nift4 · commit d4b0217681e9 · 2025-03-24T19:16:33.000+01:00
...from thread which isn't application thread. This change makes the
demo app work with player using a non-main application thread.
diff --git a/libraries/session/src/main/java/androidx/media3/session/MediaSessionLegacyStub.java b/libraries/session/src/main/java/androidx/media3/session/MediaSessionLegacyStub.java
@@ -1134,7 +1134,7 @@ public void onSessionExtrasChanged(int seq, Bundle sessionExtras) {
       PlayerWrapper playerWrapper = sessionImpl.getPlayerWrapper();
       playerWrapper.setLegacyExtras(sessionExtras);
       sessionCompat.setExtras(playerWrapper.getLegacyExtras());
-      sessionCompat.setPlaybackState(sessionImpl.getPlayerWrapper().createPlaybackStateCompat());
+      updateLegacySessionPlaybackState(sessionImpl.getPlayerWrapper());
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -1134,7 +1134,7 @@ public void onSessionExtrasChanged(int seq, Bundle sessionExtras) {`
`1134`	`1134`	`PlayerWrapper playerWrapper = sessionImpl.getPlayerWrapper();`
`1135`	`1135`	`playerWrapper.setLegacyExtras(sessionExtras);`
`1136`	`1136`	`sessionCompat.setExtras(playerWrapper.getLegacyExtras());`
`1137`		`- sessionCompat.setPlaybackState(sessionImpl.getPlayerWrapper().createPlaybackStateCompat());`
	`1137`	`+ updateLegacySessionPlaybackState(sessionImpl.getPlayerWrapper());`
`1138`	`1138`	`}`
`1139`	`1139`
`1140`	`1140`	`@Override`