From 8c63ba6fdbfcf632538acbfb6737f227923e54bf Mon Sep 17 00:00:00 2001 From: ibaker Date: Fri, 10 Jun 2022 14:32:54 +0000 Subject: [PATCH] Remove ExoPlayerTest.PlayerStateGrabber It's no longer needed PiperOrigin-RevId: 454161070 --- .../media3/exoplayer/ExoPlayerTest.java | 25 +++++++------------ 1 file changed, 9 insertions(+), 16 deletions(-) diff --git a/libraries/exoplayer/src/test/java/androidx/media3/exoplayer/ExoPlayerTest.java b/libraries/exoplayer/src/test/java/androidx/media3/exoplayer/ExoPlayerTest.java index 784f6c23df..c41f46941a 100644 --- a/libraries/exoplayer/src/test/java/androidx/media3/exoplayer/ExoPlayerTest.java +++ b/libraries/exoplayer/src/test/java/androidx/media3/exoplayer/ExoPlayerTest.java @@ -3900,13 +3900,19 @@ public final class ExoPlayerTest { ShadowAudioManager shadowAudioManager = shadowOf(context.getSystemService(AudioManager.class)); shadowAudioManager.setNextFocusRequestResponse(AudioManager.AUDIOFOCUS_REQUEST_FAILED); - PlayerStateGrabber playerStateGrabber = new PlayerStateGrabber(); + AtomicBoolean playWhenReady = new AtomicBoolean(); ActionSchedule actionSchedule = new ActionSchedule.Builder(TAG) .setAudioAttributes(AudioAttributes.DEFAULT, /* handleAudioFocus= */ true) .play() .waitForPlaybackState(Player.STATE_READY) - .executeRunnable(playerStateGrabber) + .executeRunnable( + new PlayerRunnable() { + @Override + public void run(ExoPlayer player) { + playWhenReady.set(player.getPlayWhenReady()); + } + }) .build(); AtomicBoolean seenPlaybackSuppression = new AtomicBoolean(); Player.Listener listener = @@ -3924,7 +3930,7 @@ public final class ExoPlayerTest { .start() .blockUntilActionScheduleFinished(TIMEOUT_MS); - assertThat(playerStateGrabber.playWhenReady).isFalse(); + assertThat(playWhenReady.get()).isFalse(); assertThat(seenPlaybackSuppression.get()).isFalse(); } @@ -12224,19 +12230,6 @@ public final class ExoPlayerTest { } } - private static final class PlayerStateGrabber extends PlayerRunnable { - - public boolean playWhenReady; - public @Player.State int playbackState; - @Nullable public Timeline timeline; - - @Override - public void run(ExoPlayer player) { - playWhenReady = player.getPlayWhenReady(); - playbackState = player.getPlaybackState(); - timeline = player.getCurrentTimeline(); - } - } /** * Provides a wrapper for a {@link Runnable} which does collect playback states and window counts. * Can be used with {@link ActionSchedule.Builder#executeRunnable(Runnable)} to verify that a