Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e0f65a2c authored by Eric Laurent's avatar Eric Laurent Committed by Automerger Merge Worker
Browse files

Merge "Audio policy: fix failure to unmute strategy." into rvc-dev am: 6b72bc88 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/av/+/11845532

Change-Id: I99d763e5149e7d0ebfd61690caca1ae9499225b6
parents 776e2b54 6b72bc88

services/audiopolicy/managerdefault/AudioPolicyManager.cpp

+11 −9
Original line number Diff line number Diff line
@@ -5770,15 +5770,6 @@ uint32_t AudioPolicyManager::setOutputDevices(const sp<SwAudioOutputDescriptor>& 
    DeviceVector filteredDevices = outputDesc->filterSupportedDevices(devices);

    DeviceVector prevDevices = outputDesc->devices();



    // no need to proceed if new device is not AUDIO_DEVICE_NONE and not supported by current

    // output profile or if new device is not supported AND previous device(s) is(are) still

    // available (otherwise reset device must be done on the output)

    if (!devices.isEmpty() && filteredDevices.isEmpty() &&

            !mAvailableOutputDevices.filter(prevDevices).empty()) {

        ALOGV("%s: unsupported device %s for output", __func__, devices.toString().c_str());

        return 0;

    }



    ALOGV("setOutputDevices() prevDevice %s", prevDevices.toString().c_str());



    if (!filteredDevices.isEmpty()) {
@@ -5793,6 +5784,17 @@ uint32_t AudioPolicyManager::setOutputDevices(const sp<SwAudioOutputDescriptor>& 
        muteWaitMs = 0;

    }



    // no need to proceed if new device is not AUDIO_DEVICE_NONE and not supported by current

    // output profile or if new device is not supported AND previous device(s) is(are) still

    // available (otherwise reset device must be done on the output)

    if (!devices.isEmpty() && filteredDevices.isEmpty() &&

            !mAvailableOutputDevices.filter(prevDevices).empty()) {

        ALOGV("%s: unsupported device %s for output", __func__, devices.toString().c_str());

        // restore previous device after evaluating strategy mute state

        outputDesc->setDevices(prevDevices);

        return muteWaitMs;

    }



    // Do not change the routing if:

    //      the requested device is AUDIO_DEVICE_NONE

    //      OR the requested device is the same as current device