Fix incorrect use of checkNotNull in CompositionPlayer

icbaker · copybara-github · commit 6cb0d7aefa4b · 2025-08-05T04:27:28.000-07:00
This was revealed by a compilation failure as part of migrating to
Guava's `Preconditions` methods.

PiperOrigin-RevId: 791150880
diff --git a/libraries/transformer/src/main/java/androidx/media3/transformer/CompositionPlayer.java b/libraries/transformer/src/main/java/androidx/media3/transformer/CompositionPlayer.java
@@ -1301,7 +1301,7 @@ private void repeatCompositionPlayback() {
   }
 
   private ImmutableList<MediaItemData> createPlaylist() {
-    checkNotNull(compositionDurationUs != C.TIME_UNSET);
+    checkState(compositionDurationUs != C.TIME_UNSET);
     return ImmutableList.of(
         new MediaItemData.Builder("CompositionTimeline")
             .setMediaItem(MediaItem.EMPTY)

Original file line number	Diff line number	Diff line change
`@@ -1301,7 +1301,7 @@ private void repeatCompositionPlayback() {`
`1301`	`1301`	`}`
`1302`	`1302`
`1303`	`1303`	`private ImmutableList<MediaItemData> createPlaylist() {`
`1304`		`- checkNotNull(compositionDurationUs != C.TIME_UNSET);`
	`1304`	`+ checkState(compositionDurationUs != C.TIME_UNSET);`
`1305`	`1305`	`return ImmutableList.of(`
`1306`	`1306`	`new MediaItemData.Builder("CompositionTimeline")`
`1307`	`1307`	`.setMediaItem(MediaItem.EMPTY)`