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

Commit 83adcb4b authored by Tyler Gunn's avatar Tyler Gunn Committed by Automerger Merge Worker
Browse files

Merge "Correct issue where ringtone is not heard on BT headset in vibrate... 

Merge "Correct issue where ringtone is not heard on BT headset in vibrate mode." into sc-dev am: 7ef7e2bb

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/services/Telecomm/+/14737712

Change-Id: I8dad4f151c239bbf16ab072fcb421d62e0ecca48
parents 8fcb4956 7ef7e2bb

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

+11 −0
Original line number Diff line number Diff line
@@ -1347,6 +1347,15 @@ public class CallAudioRouteStateMachine extends StateMachine { 
                    } else {

                        sendInternalMessage(MUTE_EXTERNALLY_CHANGED);

                    }

                } else if (AudioManager.STREAM_MUTE_CHANGED_ACTION.equals(intent.getAction())) {

                    int streamType = intent.getIntExtra(AudioManager.EXTRA_VOLUME_STREAM_TYPE, -1);

                    boolean isStreamMuted = intent.getBooleanExtra(

                            AudioManager.EXTRA_STREAM_VOLUME_MUTED, false);



                    if (streamType == AudioManager.STREAM_RING && !isStreamMuted) {

                        Log.i(this, "Ring stream was un-muted.");

                        mCallAudioManager.onRingerModeChange();

                    }

                } else {

                    Log.w(this, "Received non-mute-change intent");

                }
@@ -1530,6 +1539,8 @@ public class CallAudioRouteStateMachine extends StateMachine { 
        mWasOnSpeaker = false;

        mContext.registerReceiver(mMuteChangeReceiver,

                new IntentFilter(AudioManager.ACTION_MICROPHONE_MUTE_CHANGED));

        mContext.registerReceiver(mMuteChangeReceiver,

                new IntentFilter(AudioManager.STREAM_MUTE_CHANGED_ACTION));

        mContext.registerReceiver(mSpeakerPhoneChangeReceiver,

                new IntentFilter(AudioManager.ACTION_SPEAKERPHONE_STATE_CHANGED));

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

+46 −0
Original line number Diff line number Diff line
@@ -17,7 +17,10 @@ 
package com.android.server.telecom.tests;



import android.bluetooth.BluetoothDevice;

import android.content.BroadcastReceiver;

import android.content.Context;

import android.content.Intent;

import android.content.IntentFilter;

import android.media.AudioManager;

import android.media.IAudioService;

import android.os.HandlerThread;
@@ -149,6 +152,49 @@ public class CallAudioRouteStateMachineTest extends TelecomTestCase { 
        assertNotNull(stateMachine);

    }



    @MediumTest

    @Test

    public void testStreamRingMuteChange() {

        CallAudioRouteStateMachine stateMachine = new CallAudioRouteStateMachine(

                mContext,

                mockCallsManager,

                mockBluetoothRouteManager,

                mockWiredHeadsetManager,

                mockStatusBarNotifier,

                mAudioServiceFactory,

                CallAudioRouteStateMachine.EARPIECE_FORCE_ENABLED,

                mThreadHandler.getLooper());

        stateMachine.setCallAudioManager(mockCallAudioManager);

        CallAudioState initState = new CallAudioState(false, CallAudioState.ROUTE_SPEAKER,

                CallAudioState.ROUTE_EARPIECE | CallAudioState.ROUTE_SPEAKER);

        stateMachine.initialize(initState);



        // Make sure we register a receiver for the STREAM_MUTE_CHANGED_ACTION so we can see if the

        // ring stream unmutes.

        ArgumentCaptor<BroadcastReceiver> brCaptor = ArgumentCaptor.forClass(

                BroadcastReceiver.class);

        ArgumentCaptor<IntentFilter> filterCaptor = ArgumentCaptor.forClass(IntentFilter.class);

        verify(mContext, times(3)).registerReceiver(brCaptor.capture(), filterCaptor.capture());

        boolean foundValid = false;

        for (int ix = 0; ix < brCaptor.getAllValues().size(); ix++) {

            BroadcastReceiver receiver = brCaptor.getAllValues().get(ix);

            IntentFilter filter = filterCaptor.getAllValues().get(ix);

            if (!filter.hasAction(AudioManager.STREAM_MUTE_CHANGED_ACTION)) {

                continue;

            }



            // Fake out a call to the broadcast receiver and make sure we call into audio manager

            // to trigger re-evaluation of ringing.

            Intent intent = new Intent(AudioManager.STREAM_MUTE_CHANGED_ACTION);

            intent.putExtra(AudioManager.EXTRA_STREAM_VOLUME_MUTED, false);

            intent.putExtra(AudioManager.EXTRA_VOLUME_STREAM_TYPE, AudioManager.STREAM_RING);

            receiver.onReceive(mContext, intent);

            verify(mockCallAudioManager).onRingerModeChange();

            foundValid = true;

        }

        assertTrue(foundValid);

    }



    @MediumTest

    @Test

    public void testSpeakerPersistence() {