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

Commit c6af90d5 authored by Eric Laurent's avatar Eric Laurent
Browse files

audio policy: fix capture policy for HOTWORD source 

Do not consider Assistant capturing with HOTWORD source
when determining lastest active capture client. As assistant
capturing for HOTWORD does not silence other captures, it should not
prevent another client from being selected as the latest active.

Bug: 135806801
Test: Start capture with Recorder app, place in background and trigger
false OK G detection

Change-Id: Ic17bfd70675ef749d6a678d067112b1dbd205746
parent c2a3a824

services/audiopolicy/service/AudioPolicyService.cpp

+6 −2
Original line number Diff line number Diff line
@@ -459,16 +459,20 @@ void AudioPolicyService::updateUidStates_l() 
            continue;

        }



        bool isAssistant = mUidPolicy->isAssistantUid(current->uid);

        if (appState == APP_STATE_TOP) {

            if (current->startTimeNs > topStartNs) {

                topActive = current;

                topStartNs = current->startTimeNs;

            }

            if (mUidPolicy->isAssistantUid(current->uid)) {

            if (isAssistant) {

                isAssistantOnTop = true;

            }

        }

        if (current->startTimeNs > latestStartNs) {

        // Assistant capturing for HOTWORD not considered for latest active to avoid

        // masking regular clients started before

        if (current->startTimeNs > latestStartNs &&

            !(current->attributes.source == AUDIO_SOURCE_HOTWORD && isAssistant)) {

            latestActive = current;

            latestStartNs = current->startTimeNs;

        }