Snap for 11005173 from 49d602209227c42e21393064e6b08bcf5ac035c8 to 24Q1-release (f3fc7487) · Commits · e / os / android_packages_services_Telecomm

flags/telecom_callaudioroutestatemachine_flags.aconfig

+15 −1

Original line number	Diff line number	Diff line
		@@ -6,3 +6,17 @@ flag {
		description: "Fix supported routes wrongly include bluetooth issue."
		bug: "292599751"
		}

		flag {
		name: "ensure_audio_mode_updates_on_foreground_call_change"
		namespace: "telecom"
		description: "Ensure that the audio mode is updated anytime the foreground call changes."
		bug: "289861657"
		}

		flag {
		name: "transit_route_before_audio_disconnect_bt"
		namespace: "telecom"
		description: "Fix audio route transition issue on call disconnection when bt audio connected."
		bug: "306113816"
		}

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

+26 −15

Original line number	Diff line number	Diff line
		@@ -31,6 +31,7 @@ import com.android.internal.annotations.VisibleForTesting;
		import com.android.internal.util.IndentingPrintWriter;
		import com.android.server.telecom.CallAudioModeStateMachine.MessageArgs.Builder;
		import com.android.server.telecom.bluetooth.BluetoothStateReceiver;
		import com.android.server.telecom.flags.FeatureFlags;

		import java.util.Collection;
		import java.util.HashSet;
		@@ -62,6 +63,7 @@ public class CallAudioManager extends CallsManagerListenerBase {
		private final Ringer mRinger;
		private final RingbackPlayer mRingbackPlayer;
		private final DtmfLocalTonePlayer mDtmfLocalTonePlayer;
		private final FeatureFlags mFeatureFlags;

		private Call mStreamingCall;
		private Call mForegroundCall;
		@@ -76,7 +78,8 @@ public class CallAudioManager extends CallsManagerListenerBase {
		Ringer ringer,
		RingbackPlayer ringbackPlayer,
		BluetoothStateReceiver bluetoothStateReceiver,
		DtmfLocalTonePlayer dtmfLocalTonePlayer) {
		DtmfLocalTonePlayer dtmfLocalTonePlayer,
		FeatureFlags featureFlags) {
		mActiveDialingOrConnectingCalls = new LinkedHashSet<>(1);
		mRingingCalls = new LinkedHashSet<>(1);
		mHoldingCalls = new LinkedHashSet<>(1);
		@@ -102,6 +105,7 @@ public class CallAudioManager extends CallsManagerListenerBase {
		mRingbackPlayer = ringbackPlayer;
		mBluetoothStateReceiver = bluetoothStateReceiver;
		mDtmfLocalTonePlayer = dtmfLocalTonePlayer;
		mFeatureFlags = featureFlags;

		mPlayerFactory.setCallAudioManager(this);
		mCallAudioModeStateMachine.setCallAudioManager(this);
		@@ -772,6 +776,7 @@ public class CallAudioManager extends CallsManagerListenerBase {
		possibleConnectingCall = call;
		}
		}
		if (mFeatureFlags.ensureAudioModeUpdatesOnForegroundCallChange()) {
		// Prefer a connecting call
		if (possibleConnectingCall != null) {
		mForegroundCall = possibleConnectingCall;
		@@ -787,6 +792,11 @@ public class CallAudioManager extends CallsManagerListenerBase {
		// If we can't find one, then just fall back to the first one.
		.orElse(mActiveDialingOrConnectingCalls.iterator().next());
		}
		} else {
		// Legacy (buggy) behavior.
		mForegroundCall = possibleConnectingCall == null ?
		mActiveDialingOrConnectingCalls.iterator().next() : possibleConnectingCall;
		}
		} else if (mRingingCalls.size() > 0) {
		mForegroundCall = mRingingCalls.iterator().next();
		} else if (mHoldingCalls.size() > 0) {
		@@ -806,7 +816,8 @@ public class CallAudioManager extends CallsManagerListenerBase {
		mCallAudioRouteStateMachine.sendMessageWithSessionInfo(
		CallAudioRouteStateMachine.UPDATE_SYSTEM_AUDIO_ROUTE);

		if (mForegroundCall != null) {
		if (mForegroundCall != null
		&& mFeatureFlags.ensureAudioModeUpdatesOnForegroundCallChange()) {
		// Ensure the voip audio mode for the new foreground call is taken into account.
		mCallAudioModeStateMachine.sendMessageWithArgs(
		CallAudioModeStateMachine.FOREGROUND_VOIP_MODE_CHANGE,

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

+19 −8

Original line number	Diff line number	Diff line
		@@ -23,7 +23,6 @@ import android.content.BroadcastReceiver;
		import android.content.Context;
		import android.content.Intent;
		import android.content.IntentFilter;
		import android.content.pm.PackageManager;
		import android.content.pm.UserInfo;
		import android.media.AudioDeviceInfo;
		import android.media.AudioManager;
		@@ -45,6 +44,7 @@ import com.android.internal.util.IndentingPrintWriter;
		import com.android.internal.util.State;
		import com.android.internal.util.StateMachine;
		import com.android.server.telecom.bluetooth.BluetoothRouteManager;
		import com.android.server.telecom.flags.FeatureFlags;

		import java.util.Collection;
		import java.util.HashMap;
		@@ -85,7 +85,8 @@ public class CallAudioRouteStateMachine extends StateMachine {
		CallAudioManager.AudioServiceFactory audioServiceFactory,
		int earpieceControl,
		Executor asyncTaskExecutor,
		CallAudioCommunicationDeviceTracker communicationDeviceTracker) {
		CallAudioCommunicationDeviceTracker communicationDeviceTracker,
		FeatureFlags featureFlags) {
		return new CallAudioRouteStateMachine(context,
		callsManager,
		bluetoothManager,
		@@ -94,7 +95,8 @@ public class CallAudioRouteStateMachine extends StateMachine {
		audioServiceFactory,
		earpieceControl,
		asyncTaskExecutor,
		communicationDeviceTracker);
		communicationDeviceTracker,
		featureFlags);
		}
		}
		/** Values for CallAudioRouteStateMachine constructor's earPieceRouting arg. */
		@@ -918,8 +920,13 @@ public class CallAudioRouteStateMachine extends StateMachine {
		case SWITCH_FOCUS:
		if (msg.arg1 == NO_FOCUS) {
		// Only disconnect audio here instead of routing away from BT entirely.
		if (mFeatureFlags.transitRouteBeforeAudioDisconnectBt()) {
		transitionTo(mQuiescentBluetoothRoute);
		mBluetoothRouteManager.disconnectAudio();
		} else {
		mBluetoothRouteManager.disconnectAudio();
		transitionTo(mQuiescentBluetoothRoute);
		}
		mCallAudioManager.notifyAudioOperationsComplete();
		} else if (msg.arg1 == RINGING_FOCUS
		&& !mBluetoothRouteManager.isInbandRingingEnabled()) {
		@@ -1548,6 +1555,7 @@ public class CallAudioRouteStateMachine extends StateMachine {

		private CallAudioManager mCallAudioManager;
		private CallAudioCommunicationDeviceTracker mCommunicationDeviceTracker;
		private FeatureFlags mFeatureFlags;

		public CallAudioRouteStateMachine(
		Context context,
		@@ -1558,7 +1566,8 @@ public class CallAudioRouteStateMachine extends StateMachine {
		CallAudioManager.AudioServiceFactory audioServiceFactory,
		int earpieceControl,
		Executor asyncTaskExecutor,
		CallAudioCommunicationDeviceTracker communicationDeviceTracker) {
		CallAudioCommunicationDeviceTracker communicationDeviceTracker,
		FeatureFlags featureFlags) {
		super(NAME);
		mContext = context;
		mCallsManager = callsManager;
		@@ -1570,6 +1579,7 @@ public class CallAudioRouteStateMachine extends StateMachine {
		mLock = callsManager.getLock();
		mAsyncTaskExecutor = asyncTaskExecutor;
		mCommunicationDeviceTracker = communicationDeviceTracker;
		mFeatureFlags = featureFlags;
		createStates(earpieceControl);
		}

		@@ -1582,7 +1592,8 @@ public class CallAudioRouteStateMachine extends StateMachine {
		StatusBarNotifier statusBarNotifier,
		CallAudioManager.AudioServiceFactory audioServiceFactory,
		int earpieceControl, Looper looper, Executor asyncTaskExecutor,
		CallAudioCommunicationDeviceTracker communicationDeviceTracker) {
		CallAudioCommunicationDeviceTracker communicationDeviceTracker,
		FeatureFlags featureFlags) {
		super(NAME, looper);
		mContext = context;
		mCallsManager = callsManager;
		@@ -1594,7 +1605,7 @@ public class CallAudioRouteStateMachine extends StateMachine {
		mLock = callsManager.getLock();
		mAsyncTaskExecutor = asyncTaskExecutor;
		mCommunicationDeviceTracker = communicationDeviceTracker;

		mFeatureFlags = featureFlags;
		createStates(earpieceControl);
		}

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

+3 −2

Original line number	Diff line number	Diff line
		@@ -616,7 +616,8 @@ public class CallsManager extends Call.ListenerBase
		audioServiceFactory,
		CallAudioRouteStateMachine.EARPIECE_AUTO_DETECT,
		asyncCallAudioTaskExecutor,
		communicationDeviceTracker
		communicationDeviceTracker,
		featureFlags
		);
		callAudioRouteStateMachine.initialize();

		@@ -657,7 +658,7 @@ public class CallsManager extends Call.ListenerBase
		this, callAudioModeStateMachineFactory.create(systemStateHelper,
		(AudioManager) mContext.getSystemService(Context.AUDIO_SERVICE), featureFlags),
		playerFactory, mRinger, new RingbackPlayer(playerFactory),
		bluetoothStateReceiver, mDtmfLocalTonePlayer);
		bluetoothStateReceiver, mDtmfLocalTonePlayer, featureFlags);

		mConnectionSvrFocusMgr = connectionServiceFocusManagerFactory.create(mRequester);
		mHeadsetMediaButton = headsetMediaButtonFactory.create(context, this, mLock);

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

+45 −11

Original line number	Diff line number	Diff line
		@@ -36,6 +36,7 @@ import com.android.server.telecom.RingbackPlayer;
		import com.android.server.telecom.Ringer;
		import com.android.server.telecom.TelecomSystem;
		import com.android.server.telecom.bluetooth.BluetoothStateReceiver;
		import com.android.server.telecom.flags.FeatureFlags;

		import org.junit.After;
		import org.junit.Before;
		@@ -79,6 +80,8 @@ public class CallAudioManagerTest extends TelecomTestCase {
		@Mock private BluetoothStateReceiver mBluetoothStateReceiver;
		@Mock private TelecomSystem.SyncRoot mLock;

		@Mock private FeatureFlags mFlags;

		private CallAudioManager mCallAudioManager;

		@Override
		@@ -94,6 +97,7 @@ public class CallAudioManagerTest extends TelecomTestCase {
		return mockInCallTonePlayer;
		}).when(mPlayerFactory).createPlayer(anyInt());
		when(mCallsManager.getLock()).thenReturn(mLock);
		when(mFlags.ensureAudioModeUpdatesOnForegroundCallChange()).thenReturn(true);
		mCallAudioManager = new CallAudioManager(
		mCallAudioRouteStateMachine,
		mCallsManager,
		@@ -102,7 +106,8 @@ public class CallAudioManagerTest extends TelecomTestCase {
		mRinger,
		mRingbackPlayer,
		mBluetoothStateReceiver,
		mDtmfLocalTonePlayer);
		mDtmfLocalTonePlayer,
		mFlags);
		}

		@Override
		@@ -278,19 +283,27 @@ public class CallAudioManagerTest extends TelecomTestCase {
		verify(mCallAudioModeStateMachine, times(2)).sendMessageWithArgs(
		eq(CallAudioModeStateMachine.NEW_ACTIVE_OR_DIALING_CALL), captor.capture());
		assertMessageArgEquality(expectedArgs, captor.getValue());
		// Expet another invocation due to audio mode change signal.
		if (mFlags.ensureAudioModeUpdatesOnForegroundCallChange()) {
		// Expect another invocation due to audio mode change signal.
		verify(mCallAudioModeStateMachine, times(3)).sendMessageWithArgs(
		anyInt(), any(CallAudioModeStateMachine.MessageArgs.class));

		} else {
		verify(mCallAudioModeStateMachine, times(2)).sendMessageWithArgs(
		anyInt(), any(CallAudioModeStateMachine.MessageArgs.class));
		}

		when(call.getState()).thenReturn(CallState.ACTIVE);
		mCallAudioManager.onCallStateChanged(call, CallState.DIALING, CallState.ACTIVE);
		verify(mCallAudioModeStateMachine, times(3)).sendMessageWithArgs(
		eq(CallAudioModeStateMachine.NEW_ACTIVE_OR_DIALING_CALL), captor.capture());
		assertMessageArgEquality(expectedArgs, captor.getValue());
		if (mFlags.ensureAudioModeUpdatesOnForegroundCallChange()) {
		verify(mCallAudioModeStateMachine, times(4)).sendMessageWithArgs(
		anyInt(), any(CallAudioModeStateMachine.MessageArgs.class));

		} else {
		verify(mCallAudioModeStateMachine, times(3)).sendMessageWithArgs(
		anyInt(), any(CallAudioModeStateMachine.MessageArgs.class));
		}
		disconnectCall(call);
		stopTone();

		@@ -298,6 +311,12 @@ public class CallAudioManagerTest extends TelecomTestCase {
		verifyProperCleanup();
		}

		@Test
		public void testSingleOutgoingCallWithoutAudioModeUpdateOnForegroundCallChange() {
		when(mFlags.ensureAudioModeUpdatesOnForegroundCallChange()).thenReturn(false);
		testSingleOutgoingCall();
		}

		@MediumTest
		@Test
		public void testRingbackStartStop() {
		@@ -329,9 +348,14 @@ public class CallAudioManagerTest extends TelecomTestCase {
		verify(mCallAudioModeStateMachine, times(2)).sendMessageWithArgs(
		eq(CallAudioModeStateMachine.NEW_ACTIVE_OR_DIALING_CALL), captor.capture());
		assertMessageArgEquality(expectedArgs, captor.getValue());
		if (mFlags.ensureAudioModeUpdatesOnForegroundCallChange()) {
		// Expect an extra time due to audio mode change signal
		verify(mCallAudioModeStateMachine, times(3)).sendMessageWithArgs(
		anyInt(), any(CallAudioModeStateMachine.MessageArgs.class));
		} else {
		verify(mCallAudioModeStateMachine, times(2)).sendMessageWithArgs(
		anyInt(), any(CallAudioModeStateMachine.MessageArgs.class));
		}

		// Ensure we started ringback.
		verify(mRingbackPlayer).startRingbackForCall(any(Call.class));
		@@ -353,6 +377,12 @@ public class CallAudioManagerTest extends TelecomTestCase {
		verify(mRingbackPlayer, times(1)).startRingbackForCall(any(Call.class));
		}

		@Test
		public void testRingbackStartStopWithoutAudioModeUpdateOnForegroundCallChange() {
		when(mFlags.ensureAudioModeUpdatesOnForegroundCallChange()).thenReturn(false);
		testRingbackStartStop();
		}

		@SmallTest
		@Test
		public void testNewCallGoesToAudioProcessing() {
		@@ -708,6 +738,10 @@ public class CallAudioManagerTest extends TelecomTestCase {
		@SmallTest
		@Test
		public void testTriggerAudioManagerModeChange() {
		if (!mFlags.ensureAudioModeUpdatesOnForegroundCallChange()) {
		// Skip if the new behavior isn't in use.
		return;
		}
		// Start with an incoming PSTN call
		Call pstnCall = mock(Call.class);
		when(pstnCall.getState()).thenReturn(CallState.RINGING);