Avoid calling audioTrack.play() on stopped AudioTrack

We already avoid calling pause() because it has no effect once
the AudioTrack is stopped. But still call play(), leaving the
platform AudioTrack in a strange perpetual playing state that
does not match our internal state tracking.

Also simplify the flow a bit by letting DefaultAudioSink decide
that by itself without going via return values from
AudioTrackPositionTracker.

PiperOrigin-RevId: 784190418

Author: tonihei

libraries/exoplayer/src/main/java/androidx/media3/exoplayer/audio/AudioTrackPositionTracker.java

@@ -450,22 +450,15 @@ public boolean hasPendingData(long writtenFrames) {
         || forceHasPendingData();
   }
 
-  /**
-   * Pauses the audio track position tracker, returning whether the audio track needs to be paused
-   * to cause playback to pause. If {@code false} is returned the audio track will pause without
-   * further interaction, as the end of stream has been handled.
-   */
-  public boolean pause() {
+  /** Pauses the audio track position tracker. */
+  public void pause() {
     resetSyncParams();
     if (stopTimestampUs == C.TIME_UNSET) {
       // The audio track is going to be paused, so reset the timestamp poller to ensure it doesn't
       // supply an advancing position.
       checkNotNull(audioTimestampPoller).reset();
-      return true;
     }
     stopPlaybackHeadPosition = getPlaybackHeadPosition();
-    // We've handled the end of the stream already, so there's no need to pause the track.
-    return false;
   }
 
   /**

libraries/exoplayer/src/main/java/androidx/media3/exoplayer/audio/DefaultAudioSink.java

@@ -935,7 +935,9 @@ public void play() {
     playing = true;
     if (isAudioTrackInitialized()) {
       audioTrackPositionTracker.start();
-      audioTrack.play();
+      if (!stoppedAudioTrack || isOffloadedPlayback(audioTrack)) {
+        audioTrack.play();
+      }
     }
   }
 
@@ -1599,9 +1601,11 @@ private void setVolumeInternal() {
   @Override
   public void pause() {
     playing = false;
-    if (isAudioTrackInitialized()
-        && (audioTrackPositionTracker.pause() || isOffloadedPlayback(audioTrack))) {
-      audioTrack.pause();
+    if (isAudioTrackInitialized()) {
+      audioTrackPositionTracker.pause();
+      if (!stoppedAudioTrack || isOffloadedPlayback(audioTrack)) {
+        audioTrack.pause();
+      }
     }
   }