commit 3ab499df5e8470a2d422fbd9bb0a0c0ee8d61a8c
author Jayant Chowdhary <jchowdhary@google.com> Wed Jun 15 15:43:33 2022 -0700
committer Jayant Chowdhary <jchowdhary@google.com> Wed Jun 15 23:43:08 2022 +0000
tree 996b7b6910e101c2150770c12ee68b9d65d871cc
parent 8199b3f7927b4fd52e713d80a30234ae97aab865

cameraserver: Reduce Camera2ClientBase destructor log confusion.

We log disconnect in CameraService::BasicClient::disconnect() and also
in ~Camera2ClientBase(). Since ~Camera2ClientBase() can often run a few
seconds after BasicClient::disconnect(depending on when we get another
binder call), the disconnect logs from the Camera2ClientBase destructor
can be confusing to look at on quick inspection.

Bug: 235352466

Test: Inspect logs

Change-Id: I3fbaa0bde2c21c3b13d5d86f7691eb49f9eb53a2
Signed-off-by: Jayant Chowdhary <jchowdhary@google.com>

services/camera/libcameraservice/common/Camera2ClientBase.cpp

diff --git a/services/camera/libcameraservice/common/Camera2ClientBase.cpp b/services/camera/libcameraservice/common/Camera2ClientBase.cpp
index ae22717..e54813f 100644
--- a/services/camera/libcameraservice/common/Camera2ClientBase.cpp
+++ b/services/camera/libcameraservice/common/Camera2ClientBase.cpp
@@ -168,8 +168,8 @@
         mCameraServiceWatchdog.clear();
     }
 
-    ALOGI("Closed Camera %s. Client was: %s (PID %d, UID %u)",
-            TClientBase::mCameraIdStr.string(),
+    ALOGI("%s: Client object's dtor for Camera Id %s completed. Client was: %s (PID %d, UID %u)",
+            __FUNCTION__, TClientBase::mCameraIdStr.string(),
             String8(TClientBase::mClientPackageName).string(),
             mInitialClientPid, TClientBase::mClientUid);
 }