commit c2a3a82419327c6ffbad9711ed07415341eca00d
author Ricardo Garcia <rago@google.com> Wed Jul 17 14:29:12 2019 -0700
committer Eric Laurent <elaurent@google.com> Wed Jul 17 23:17:10 2019 +0000
tree 005a3560290a9682742136605f02b1250ccfbb30
parent d2d66abd1aa5fcb2fad9cc43e397015d5580f058

Fix for untimely suspension of Dynamics Processing

Added exception to avoid suspending Dynamics Processing Effect.

Test: Manual testing with popular apps
Bug: 134852575
Change-Id: I1ffee570984897304169b30b7af417f61f762ffa

services/audioflinger/Effects.cpp

diff --git a/services/audioflinger/Effects.cpp b/services/audioflinger/Effects.cpp
index 3c4fbba..13152d0 100644
--- a/services/audioflinger/Effects.cpp
+++ b/services/audioflinger/Effects.cpp
@@ -24,6 +24,7 @@
 #include "Configuration.h"
 #include <utils/Log.h>
 #include <system/audio_effects/effect_aec.h>
+#include <system/audio_effects/effect_dynamicsprocessing.h>
 #include <system/audio_effects/effect_ns.h>
 #include <system/audio_effects/effect_visualizer.h>
 #include <audio_utils/channels.h>
@@ -2569,7 +2570,8 @@
     if ((mSessionId == AUDIO_SESSION_OUTPUT_MIX) &&
         (((desc.flags & EFFECT_FLAG_TYPE_MASK) == EFFECT_FLAG_TYPE_AUXILIARY) ||
          (memcmp(&desc.type, SL_IID_VISUALIZATION, sizeof(effect_uuid_t)) == 0) ||
-         (memcmp(&desc.type, SL_IID_VOLUME, sizeof(effect_uuid_t)) == 0))) {
+         (memcmp(&desc.type, SL_IID_VOLUME, sizeof(effect_uuid_t)) == 0) ||
+         (memcmp(&desc.type, SL_IID_DYNAMICSPROCESSING, sizeof(effect_uuid_t)) == 0))) {
         return false;
     }
     return true;