commit 0351ad9c0b2dffb38b5b36ff860122cd1628f74f
author Preetam Singh Ranawat <apranawat@codeaurora.org> Mon May 27 14:31:47 2019 +0530
committer Preetam Singh Ranawat <apranawat@codeaurora.org> Mon May 27 14:39:29 2019 +0530
tree 0f3b2d792ae063624b1d839b2ef338602accaf44
parent 919f4e8dc35a57e26104ebb2426514aea1a81b9a

policy_hal: fix CTS concurrent audio playback

 -fall back to non-direct output is not working for 3rd playback request
  when both compress offload and direct pcm are in use as only one flag is
  set to true due to break statement.
 -Remove break statement to set both flags to true when both compress offload
  and direct pcm are in use.

Change-Id: I60258ab8c6e3aa611268c087f2d0e02aabe51b41

policy_hal/AudioPolicyManager.cpp

diff --git a/policy_hal/AudioPolicyManager.cpp b/policy_hal/AudioPolicyManager.cpp
index 99f1a24..26064c9 100644
--- a/policy_hal/AudioPolicyManager.cpp
+++ b/policy_hal/AudioPolicyManager.cpp
@@ -1663,12 +1663,10 @@
             if (desc->mFlags == AUDIO_OUTPUT_FLAG_DIRECT) {
                 direct_pcm_already_in_use = true;
                 ALOGD("Direct PCM already in use");
-                break;
             }
             if (desc->mFlags & AUDIO_OUTPUT_FLAG_COMPRESS_OFFLOAD) {
                 compress_offload_already_in_use = true;
                 ALOGD("Compress Offload already in use");
-                break;
             }
         }
         // prevent direct pcm for non-music stream blindly if direct pcm already in use