Rollback of 311f76d

PiperOrigin-RevId: 785381545
(cherry picked from commit 2345eab)

Author: tonihei

RELEASENOTES.md

@@ -17,6 +17,8 @@ This release includes the following changes since the
         use a different instance than the one used for playback
         ([#1191](https://github.com/androidx/media/issues/1191)).
 *   Session:
+    *   Fix bug where connections from third-party non-priviledged Media3
+        controllers are ignored.
     *   Remove check for available commands when sending custom commands to a
         legacy `MediaBrowserServiceCompat`. This is in parity with the behavior
         of legacy controllers/browsers when connected to a legacy app.

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

@@ -844,16 +844,6 @@ public void connect(
       int uid = Binder.getCallingUid();
       long token = Binder.clearCallingIdentity();
       int pid = (callingPid != 0) ? callingPid : request.pid;
-      if (!SessionUtil.isValidPackage(mediaSessionService, request.packageName, uid)) {
-        Log.w(
-            TAG,
-            "Ignoring connection from invalid package name "
-                + request.packageName
-                + " (uid="
-                + uid
-                + ")");
-        return;
-      }
       MediaSessionManager.RemoteUserInfo remoteUserInfo =
           new MediaSessionManager.RemoteUserInfo(request.packageName, pid, uid);
       boolean isTrusted =

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

@@ -655,26 +655,15 @@ public void connect(
     }
     int uid = Binder.getCallingUid();
     int callingPid = Binder.getCallingPid();
-    @Nullable MediaSessionImpl sessionImpl = this.sessionImpl.get();
-    if (sessionImpl != null
-        && !SessionUtil.isValidPackage(sessionImpl.getContext(), request.packageName, uid)) {
-      Log.w(
-          TAG,
-          "Ignoring connection from invalid package name "
-              + request.packageName
-              + " (uid="
-              + uid
-              + ")");
-      return;
-    }
-
     long token = Binder.clearCallingIdentity();
     // Binder.getCallingPid() can be 0 for an oneway call from the remote process.
     // If it's the case, use PID from the ConnectionRequest.
     int pid = (callingPid != 0) ? callingPid : request.pid;
     try {
+
       MediaSessionManager.RemoteUserInfo remoteUserInfo =
           new MediaSessionManager.RemoteUserInfo(request.packageName, pid, uid);
+      @Nullable MediaSessionImpl sessionImpl = this.sessionImpl.get();
       boolean isTrustedForMediaControl =
           sessionImpl != null
               && MediaSessionManager.getSessionManager(sessionImpl.getContext())