Double tap detection for Bluetooth media button events only

Issue: androidx/media#233
#minor-release
PiperOrigin-RevId: 505078751
(cherry picked from commit 5c82d6bc)

RELEASENOTES.md

@@ -49,6 +49,8 @@
     *   Add `onSetMediaItems` callback listener to provide means to modify/set
     *   Add `onSetMediaItems` callback listener to provide means to modify/set
         `MediaItem` list, starting index and position by session before setting
         `MediaItem` list, starting index and position by session before setting
         onto Player ([#156](https://github.com/androidx/media/issues/156)).
         onto Player ([#156](https://github.com/androidx/media/issues/156)).
+    *   Avoid double tap detection for non-Bluetooth media button events
+        ([#233](https://github.com/androidx/media/issues/233)).
 *   Metadata:
 *   Metadata:
     *   Parse multiple null-separated values from ID3 frames, as permitted by
     *   Parse multiple null-separated values from ID3 frames, as permitted by
         ID3 v2.4.
         ID3 v2.4.

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

@@ -121,6 +121,7 @@ import org.checkerframework.checker.nullness.compatqual.NullableType;
   private final ConnectionTimeoutHandler connectionTimeoutHandler;
   private final ConnectionTimeoutHandler connectionTimeoutHandler;
   private final MediaPlayPauseKeyHandler mediaPlayPauseKeyHandler;
   private final MediaPlayPauseKeyHandler mediaPlayPauseKeyHandler;
   private final MediaSessionCompat sessionCompat;
   private final MediaSessionCompat sessionCompat;
+  private final String appPackageName;
   @Nullable private VolumeProviderCompat volumeProviderCompat;
   @Nullable private VolumeProviderCompat volumeProviderCompat;
 
 
   private volatile long connectionTimeoutMs;
   private volatile long connectionTimeoutMs;
@@ -133,6 +134,7 @@ import org.checkerframework.checker.nullness.compatqual.NullableType;
       Handler handler) {
       Handler handler) {
     sessionImpl = session;
     sessionImpl = session;
     Context context = sessionImpl.getContext();
     Context context = sessionImpl.getContext();
+    appPackageName = context.getPackageName();
     sessionManager = MediaSessionManager.getSessionManager(context);
     sessionManager = MediaSessionManager.getSessionManager(context);
     controllerLegacyCbForBroadcast = new ControllerLegacyCbForBroadcast();
     controllerLegacyCbForBroadcast = new ControllerLegacyCbForBroadcast();
     connectionTimeoutHandler =
     connectionTimeoutHandler =
@@ -225,7 +227,11 @@ import org.checkerframework.checker.nullness.compatqual.NullableType;
     switch (keyCode) {
     switch (keyCode) {
       case KeyEvent.KEYCODE_MEDIA_PLAY_PAUSE:
       case KeyEvent.KEYCODE_MEDIA_PLAY_PAUSE:
       case KeyEvent.KEYCODE_HEADSETHOOK:
       case KeyEvent.KEYCODE_HEADSETHOOK:
-        if (keyEvent.getRepeatCount() == 0) {
+        // Double tap detection only for media button events from external sources (for instance
+        // Bluetooth). Media button events from the app package are coming from the notification
+        // below targetApiLevel 33.
+        if (!appPackageName.equals(remoteUserInfo.getPackageName())
+            && keyEvent.getRepeatCount() == 0) {
           if (mediaPlayPauseKeyHandler.hasPendingMediaPlayPauseKey()) {
           if (mediaPlayPauseKeyHandler.hasPendingMediaPlayPauseKey()) {
             mediaPlayPauseKeyHandler.clearPendingMediaPlayPauseKey();
             mediaPlayPauseKeyHandler.clearPendingMediaPlayPauseKey();
             onSkipToNext();
             onSkipToNext();

libraries/test_session_current/src/androidTest/java/androidx/media3/session/MediaSessionKeyEventTest.java

@@ -237,15 +237,6 @@ public class MediaSessionKeyEventTest {
     player.awaitMethodCalled(MockPlayer.METHOD_PAUSE, TIMEOUT_MS);
     player.awaitMethodCalled(MockPlayer.METHOD_PAUSE, TIMEOUT_MS);
   }
   }
 
 
-  @Test
-  public void playPauseKeyEvent_doubleTapIsTranslatedToSkipToNext() throws Exception {
-    dispatchMediaKeyEvent(KeyEvent.KEYCODE_MEDIA_PLAY_PAUSE, true);
-
-    player.awaitMethodCalled(MockPlayer.METHOD_SEEK_TO_NEXT, TIMEOUT_MS);
-    assertThat(player.hasMethodBeenCalled(MockPlayer.METHOD_PLAY)).isFalse();
-    assertThat(player.hasMethodBeenCalled(MockPlayer.METHOD_PAUSE)).isFalse();
-  }
-
   private void dispatchMediaKeyEvent(int keyCode, boolean doubleTap) {
   private void dispatchMediaKeyEvent(int keyCode, boolean doubleTap) {
     audioManager.dispatchMediaKeyEvent(new KeyEvent(KeyEvent.ACTION_DOWN, keyCode));
     audioManager.dispatchMediaKeyEvent(new KeyEvent(KeyEvent.ACTION_DOWN, keyCode));
     audioManager.dispatchMediaKeyEvent(new KeyEvent(KeyEvent.ACTION_UP, keyCode));
     audioManager.dispatchMediaKeyEvent(new KeyEvent(KeyEvent.ACTION_UP, keyCode));