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

Commit 99e4576c authored by Ravi Paluri's avatar Ravi Paluri Committed by Steve Kondik
Browse files

IMS-VT: Update videoProvider in main thread 

Setting video provider in seperate thread is causing timing
issues due to which media signalling is happening on
video provider of closed session. So, process merge complete
indication after video provider is updated to ensure that
media signalling happens on video provider attached with
conference imscallsession.

Change-Id: I9a5ca67402effbf22135addcbd3be240d36669b8
CRs-Fixed: 940142
parent 10b5e03b

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

+9 −18
Original line number Diff line number Diff line
@@ -1415,11 +1415,6 @@ public final class ImsPhoneCallTracker extends CallTracker { 
            }

            foregroundImsPhoneCall.merge(peerImsPhoneCall, ImsPhoneCall.State.ACTIVE);



            // TODO Temporary code. Remove the try-catch block from the runnable once thread

            // synchronization is fixed.

            Runnable r = new Runnable() {

                @Override

                public void run() {

            try {

                final ImsPhoneConnection conn = findConnection(call);

                log("onCallMerged: ImsPhoneConnection=" + conn);
@@ -1429,10 +1424,6 @@ public final class ImsPhoneCallTracker extends CallTracker { 
            } catch (Exception e) {

                loge("onCallMerged: exception " + e);

            }

                }

            };



            ImsPhoneCallTracker.this.post(r);



            // After merge complete, update foreground as Active

            // and background call as Held, if background call exists