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

Commit 77474bd6 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Suppress state changes in CallAudioManager when state doesn't change."... 

Merge "Suppress state changes in CallAudioManager when state doesn't change." into tm-dev am: 7ad98a55

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



Change-Id: I26e342c0e44e3aa8f77a1e6b5937020eadbb1932
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents f6d50225 7ad98a55

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

+1 −1
Original line number Diff line number Diff line
@@ -1262,7 +1262,7 @@ public class Call implements CreateConnectionResponse, EventManager.Loggable, 
        }

    }



    boolean isRingbackRequested() {

    public boolean isRingbackRequested() {

        return mRingbackRequested;

    }

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

+4 −0
Original line number Diff line number Diff line
@@ -107,6 +107,10 @@ public class CallAudioManager extends CallsManagerListenerBase { 
            // No audio management for calls in a conference, or external calls.

            return;

        }

        if (oldState == newState) {

            // State did not change, so no need to do anything.

            return;

        }

        Log.d(LOG_TAG, "Call state changed for TC@%s: %s -> %s", call.getId(),

                CallState.toString(oldState), CallState.toString(newState));

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

+1 −1
Original line number Diff line number Diff line
@@ -145,9 +145,9 @@ public class CallScreeningServiceHelper { 


        if (!bindCallScreeningService(mContext, mUserHandle, mPackageName, serviceConnection)) {

            Log.i(this, "bindAndGetCallIdentification - bind failed");

            Log.addEvent(mCall, LogUtils.Events.BIND_SCREENING, mPackageName);

            mFuture.complete(null);

        }

        Log.addEvent(mCall, LogUtils.Events.BIND_SCREENING, mPackageName);



        // Set up a timeout so that we're not waiting forever for the caller ID information.

        Handler handler = new Handler();

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

+54 −0
Original line number Diff line number Diff line
@@ -294,6 +294,60 @@ public class CallAudioManagerTest extends TelecomTestCase { 
        verifyProperCleanup();

    }



    @MediumTest

    @Test

    public void testRingbackStartStop() {

        Call call = mock(Call.class);

        ArgumentCaptor<CallAudioModeStateMachine.MessageArgs> captor = makeNewCaptor();

        when(call.getState()).thenReturn(CallState.CONNECTING);

        when(call.isRingbackRequested()).thenReturn(true);



        mCallAudioManager.onCallAdded(call);

        assertEquals(call, mCallAudioManager.getForegroundCall());

        verify(mCallAudioRouteStateMachine).sendMessageWithSessionInfo(

                CallAudioRouteStateMachine.UPDATE_SYSTEM_AUDIO_ROUTE);

        verify(mCallAudioModeStateMachine).sendMessageWithArgs(

                eq(CallAudioModeStateMachine.NEW_ACTIVE_OR_DIALING_CALL), captor.capture());

        CallAudioModeStateMachine.MessageArgs expectedArgs =

                new Builder()

                        .setHasActiveOrDialingCalls(true)

                        .setHasRingingCalls(false)

                        .setHasHoldingCalls(false)

                        .setIsTonePlaying(false)

                        .setHasAudioProcessingCalls(false)

                        .setForegroundCallIsVoip(false)

                        .setSession(null)

                        .build();

        assertMessageArgEquality(expectedArgs, captor.getValue());



        when(call.getState()).thenReturn(CallState.DIALING);

        mCallAudioManager.onCallStateChanged(call, CallState.CONNECTING, CallState.DIALING);

        verify(mCallAudioModeStateMachine, times(2)).sendMessageWithArgs(

                eq(CallAudioModeStateMachine.NEW_ACTIVE_OR_DIALING_CALL), captor.capture());

        assertMessageArgEquality(expectedArgs, captor.getValue());

        verify(mCallAudioModeStateMachine, times(2)).sendMessageWithArgs(

                anyInt(), any(CallAudioModeStateMachine.MessageArgs.class));



        // Ensure we started ringback.

        verify(mRingbackPlayer).startRingbackForCall(any(Call.class));



        // Report state change from dialing to dialing, which happens when a call is locally

        // disconnected.

        mCallAudioManager.onCallStateChanged(call, CallState.DIALING, CallState.DIALING);

        // Should not have stopped ringback.

        verify(mRingbackPlayer, never()).stopRingbackForCall(any(Call.class));

        // Should still only have initial ringback start

        verify(mRingbackPlayer, times(1)).startRingbackForCall(any(Call.class));



        // Report state to disconnected

        when(call.getState()).thenReturn(CallState.DISCONNECTED);

        mCallAudioManager.onCallStateChanged(call, CallState.DIALING, CallState.DISCONNECTED);

        // Now we should have stopped ringback.

        verify(mRingbackPlayer).stopRingbackForCall(any(Call.class));

        // Should still only have initial ringback start recorded from before (don't restart it).

        verify(mRingbackPlayer, times(1)).startRingbackForCall(any(Call.class));

    }



    @SmallTest

    @Test

    public void testNewCallGoesToAudioProcessing() {