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

Commit 611efd59 authored by Liana Kazanova (xWF)'s avatar Liana Kazanova (xWF) Committed by Android (Google) Code Review
Browse files

Revert "audiopolicy: speaker cleanup AudioAttributes usage" 

Revert submission 30116824-spkr_cleanup

Reason for revert: DroidMonitor: Potential culprit for http://b/376600163 - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.
Reverted changes: /q/submissionid:30116824-spkr_cleanup

Change-Id: I08aa4728f3ad8f24d1cf0275c701c82e8c7bbdc0
parent d97c3ae9

media/audioaidlconversion/AidlConversionCppNdk.cpp

+0 −4
Original line number Diff line number Diff line
@@ -1797,8 +1797,6 @@ aidl2legacy_AudioUsage_audio_usage_t(AudioUsage aidl) { 
            return AUDIO_USAGE_VEHICLE_STATUS;

        case AudioUsage::ANNOUNCEMENT:

            return AUDIO_USAGE_ANNOUNCEMENT;

        case AudioUsage::SPEAKER_CLEANUP:

            return AUDIO_USAGE_SPEAKER_CLEANUP;

    }

    return unexpected(BAD_VALUE);

}
@@ -1850,8 +1848,6 @@ legacy2aidl_audio_usage_t_AudioUsage(audio_usage_t legacy) { 
            return AudioUsage::VEHICLE_STATUS;

        case AUDIO_USAGE_ANNOUNCEMENT:

            return AudioUsage::ANNOUNCEMENT;

        case AUDIO_USAGE_SPEAKER_CLEANUP:

            return AudioUsage::SPEAKER_CLEANUP;

    }

    return unexpected(BAD_VALUE);

}

services/audiopolicy/engine/common/src/EngineDefaultConfig.h

+0 −2
Original line number Diff line number Diff line
@@ -131,8 +131,6 @@ const engineConfig::ProductStrategies gOrderedStrategies = { 
              {AUDIO_CONTENT_TYPE_UNKNOWN, AUDIO_USAGE_UNKNOWN, AUDIO_SOURCE_DEFAULT,

                AUDIO_FLAG_BEACON, ""},

              {AUDIO_CONTENT_TYPE_ULTRASOUND, AUDIO_USAGE_UNKNOWN, AUDIO_SOURCE_DEFAULT,

                AUDIO_FLAG_NONE, ""},

              {AUDIO_CONTENT_TYPE_UNKNOWN, AUDIO_USAGE_SPEAKER_CLEANUP, AUDIO_SOURCE_DEFAULT,

                AUDIO_FLAG_NONE, ""}

          }

         }

services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp

+0 −11
Original line number Diff line number Diff line
@@ -425,17 +425,6 @@ Status AudioPolicyService::getOutputForAttr(const media::audio::common::AudioAtt 
        }

    }



    //TODO this permission check should extend to all system usages

    if (attr.usage == AUDIO_USAGE_SPEAKER_CLEANUP) {

        if (!(audioserver_permissions() ?

              CHECK_PERM(MODIFY_AUDIO_ROUTING, attributionSource.uid)

              : modifyAudioRoutingAllowed())) {

            ALOGE("%s: permission denied: SPEAKER_CLEANUP not allowed for uid %d pid %d",

                    __func__, attributionSource.uid, attributionSource.pid);

            return binderStatusFromStatusT(PERMISSION_DENIED);

        }

    }



    AutoCallerClear acc;

    AudioPolicyInterface::output_type_t outputType;

    bool isSpatialized = false;