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

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

Merge "Change how reject versus missed calls are reported from... 

Merge "Change how reject versus missed calls are reported from ImsPhoneCallTracker." into tm-dev am: 93ed722b am: 0adffb85

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/opt/telephony/+/17877166



Change-Id: I1eea0d57091ab58d0fdde7e5382407f49ed67813
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents e61d0b74 0adffb85

src/java/com/android/internal/telephony/imsphone/ImsPhoneCallTracker.java

+9 −5
Original line number Diff line number Diff line
@@ -3259,12 +3259,16 @@ public class ImsPhoneCallTracker extends CallTracker implements ImsPullCall { 


                } else if (conn.isIncoming() && conn.getConnectTime() == 0

                        && cause != DisconnectCause.ANSWERED_ELSEWHERE) {

                    // Missed

                    if (cause == DisconnectCause.NORMAL

                            || cause == DisconnectCause.INCOMING_AUTO_REJECTED) {

                        cause = DisconnectCause.INCOMING_MISSED;

                    } else {



                    if (conn.getDisconnectCause() == DisconnectCause.LOCAL) {

                        // If the user initiated a disconnect of this connection, then we will treat

                        // this is a rejected call.

                        // Note; the record the fact that this is a local disconnect in

                        // ImsPhoneConnection#onHangupLocal

                        cause = DisconnectCause.INCOMING_REJECTED;

                    } else {

                        // Otherwise in all other cases consider it missed.

                        cause = DisconnectCause.INCOMING_MISSED;

                    }

                    if (DBG) log("Incoming connection of 0 connect time detected - translated " +

                            "cause = " + cause);

src/java/com/android/internal/telephony/imsphone/ImsPhoneConnection.java

+1 −2
Original line number Diff line number Diff line
@@ -542,8 +542,7 @@ public class ImsPhoneConnection extends Connection implements 
    /**

     * Called when this Connection is being hung up locally (eg, user pressed "end")

     */

    void

    onHangupLocal() {

    public void onHangupLocal() {

        mCause = DisconnectCause.LOCAL;

    }

tests/telephonytests/src/com/android/internal/telephony/imsphone/ImsPhoneCallTrackerTest.java

+26 −0
Original line number Diff line number Diff line
@@ -484,6 +484,31 @@ public class ImsPhoneCallTrackerTest extends TelephonyTest { 
    @Test

    @SmallTest

    public void testImsMTCall() {

        ImsPhoneConnection connection = setupRingingConnection();

        assertEquals(android.telecom.Connection.VERIFICATION_STATUS_PASSED,

                connection.getNumberVerificationStatus());

    }



    @Test

    @SmallTest

    public void testImsMTCallMissed() {

        ImsPhoneConnection connection = setupRingingConnection();

        mImsCallListener.onCallTerminated(connection.getImsCall(),

                new ImsReasonInfo(ImsReasonInfo.CODE_USER_TERMINATED_BY_REMOTE, 0));

        assertEquals(DisconnectCause.INCOMING_MISSED, connection.getDisconnectCause());

    }



    @Test

    @SmallTest

    public void testImsMTCallRejected() {

        ImsPhoneConnection connection = setupRingingConnection();

        connection.onHangupLocal();

        mImsCallListener.onCallTerminated(connection.getImsCall(),

                new ImsReasonInfo(ImsReasonInfo.CODE_SIP_REQUEST_TIMEOUT, 0));

        assertEquals(DisconnectCause.INCOMING_REJECTED, connection.getDisconnectCause());

    }



    private ImsPhoneConnection setupRingingConnection() {

        mImsCallProfile.setCallerNumberVerificationStatus(

                ImsCallProfile.VERIFICATION_STATUS_PASSED);

        assertEquals(PhoneConstants.State.IDLE, mCTUT.getState());
@@ -500,6 +525,7 @@ public class ImsPhoneCallTrackerTest extends TelephonyTest { 
        connection.addListener(mImsPhoneConnectionListener);

        assertEquals(android.telecom.Connection.VERIFICATION_STATUS_PASSED,

                connection.getNumberVerificationStatus());

        return connection;

    }



    @Test