Merge qt-r1-dev-plus-aosp-without-vendor (5817612) into stage-aosp-master (a6b44cc4) · Commits · e / os / android_packages_services_Telecomm

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

+72 −56

Original line number	Diff line number	Diff line
		@@ -26,6 +26,7 @@ import android.os.Handler;
		import android.os.HandlerThread;
		import android.os.Message;
		import android.telecom.Log;
		import android.telecom.Logging.Session;

		import com.android.internal.annotations.VisibleForTesting;
		import com.android.internal.os.SomeArgs;
		@@ -107,6 +108,7 @@ public class AsyncRingtonePlayer {
		args.arg2 = incomingCall;
		args.arg3 = volumeShaperConfig;
		args.arg4 = isVibrationEnabled;
		args.arg5 = Log.createSubsession();
		postMessage(EVENT_PLAY, true /* shouldCreateHandler */, args);
		return mHapticsFuture;
		}
		@@ -173,11 +175,17 @@ public class AsyncRingtonePlayer {
		Call incomingCall = (Call) args.arg2;
		VolumeShaper.Configuration volumeShaperConfig = (VolumeShaper.Configuration) args.arg3;
		boolean isVibrationEnabled = (boolean) args.arg4;
		Session session = (Session) args.arg5;
		args.recycle();

		Log.continueSession(session, "ARP.hP");
		try {
		// don't bother with any of this if there is an EVENT_STOP waiting.
		if (mHandler.hasMessages(EVENT_STOP)) {
		if (mHapticsFuture != null) {
		mHapticsFuture.complete(false /* ringtoneHasHaptics */);
		mHapticsFuture = null;
		}
		return;
		}

		@@ -185,8 +193,11 @@ public class AsyncRingtonePlayer {
		// anything.
		if (Uri.EMPTY.equals(incomingCall.getRingtone())) {
		mRingtone = null;
		if (mHapticsFuture != null) {
		mHapticsFuture.complete(false /* ringtoneHasHaptics */);
		mHapticsFuture = null;
		}

		return;
		}

		@@ -201,8 +212,10 @@ public class AsyncRingtonePlayer {
		ringtoneUri.toSafeString();
		Log.addEvent(null, LogUtils.Events.ERROR_LOG, "Failed to get ringtone from " +
		"factory. Skipping ringing. Uri was: " + ringtoneUriString);
		if (mHapticsFuture != null) {
		mHapticsFuture.complete(false /* ringtoneHasHaptics */);
		mHapticsFuture = null;
		}
		return;
		}

		@@ -241,6 +254,9 @@ public class AsyncRingtonePlayer {
		mRingtone.play();
		Log.i(this, "Play ringtone, looping.");
		}
		} finally {
		Log.cancelSubsession(session);
		}
		}

		private void handleRepeat() {

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

+27 −15

Original line number	Diff line number	Diff line
		@@ -431,6 +431,7 @@ public class CallAudioManager extends CallsManagerListenerBase {
		}

		void silenceRingers() {
		synchronized (mCallsManager.getLock()) {
		for (Call call : mRingingCalls) {
		call.silence();
		}
		@@ -438,29 +439,38 @@ public class CallAudioManager extends CallsManagerListenerBase {
		mRinger.stopRinging();
		mRinger.stopCallWaiting();
		}
		}

		@VisibleForTesting
		public boolean startRinging() {
		synchronized (mCallsManager.getLock()) {
		return mRinger.startRinging(mForegroundCall,
		mCallAudioRouteStateMachine.isHfpDeviceAvailable());
		}
		}

		@VisibleForTesting
		public void startCallWaiting(String reason) {
		synchronized (mCallsManager.getLock()) {
		if (mRingingCalls.size() == 1) {
		mRinger.startCallWaiting(mRingingCalls.iterator().next(), reason);
		}
		}
		}

		@VisibleForTesting
		public void stopRinging() {
		synchronized (mCallsManager.getLock()) {
		mRinger.stopRinging();
		}
		}

		@VisibleForTesting
		public void stopCallWaiting() {
		synchronized (mCallsManager.getLock()) {
		mRinger.stopCallWaiting();
		}
		}

		@VisibleForTesting
		public void setCallAudioRouteFocusState(int focusState) {
		@@ -801,12 +811,14 @@ public class CallAudioManager extends CallsManagerListenerBase {
		private void maybeStopRingingAndCallWaitingForAnsweredOrRejectedCall(Call call) {
		// Check to see if the call being answered/rejected is the only ringing call, since this
		// will be called before the connection service acknowledges the state change.
		synchronized (mCallsManager.getLock()) {
		if (mRingingCalls.size() == 0 \|\|
		(mRingingCalls.size() == 1 && call == mRingingCalls.iterator().next())) {
		mRinger.stopRinging();
		mRinger.stopCallWaiting();
		}
		}
		}

		private boolean shouldPlayDisconnectTone(int oldState, int newState) {
		if (newState != CallState.DISCONNECTED) {

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

+20 −6

Original line number	Diff line number	Diff line
		@@ -327,18 +327,25 @@ public class CallAudioModeStateMachine extends StateMachine {
		}
		return HANDLED;
		case NO_MORE_HOLDING_CALLS:
		// Do nothing.
		if (args.foregroundCallIsVoip) {
		transitionTo(mVoipCallFocusState);
		}
		return HANDLED;
		case NEW_ACTIVE_OR_DIALING_CALL:
		// Do nothing. Already active.
		if (args.foregroundCallIsVoip) {
		transitionTo(mVoipCallFocusState);
		}
		return HANDLED;
		case NEW_RINGING_CALL:
		// Don't make a call ring over an active call, but do play a call waiting tone.
		mCallAudioManager.startCallWaiting("call already active");
		return HANDLED;
		case NEW_HOLDING_CALL:
		// Don't do anything now. Putting an active call on hold will be handled when
		// Just check the voip mode. Putting an active call on hold will be handled when
		// NO_MORE_ACTIVE_CALLS is processed.
		if (args.foregroundCallIsVoip) {
		transitionTo(mVoipCallFocusState);
		}
		return HANDLED;
		case FOREGROUND_VOIP_MODE_CHANGE:
		if (args.foregroundCallIsVoip) {
		@@ -382,18 +389,25 @@ public class CallAudioModeStateMachine extends StateMachine {
		}
		return HANDLED;
		case NO_MORE_HOLDING_CALLS:
		// Do nothing.
		if (!args.foregroundCallIsVoip) {
		transitionTo(mSimCallFocusState);
		}
		return HANDLED;
		case NEW_ACTIVE_OR_DIALING_CALL:
		// Do nothing. Already active.
		if (!args.foregroundCallIsVoip) {
		transitionTo(mSimCallFocusState);
		}
		return HANDLED;
		case NEW_RINGING_CALL:
		// Don't make a call ring over an active call, but do play a call waiting tone.
		mCallAudioManager.startCallWaiting("call already active");
		return HANDLED;
		case NEW_HOLDING_CALL:
		// Don't do anything now. Putting an active call on hold will be handled when
		// Just check the voip mode. Putting an active call on hold will be handled when
		// NO_MORE_ACTIVE_CALLS is processed.
		if (!args.foregroundCallIsVoip) {
		transitionTo(mSimCallFocusState);
		}
		return HANDLED;
		case FOREGROUND_VOIP_MODE_CHANGE:
		if (!args.foregroundCallIsVoip) {

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

+3 −0

Original line number	Diff line number	Diff line
		@@ -31,6 +31,7 @@ import com.android.server.telecom.DtmfLocalTonePlayer;
		import com.android.server.telecom.InCallTonePlayer;
		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 org.junit.Before;
		@@ -66,6 +67,7 @@ public class CallAudioManagerTest extends TelecomTestCase {
		@Mock private RingbackPlayer mRingbackPlayer;
		@Mock private DtmfLocalTonePlayer mDtmfLocalTonePlayer;
		@Mock private BluetoothStateReceiver mBluetoothStateReceiver;
		@Mock private TelecomSystem.SyncRoot mLock;

		private CallAudioManager mCallAudioManager;

		@@ -81,6 +83,7 @@ public class CallAudioManagerTest extends TelecomTestCase {
		}).when(mockInCallTonePlayer).startTone();
		return mockInCallTonePlayer;
		}).when(mPlayerFactory).createPlayer(anyInt());
		when(mCallsManager.getLock()).thenReturn(mLock);
		mCallAudioManager = new CallAudioManager(
		mCallAudioRouteStateMachine,
		mCallsManager,

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

+76 −0

Original line number	Diff line number	Diff line
		@@ -413,6 +413,82 @@ public class CallAudioModeTransitionTests extends TelecomTestCase {
		NO_CHANGE // expectedCallWaitingInteraction
		));

		result.add(new ModeTestParameters(
		"Swap between voip and sim calls - 1",
		CallAudioModeStateMachine.ENTER_CALL_FOCUS_FOR_TESTING, // initialAudioState
		CallAudioModeStateMachine.NEW_HOLDING_CALL, // messageType
		new CallAudioModeStateMachine.MessageArgs(
		false, // hasActiveOrDialingCalls
		false, // hasRingingCalls
		true, // hasHoldingCalls
		false, // isTonePlaying
		true, // foregroundCallIsVoip
		null // session
		),
		CallAudioModeStateMachine.COMMS_STATE_NAME, // expectedFinalStateName
		FOCUS_VOICE, // expectedFocus
		AudioManager.MODE_IN_COMMUNICATION, // expectedMode
		NO_CHANGE, // expectedRingingInteraction
		NO_CHANGE // expectedCallWaitingInteraction
		));

		result.add(new ModeTestParameters(
		"Swap between voip and sim calls - 2",
		CallAudioModeStateMachine.ENTER_COMMS_FOCUS_FOR_TESTING, // initialAudioState
		CallAudioModeStateMachine.NEW_HOLDING_CALL, // messageType
		new CallAudioModeStateMachine.MessageArgs(
		false, // hasActiveOrDialingCalls
		false, // hasRingingCalls
		true, // hasHoldingCalls
		false, // isTonePlaying
		false, // foregroundCallIsVoip
		null // session
		),
		CallAudioModeStateMachine.CALL_STATE_NAME, // expectedFinalStateName
		FOCUS_VOICE, // expectedFocus
		AudioManager.MODE_IN_CALL, // expectedMode
		NO_CHANGE, // expectedRingingInteraction
		NO_CHANGE // expectedCallWaitingInteraction
		));

		result.add(new ModeTestParameters(
		"Swap between voip and sim calls - 3",
		CallAudioModeStateMachine.ENTER_COMMS_FOCUS_FOR_TESTING, // initialAudioState
		CallAudioModeStateMachine.NEW_ACTIVE_OR_DIALING_CALL, // messageType
		new CallAudioModeStateMachine.MessageArgs(
		false, // hasActiveOrDialingCalls
		false, // hasRingingCalls
		true, // hasHoldingCalls
		false, // isTonePlaying
		false, // foregroundCallIsVoip
		null // session
		),
		CallAudioModeStateMachine.CALL_STATE_NAME, // expectedFinalStateName
		FOCUS_VOICE, // expectedFocus
		AudioManager.MODE_IN_CALL, // expectedMode
		NO_CHANGE, // expectedRingingInteraction
		NO_CHANGE // expectedCallWaitingInteraction
		));

		result.add(new ModeTestParameters(
		"Swap between voip and sim calls - 4",
		CallAudioModeStateMachine.ENTER_CALL_FOCUS_FOR_TESTING, // initialAudioState
		CallAudioModeStateMachine.NEW_HOLDING_CALL, // messageType
		new CallAudioModeStateMachine.MessageArgs(
		false, // hasActiveOrDialingCalls
		false, // hasRingingCalls
		true, // hasHoldingCalls
		false, // isTonePlaying
		true, // foregroundCallIsVoip
		null // session
		),
		CallAudioModeStateMachine.COMMS_STATE_NAME, // expectedFinalStateName
		FOCUS_VOICE, // expectedFocus
		AudioManager.MODE_IN_COMMUNICATION, // expectedMode
		NO_CHANGE, // expectedRingingInteraction
		NO_CHANGE // expectedCallWaitingInteraction
		));

		result.add(new ModeTestParameters(
		"Call is taken off hold - 1",
		CallAudioModeStateMachine.ENTER_TONE_OR_HOLD_FOCUS_FOR_TESTING, // initialAudioState

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

+1 −1

File changed.

Contains only whitespace changes.