Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 664c45ba authored by Hyundo Moon's avatar Hyundo Moon
Browse files

MediaSession2Service: Log ControllerInfo in connection process 

Since printing a binder in a log doesn't help debugging, this CL
makes the code print ControllerInfo instead of the binder.

Bug: 130706459
Test: N/A
Change-Id: I225f2854d15f5b9e7e6a9dca113f9050bbc8dd43
parent cf44c411

media/apex/java/android/media/MediaSession2Service.java

+6 −5
Original line number Diff line number Diff line
@@ -370,10 +370,6 @@ public abstract class MediaSession2Service extends Service { 
                            }

                            return;

                        }

                        if (DEBUG) {

                            Log.d(TAG, "Handling incoming connection request from the"

                                    + " controller, controller=" + caller + ", uid=" + uid);

                        }



                        String callingPkg = connectionRequest.getString(KEY_PACKAGE_NAME);

                        // The Binder.getCallingPid() can be 0 for an oneway call from the
@@ -389,13 +385,18 @@ public abstract class MediaSession2Service extends Service { 
                                caller,

                                connectionRequest.getBundle(KEY_CONNECTION_HINTS));



                        if (DEBUG) {

                            Log.d(TAG, "Handling incoming connection request from the"

                                    + " controller=" + controllerInfo);

                        }



                        final MediaSession2 session;

                        session = service.onGetSession(controllerInfo);



                        if (session == null) {

                            if (DEBUG) {

                                Log.d(TAG, "Rejecting incoming connection request from the"

                                        + " controller, controller=" + caller + ", uid=" + uid);

                                        + " controller=" + controllerInfo);

                            }

                            // Note: Trusted controllers also can be rejected according to the

                            // service implementation.