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

Commit 57d5bd9f authored by Omkar Kolangade's avatar Omkar Kolangade Committed by Tyler Gunn
Browse files

IMS-VT: Update video state of the connection first. 

Video state is used to determine if connection modification
happened. Delaying video state updates confuses other layers,
which use the video state notification to determine if session
upgrade/downgrade happened.

Bug: 27477899
Change-Id: Ibc5f89dc631625322b04f8f68686877a2b25999b
parent 5ba11026

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

+2 −1
Original line number Diff line number Diff line
@@ -400,6 +400,7 @@ public class ImsPhoneCallTracker extends CallTracker { 
            mPendingMO = new ImsPhoneConnection(mPhone,

                    checkForTestEmergencyNumber(dialString), this, mForegroundCall,

                    isEmergencyNumber);

            mPendingMO.setVideoState(videoState);

        }

        addConnection(mPendingMO);
@@ -1073,9 +1074,9 @@ public class ImsPhoneCallTracker extends CallTracker { 
        // It should modify only other capabilities of call through updateMediaCapabilities

        // State updates will be triggered through individual callbacks

        // i.e. onCallHeld, onCallResume, etc and conn.update will be responsible for the update

        conn.updateMediaCapabilities(imsCall);

        if (ignoreState) {

            conn.updateAddressDisplay(imsCall);

            conn.updateMediaCapabilities(imsCall);

            conn.updateExtras(imsCall);

            return;

        }