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

Commit 0814db7b authored by Hall Liu's avatar Hall Liu
Browse files

Reset speakerphone at beginning and end of calls 

Always call AudioManager#setSpeakerphoneOn(false) at the beginning and
end of calls.

Bug: 69305994
Bug: 70338906
Test: unit
Change-Id: Ifcd887ca7bdeab8cff8a2773a306548142979238
Merged-In: Ifcd887ca7bdeab8cff8a2773a306548142979238
parent ae4cd3df

src/com/android/server/telecom/CallAudioRouteStateMachine.java

+4 −5
Original line number Diff line number Diff line
@@ -1293,12 +1293,10 @@ public class CallAudioRouteStateMachine extends StateMachine { 
    }



    private void setSpeakerphoneOn(boolean on) {

        if (mAudioManager.isSpeakerphoneOn() != on) {

        Log.i(this, "turning speaker phone %s", on);

        mAudioManager.setSpeakerphoneOn(on);

        mStatusBarNotifier.notifySpeakerphone(on);

    }

    }



    private void setBluetoothOn(boolean on) {

        if (mBluetoothRouteManager.isBluetoothAvailable()) {
@@ -1504,6 +1502,7 @@ public class CallAudioRouteStateMachine extends StateMachine { 
        mDeviceSupportedRoutes = initState.getSupportedRouteMask();

        mAvailableRoutes = mDeviceSupportedRoutes & getCurrentCallSupportedRoutes();

        mIsMuted = initState.isMuted();

        setSpeakerphoneOn(initState.getRoute() == CallAudioState.ROUTE_SPEAKER);

        setMuteOn(mIsMuted);

        mWasOnSpeaker = false;

        mHasUserExplicitlyLeftBluetooth = false;

tests/src/com/android/server/telecom/tests/CallAudioRouteStateMachineTest.java

+27 −18
Original line number Diff line number Diff line
@@ -68,6 +68,7 @@ public class CallAudioRouteStateMachineTest 
    private static final int NONE = 0;

    private static final int ON = 1;

    private static final int OFF = 2;

    private static final int OPTIONAL = 3;



    static class RoutingTestParameters extends TestParameters {

        public String name;
@@ -405,7 +406,7 @@ public class CallAudioRouteStateMachineTest 
                "Connect headset during earpiece", // name

                CallAudioState.ROUTE_EARPIECE, // initialRoute

                CallAudioState.ROUTE_EARPIECE, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                NONE, // bluetoothInteraction

                CallAudioRouteStateMachine.CONNECT_WIRED_HEADSET, // action

                CallAudioState.ROUTE_WIRED_HEADSET, // expectedRoute
@@ -418,7 +419,7 @@ public class CallAudioRouteStateMachineTest 
                "Connect headset during bluetooth", // name

                CallAudioState.ROUTE_BLUETOOTH, // initialRoute

                CallAudioState.ROUTE_EARPIECE | CallAudioState.ROUTE_BLUETOOTH, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                OFF, // bluetoothInteraction

                CallAudioRouteStateMachine.CONNECT_WIRED_HEADSET, // action

                CallAudioState.ROUTE_WIRED_HEADSET, // expectedRoute
@@ -444,7 +445,7 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect headset during headset", // name

                CallAudioState.ROUTE_WIRED_HEADSET, // initialRoute

                CallAudioState.ROUTE_WIRED_HEADSET, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                NONE, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_WIRED_HEADSET, // action

                CallAudioState.ROUTE_EARPIECE, // expectedRoute
@@ -457,8 +458,8 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect headset during headset with bluetooth available", // name

                CallAudioState.ROUTE_WIRED_HEADSET, // initialRoute

                CallAudioState.ROUTE_WIRED_HEADSET | CallAudioState.ROUTE_BLUETOOTH, // availableRou

                NONE, // speakerInteraction

                NONE, // bluetoothInteraction

                OPTIONAL, // speakerInteraction

                ON, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_WIRED_HEADSET, // action

                CallAudioState.ROUTE_BLUETOOTH, // expectedRoute

                CallAudioState.ROUTE_EARPIECE | CallAudioState.ROUTE_BLUETOOTH, // expectedAvailable
@@ -470,7 +471,7 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect headset during bluetooth", // name

                CallAudioState.ROUTE_BLUETOOTH, // initialRoute

                CallAudioState.ROUTE_WIRED_HEADSET | CallAudioState.ROUTE_BLUETOOTH, // availableRou

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                NONE, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_WIRED_HEADSET, // action

                CallAudioState.ROUTE_BLUETOOTH, // expectedRoute
@@ -483,7 +484,7 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect headset during speakerphone", // name

                CallAudioState.ROUTE_SPEAKER, // initialRoute

                CallAudioState.ROUTE_WIRED_HEADSET, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                NONE, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_WIRED_HEADSET, // action

                CallAudioState.ROUTE_SPEAKER, // expectedRoute
@@ -496,7 +497,7 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect headset during speakerphone with bluetooth available", // name

                CallAudioState.ROUTE_SPEAKER, // initialRoute

                CallAudioState.ROUTE_WIRED_HEADSET | CallAudioState.ROUTE_BLUETOOTH, // availableRou

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                NONE, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_WIRED_HEADSET, // action

                CallAudioState.ROUTE_SPEAKER, // expectedRoute
@@ -509,7 +510,7 @@ public class CallAudioRouteStateMachineTest 
                "Connect bluetooth during earpiece", // name

                CallAudioState.ROUTE_EARPIECE, // initialRoute

                CallAudioState.ROUTE_EARPIECE, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                ON, // bluetoothInteraction

                CallAudioRouteStateMachine.CONNECT_BLUETOOTH, // action

                CallAudioState.ROUTE_BLUETOOTH, // expectedRoute
@@ -522,7 +523,7 @@ public class CallAudioRouteStateMachineTest 
                "Connect bluetooth during wired headset", // name

                CallAudioState.ROUTE_WIRED_HEADSET, // initialRoute

                CallAudioState.ROUTE_WIRED_HEADSET, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                ON, // bluetoothInteraction

                CallAudioRouteStateMachine.CONNECT_BLUETOOTH, // action

                CallAudioState.ROUTE_BLUETOOTH, // expectedRoute
@@ -548,7 +549,7 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect bluetooth during bluetooth without headset in", // name

                CallAudioState.ROUTE_BLUETOOTH, // initialRoute

                CallAudioState.ROUTE_EARPIECE | CallAudioState.ROUTE_BLUETOOTH, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                OFF, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_BLUETOOTH, // action

                CallAudioState.ROUTE_EARPIECE, // expectedRoute
@@ -561,7 +562,7 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect bluetooth during bluetooth without headset in, priority mode ", // name

                CallAudioState.ROUTE_BLUETOOTH, // initialRoute

                CallAudioState.ROUTE_EARPIECE | CallAudioState.ROUTE_BLUETOOTH, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                OFF, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_BLUETOOTH, // action

                CallAudioState.ROUTE_EARPIECE, // expectedRoute
@@ -574,7 +575,7 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect bluetooth during bluetooth with headset in", // name

                CallAudioState.ROUTE_BLUETOOTH, // initialRoute

                CallAudioState.ROUTE_WIRED_HEADSET | CallAudioState.ROUTE_BLUETOOTH, // availableRou

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                OFF, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_BLUETOOTH, // action

                CallAudioState.ROUTE_WIRED_HEADSET, // expectedRoute
@@ -587,7 +588,7 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect bluetooth during speakerphone", // name

                CallAudioState.ROUTE_SPEAKER, // initialRoute

                CallAudioState.ROUTE_WIRED_HEADSET | CallAudioState.ROUTE_BLUETOOTH, // availableRou

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                NONE, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_BLUETOOTH, // action

                CallAudioState.ROUTE_SPEAKER, // expectedRoute
@@ -600,7 +601,7 @@ public class CallAudioRouteStateMachineTest 
                "Disconnect bluetooth during earpiece", // name

                CallAudioState.ROUTE_EARPIECE, // initialRoute

                CallAudioState.ROUTE_EARPIECE | CallAudioState.ROUTE_BLUETOOTH, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                NONE, // bluetoothInteraction

                CallAudioRouteStateMachine.DISCONNECT_BLUETOOTH, // action

                CallAudioState.ROUTE_EARPIECE, // expectedRoute
@@ -652,7 +653,7 @@ public class CallAudioRouteStateMachineTest 
                "Switch to earpiece from bluetooth", // name

                CallAudioState.ROUTE_BLUETOOTH, // initialRoute

                CallAudioState.ROUTE_EARPIECE | CallAudioState.ROUTE_BLUETOOTH, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                OFF, // bluetoothInteraction

                CallAudioRouteStateMachine.SWITCH_EARPIECE, // action

                CallAudioState.ROUTE_EARPIECE, // expectedRoute
@@ -717,7 +718,7 @@ public class CallAudioRouteStateMachineTest 
                "Switch to bluetooth from earpiece", // name

                CallAudioState.ROUTE_EARPIECE, // initialRoute

                CallAudioState.ROUTE_EARPIECE | CallAudioState.ROUTE_BLUETOOTH, // availableRoutes

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                ON, // bluetoothInteraction

                CallAudioRouteStateMachine.SWITCH_BLUETOOTH, // action

                CallAudioState.ROUTE_BLUETOOTH, // expectedRoute
@@ -730,7 +731,7 @@ public class CallAudioRouteStateMachineTest 
                "Switch to bluetooth from wired headset", // name

                CallAudioState.ROUTE_WIRED_HEADSET, // initialRoute

                CallAudioState.ROUTE_WIRED_HEADSET | CallAudioState.ROUTE_BLUETOOTH, // availableRou

                NONE, // speakerInteraction

                OPTIONAL, // speakerInteraction

                ON, // bluetoothInteraction

                CallAudioRouteStateMachine.SWITCH_BLUETOOTH, // action

                CallAudioState.ROUTE_BLUETOOTH, // expectedRoute
@@ -855,6 +856,10 @@ public class CallAudioRouteStateMachineTest 
            case OFF:

                verify(mockBluetoothRouteManager, never()).connectBluetoothAudio(null);

                verify(mockBluetoothRouteManager).disconnectBluetoothAudio();

                break;

            case OPTIONAL:

                // optional, don't test

                break;

        }



        switch (params.speakerInteraction) {
@@ -864,6 +869,10 @@ public class CallAudioRouteStateMachineTest 
            case ON: // fall through

            case OFF:

                verify(mockAudioManager).setSpeakerphoneOn(params.speakerInteraction == ON);

                break;

            case OPTIONAL:

                // optional, don't test

                break;

        }



        // Verify the end state