commit 698d353587d6d0563a6451b2d701358e6cb4496e
author Francois Gaffie <francois.gaffie@renault.com> Wed Apr 10 15:20:05 2019 +0200
committer Eric Laurent <elaurent@google.com> Wed Apr 10 12:13:36 2019 -0700
tree f2bed9fc7ae60d23cc3df7e19a30af4c84af4c7a
parent d21b6de7181840e14300e94284db48172e24004b

audiopolicy: fix mix recorder regression

Engine is not only responsible of legacy remote submix for input
device selection (aka "0" adress) but also regular remote submix
with a given address. Filtering the remote submixes prevents engine
to select the remote submix for input stream in case a policy mix
rule has been set to record specific sources.

Bug: 130284799
Test: AudioPolicyEmulatorTests --gtest_filter=RecordInjectionSourceMatch/DynamicPolicyMixRecordInjectionTest

Change-Id: Iaafd1195b9ab5c82ad3bd2766061d982a10bb9da
Signed-off-by: Francois Gaffie <francois.gaffie@renault.com>

services/audiopolicy/managerdefault/AudioPolicyManager.h

diff --git a/services/audiopolicy/managerdefault/AudioPolicyManager.h b/services/audiopolicy/managerdefault/AudioPolicyManager.h
index a700868..de447fb 100644
--- a/services/audiopolicy/managerdefault/AudioPolicyManager.h
+++ b/services/audiopolicy/managerdefault/AudioPolicyManager.h
@@ -348,7 +348,8 @@
         }
         virtual const DeviceVector getAvailableInputDevices() const
         {
-            return mAvailableInputDevices.filterForEngine();
+            // legacy and non-legacy remote-submix are managed by the engine, do not filter
+            return mAvailableInputDevices;
         }
         virtual const sp<DeviceDescriptor> &getDefaultOutputDevice() const
         {