commit f0d9a9d69d97890c5483935e8d888b67965fe279
author Jakub Pawlowski <jpawlowski@google.com> Mon Mar 12 17:00:37 2018 -0700
committer Jakub Pawlowski <jpawlowski@google.com> Mon Mar 12 17:14:49 2018 -0700
tree 587d6905aa56ec9ba8417bc00f14cdea21804d00
parent 479a612edeed4525675a3f2db4b0643e21e78b72

Disable log spam in AVRCP

Change-Id: If9f5cc8b4f6cb57bf9590e6aca7d80c86fbd4388

android/app/src/com/android/bluetooth/avrcp/Avrcp.java

diff --git a/android/app/src/com/android/bluetooth/avrcp/Avrcp.java b/android/app/src/com/android/bluetooth/avrcp/Avrcp.java
index 33bb1b2..7178f65 100644
--- a/android/app/src/com/android/bluetooth/avrcp/Avrcp.java
+++ b/android/app/src/com/android/bluetooth/avrcp/Avrcp.java
@@ -69,7 +69,7 @@
  ******************************************************************************/
 
 public final class Avrcp {
-    private static final boolean DEBUG = true;
+    private static final boolean DEBUG = false;
     private static final String TAG = "Avrcp";
     private static final String ABSOLUTE_VOLUME_BLACKLIST = "absolute_volume_blacklist";
 
@@ -1033,9 +1033,11 @@
             if (newState != null) {
                 newQueueId = newState.getActiveQueueItemId();
             }
-            Log.v(TAG,
-                    "Media update: id " + mLastQueueId + "➡" + newQueueId + "? " + currentAttributes
-                            .toRedactedString() + " : " + mMediaAttributes.toRedactedString());
+            if (DEBUG) {
+                Log.v(TAG,
+                        "Media update: id " + mLastQueueId + "➡" + newQueueId + "? " + currentAttributes
+                                .toRedactedString() + " : " + mMediaAttributes.toRedactedString());
+            }
 
             if (mAvailablePlayerViewChanged) {
                 registerNotificationRspAvalPlayerChangedNative(
@@ -1064,7 +1066,9 @@
             // Dont send now playing list changed if the player doesn't support browsing
             MediaPlayerInfo info = getAddressedPlayerInfo();
             if (info != null && info.isBrowseSupported()) {
-                Log.v(TAG, "Check if NowPlayingList is updated");
+                if (DEBUG) {
+                    Log.v(TAG, "Check if NowPlayingList is updated");
+                }
                 mAddressedMediaPlayer.updateNowPlayingList(mMediaController);
             }
 
@@ -1086,8 +1090,10 @@
         }
 
         // still send the updated play state if the playback state is none or buffering
-        Log.e(TAG, "play status change " + mReportedPlayStatus + "➡" + newPlayStatus
-                + " mPlayStatusChangedNT: " + mPlayStatusChangedNT);
+        if (DEBUG) {
+            Log.v(TAG, "play status change " + mReportedPlayStatus + "➡" + newPlayStatus
+                    + " mPlayStatusChangedNT: " + mPlayStatusChangedNT);
+        }
         if (mPlayStatusChangedNT == AvrcpConstants.NOTIFICATION_TYPE_INTERIM || (mReportedPlayStatus
                 != newPlayStatus)) {
             sendPlaybackStatus(AvrcpConstants.NOTIFICATION_TYPE_CHANGED, newPlayStatus);