commit 015afd81734746fff1cdf57c29d20ca96ec06f96
author guochuang <guochuang@xiaomi.corp-partner.google.com> Tue Jul 04 10:45:51 2023 +0800
committer Chuang Guo <guochuang@xiaomi.corp-partner.google.com> Wed Sep 27 12:54:14 2023 +0000
tree 538a9cdecb84d70f6182c323620ee49a3fba3bc0
parent 9a04a729bab519a4fd49660681b2d978c32a98c7

Avoid "android.media.mediacodec.log-session-id" print redundancy

set "log-session-id" media metrics only if it is not empty.

Test: adb shell dumpsys media.metrics | grep codec
Change-Id: I6c2fc61c587feb5d9dbb431aaa10157cb629866c
Signed-off-by: guochuang <guochuang@xiaomi.corp-partner.google.com>

media/libstagefright/MediaCodec.cpp

diff --git a/media/libstagefright/MediaCodec.cpp b/media/libstagefright/MediaCodec.cpp
index 5c9f68e..e7b0c35 100644
--- a/media/libstagefright/MediaCodec.cpp
+++ b/media/libstagefright/MediaCodec.cpp
@@ -2005,7 +2005,9 @@
         mediametrics_setInt32(nextMetricsHandle, kCodecEncoder,
                               (flags & CONFIGURE_FLAG_ENCODE) ? 1 : 0);
 
-        mediametrics_setCString(nextMetricsHandle, kCodecLogSessionId, mLogSessionId.c_str());
+        if (!mLogSessionId.empty()) {
+            mediametrics_setCString(nextMetricsHandle, kCodecLogSessionId, mLogSessionId.c_str());
+        }
 
         // moved here from ::init()
         mediametrics_setCString(nextMetricsHandle, kCodecCodec, mInitName.c_str());