summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Eric Laurent <elaurent@google.com> 2011-04-07 09:41:46 -0700
committer Eric Laurent <elaurent@google.com> 2011-04-07 09:41:46 -0700
commitb8da6cb5b2aa6d2f46c4c49e96fcbaa1a05f21e5 (patch)
tree2fd3e3c4f3c84a8b3c41d5674e6d4d9e78778822
parentfcdebf88a483faa2f9bbde2b9a4093ca98b2ba6c (diff)
Do not disable effect in AudioEffect destructor.
Current implementation of AudioEffect class destructor disables the effect before disconnecting from IAudioEffect interface. This is problematic when more than one client has a handle on the same effect engine as destroying one handle will disable the effect which is not the intended behavior. Change-Id: I10eacf981506469a7ef4eb9a1650813f0848de5d
-rw-r--r--media/libmedia/AudioEffect.cpp1
1 files changed, 0 insertions, 1 deletions
diff --git a/media/libmedia/AudioEffect.cpp b/media/libmedia/AudioEffect.cpp
index aadeba54d7d6..a043329fa3f0 100644
--- a/media/libmedia/AudioEffect.cpp
+++ b/media/libmedia/AudioEffect.cpp
@@ -170,7 +170,6 @@ AudioEffect::~AudioEffect()
LOGV("Destructor %p", this);
if (mStatus == NO_ERROR || mStatus == ALREADY_EXISTS) {
- setEnabled(false);
if (mIEffect != NULL) {
mIEffect->disconnect();
mIEffect->asBinder()->unlinkToDeath(mIEffectClient);