commit 8236964f884423258aeed51dfd81e24eadf7ca9a
author Hall Liu <hallliu@google.com> Fri May 18 11:46:55 2018 -0700
committer Hall Liu <hallliu@google.com> Fri May 18 11:46:55 2018 -0700
tree 1579ed918e7dfe09d921d290e8e07e1ecf70d368
parent 6b772309850722b0010f9280ac59ffbc14b75e9a

Remove volume logging

Remove volume logging from CARSM -- getStreamVolume is prone to long
waits, which screws up the inner workings of CARSM.

Change-Id: I43db6e39fbdc3bdce39926ae1ebfcba8b134798d
Fixes: 79563800
Test: manual

src/com/android/server/telecom/CallAudioRouteStateMachine.java

diff --git a/src/com/android/server/telecom/CallAudioRouteStateMachine.java b/src/com/android/server/telecom/CallAudioRouteStateMachine.java
index 8ea35be..9a17a3c 100644
--- a/src/com/android/server/telecom/CallAudioRouteStateMachine.java
+++ b/src/com/android/server/telecom/CallAudioRouteStateMachine.java
@@ -223,9 +223,7 @@
                     "Entering state " + getName());
             if (isActive()) {
                 Log.addEvent(mCallsManager.getForegroundCall(),
-                        AUDIO_ROUTE_TO_LOG_EVENT.get(getRouteCode(), LogUtils.Events.AUDIO_ROUTE),
-                        getVolumeString()
-                );
+                        AUDIO_ROUTE_TO_LOG_EVENT.get(getRouteCode(), LogUtils.Events.AUDIO_ROUTE));
             }
         }
 
@@ -313,20 +311,6 @@
         abstract public void updateSystemAudioState();
         abstract public boolean isActive();
         abstract public int getRouteCode();
-
-        private String getVolumeString() {
-            if (mAudioManager == null) {
-                return "";
-            }
-            StringBuilder sb = new StringBuilder();
-            sb.append("Volume: rng=");
-            sb.append(mAudioManager.getStreamVolume(AudioManager.STREAM_RING));
-            sb.append(", call=");
-            sb.append(mAudioManager.getStreamVolume(AudioManager.STREAM_VOICE_CALL));
-            sb.append(", bt=");
-            sb.append(mAudioManager.getStreamVolume(AudioManager.STREAM_BLUETOOTH_SCO));
-            return sb.toString();
-        }
     }
 
     class ActiveEarpieceRoute extends EarpieceRoute {