commit 55fd81d7e1011565e894a4530eb8693c824f45b4
author Eric Laurent <elaurent@google.com> Thu Jul 16 12:28:30 2020 -0700
committer Hayden Gomes <haydengomes@google.com> Mon Jul 20 22:41:48 2020 +0000
tree d0e29c28a094661314467df63dc930386750507f
parent 480b74d285c11a0bf08222d2f2dd2d1d6c674f62

audio policy: fix concurrent capture policy for virtual sources

Do not consider virtual sources when determining
lastest active capture client. As a virtual source
capturing does not silence other captures, it should not
prevent another client from being selected as the latest active.

Bug: 160820363
Bug: 161725794
Test: Verify capture with Solotester in background and projecting to
DHU.

Change-Id: I2f28026dbecfa72b5d44f52b95538fc13996627b
Merged-In: I2f28026dbecfa72b5d44f52b95538fc13996627b
(cherry picked from commit 14a88630162b96d0d52354261ee09216c0bed0d7)

services/audiopolicy/service/AudioPolicyService.cpp

diff --git a/services/audiopolicy/service/AudioPolicyService.cpp b/services/audiopolicy/service/AudioPolicyService.cpp
index e847f9f..a6e8989 100644
--- a/services/audiopolicy/service/AudioPolicyService.cpp
+++ b/services/audiopolicy/service/AudioPolicyService.cpp
@@ -488,9 +488,9 @@
         }
 
         bool isAccessibility = mUidPolicy->isA11yUid(current->uid);
-        // Clients capturing for Accessibility services are not considered
+        // Clients capturing for Accessibility services or virtual sources are not considered
         // for top or latest active to avoid masking regular clients started before
-        if (!isAccessibility) {
+        if (!isAccessibility && !isVirtualSource(current->attributes.source)) {
             bool isAssistant = mUidPolicy->isAssistantUid(current->uid);
             bool isPrivacySensitive =
                     (current->attributes.flags & AUDIO_FLAG_CAPTURE_PRIVATE) != 0;