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

Commit 36a58ba4 authored by Yuyang Huang's avatar Yuyang Huang
Browse files

Not use AudioManager.setParameters when isScoManagedByAudio is enabled

Bug: 312592568
Bug: 315234036
Test: m .
Change-Id: I83f23006347ad7f94b3abe59c26c1a437784dda8
parent ff1d0a2c
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
     *