commit 9378eb9634d07010bebf8a2d4bda84320ec80e57
author jiabin <jiabin@google.com> Wed May 02 15:26:35 2018 -0700
committer jiabin <jiabin@google.com> Wed May 02 17:13:07 2018 -0700
tree 9d9dd22652b7860882e3155a6d218042d55cd3f6
parent 6f4512914d90430bf135de221897887638c79f25

Restrict setting silenced in RecordTrack.

When initializing, always set mSilenced as false. Never set silenced for
patch track.

Bug: 77193174
Test: make phone call with usb headset multiple times.
Change-Id: I93f4646cc6d88749c602bdd0003afc48f7dec36f

services/audioflinger/RecordTracks.h

diff --git a/services/audioflinger/RecordTracks.h b/services/audioflinger/RecordTracks.h
index 2b993ee..fc2dbbb 100644
--- a/services/audioflinger/RecordTracks.h
+++ b/services/audioflinger/RecordTracks.h
@@ -64,7 +64,7 @@
 
     virtual bool        isFastTrack() const { return (mFlags & AUDIO_INPUT_FLAG_FAST) != 0; }
 
-            void        setSilenced(bool silenced) { mSilenced = silenced; }
+            void        setSilenced(bool silenced) { if (!isPatchTrack()) mSilenced = silenced; }
             bool        isSilenced() const { return mSilenced; }
 
             status_t    getActiveMicrophones(std::vector<media::MicrophoneInfo>* activeMicrophones);

services/audioflinger/Tracks.cpp

diff --git a/services/audioflinger/Tracks.cpp b/services/audioflinger/Tracks.cpp
index aff1239..49552a1 100644
--- a/services/audioflinger/Tracks.cpp
+++ b/services/audioflinger/Tracks.cpp
@@ -1663,7 +1663,8 @@
         mFramesToDrop(0),
         mResamplerBufferProvider(NULL), // initialize in case of early constructor exit
         mRecordBufferConverter(NULL),
-        mFlags(flags)
+        mFlags(flags),
+        mSilenced(false)
 {
     if (mCblk == NULL) {
         return;