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

Commit 831986d0 authored by Yuyang Huang's avatar Yuyang Huang Committed by Gerrit Code Review
Browse files

Merge "Not use AudioManager.setParameters when isScoManagedByAudio is enabled" into main

parents 44e05800 36a58ba4
Loading
Loading
Loading
Loading
+4 −0
Original line number Diff line number Diff line
@@ -1689,6 +1689,10 @@ public class HeadsetStateMachine extends StateMachine {
    }

    private void setAudioParameters() {
        if (Flags.isScoManagedByAudio()) {
            Log.i(TAG, "isScoManagedByAudio enabled, do not setAudioParameters");
            return;
        }
        AudioManager am = mSystemInterface.getAudioManager();
        Log.i(
                TAG,
+23 −0
Original line number Diff line number Diff line
@@ -1724,6 +1724,29 @@ public class HeadsetStateMachineTest {
        verifyAudioSystemSetParametersInvocation(false, false);
    }

    @Test
    public void testSetAudioParameters_isScoManagedByAudio() {
        mSetFlagsRule.enableFlags(Flags.FLAG_IS_SCO_MANAGED_BY_AUDIO);

        setUpConnectedState();
        mHeadsetStateMachine.sendMessage(
                HeadsetStateMachine.STACK_EVENT,
                new HeadsetStackEvent(
                        HeadsetStackEvent.EVENT_TYPE_SWB,
                        HeadsetHalConstants.BTHF_SWB_CODEC_LC3,
                        HeadsetHalConstants.BTHF_SWB_YES,
                        mTestDevice));

        mHeadsetStateMachine.sendMessage(
                HeadsetStateMachine.STACK_EVENT,
                new HeadsetStackEvent(
                        HeadsetStackEvent.EVENT_TYPE_AUDIO_STATE_CHANGED,
                        HeadsetHalConstants.AUDIO_STATE_CONNECTED,
                        mTestDevice));

        verify(mAudioManager, times(0)).setParameters(any());
    }

    /**
     * verify parameters given to audio system
     *