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

Commit 04058658 authored by Hakjun Choi's avatar Hakjun Choi
Browse files

add setDefaultExecutor to ImsCallSessionListener to set the executor when a... 

add setDefaultExecutor to ImsCallSessionListener to set the executor when a conference call is established

add a way to set the executor variable in ImsCallSessionListener, because ImsCallSessionListener has the information of ImsCallSessionImpl, which passed from vendor

Bug: 262800764
Test: aTest android.telephony.ims.*
Test: Call / Conference call E2E regression test
Merged-In: I63ba6a9d4ea4b8d963bc7bc2ee8e1d019c75ee10
Change-Id: I63ba6a9d4ea4b8d963bc7bc2ee8e1d019c75ee10
parent 325527e2

telephony/java/android/telephony/ims/ImsCallSessionListener.java

+28 −1
Original line number Diff line number Diff line
@@ -31,6 +31,7 @@ import com.android.ims.internal.IImsCallSession; 
import java.util.ArrayList;

import java.util.Objects;

import java.util.Set;

import java.util.concurrent.Executor;



/**

 * Listener interface for notifying the Framework's {@link ImsCallSession} for updates to an ongoing
@@ -44,8 +45,8 @@ import java.util.Set; 
// ImsCallSessionListenerConverter is also changed.

@SystemApi

public class ImsCallSessionListener {



    private final IImsCallSessionListener mListener;

    private Executor mExecutor = null;



    /** @hide */

    public ImsCallSessionListener(IImsCallSessionListener l) {
@@ -243,6 +244,9 @@ public class ImsCallSessionListener { 
    public void callSessionMergeStarted(ImsCallSessionImplBase newSession, ImsCallProfile profile)

    {

        try {

            if (newSession != null && mExecutor != null) {

                newSession.setDefaultExecutor(mExecutor);

            }

            mListener.callSessionMergeStarted(newSession != null ?

                            newSession.getServiceImpl() : null, profile);

        } catch (RemoteException e) {
@@ -274,6 +278,9 @@ public class ImsCallSessionListener { 
     */

    public void callSessionMergeComplete(ImsCallSessionImplBase newSession) {

        try {

            if (newSession != null && mExecutor != null) {

                newSession.setDefaultExecutor(mExecutor);

            }

            mListener.callSessionMergeComplete(newSession != null ?

                    newSession.getServiceImpl() : null);

        } catch (RemoteException e) {
@@ -361,6 +368,9 @@ public class ImsCallSessionListener { 
    public void callSessionConferenceExtended(ImsCallSessionImplBase newSession,

            ImsCallProfile profile) {

        try {

            if (newSession != null && mExecutor != null) {

                newSession.setDefaultExecutor(mExecutor);

            }

            mListener.callSessionConferenceExtended(

                    newSession != null ? newSession.getServiceImpl() : null, profile);

        } catch (RemoteException e) {
@@ -406,6 +416,9 @@ public class ImsCallSessionListener { 
    public void callSessionConferenceExtendReceived(ImsCallSessionImplBase newSession,

            ImsCallProfile profile) {

        try {

            if (newSession != null && mExecutor != null) {

                newSession.setDefaultExecutor(mExecutor);

            }

            mListener.callSessionConferenceExtendReceived(newSession != null

                    ? newSession.getServiceImpl() : null, profile);

        } catch (RemoteException e) {
@@ -808,5 +821,19 @@ public class ImsCallSessionListener { 
            e.rethrowFromSystemServer();

        }

    }



    /**

     * Set default Executor from ImsService.

     * @param executor The default executor to use when executing the methods by the vendor

     *                 implementation of {@link ImsCallSessionImplBase} for conference call.

     *                 This executor is dedicated to set vendor CallSessionImpl

     *                 only when conference call is established.

     * @hide

     */

    public final void setDefaultExecutor(@NonNull Executor executor) {

        if (mExecutor == null) {

            mExecutor = executor;

        }

    }

}

telephony/java/android/telephony/ims/feature/MmTelFeature.java

+1 −0
Original line number Diff line number Diff line
@@ -644,6 +644,7 @@ public class MmTelFeature extends ImsFeature { 
            throw new IllegalStateException("Session is not available.");

        }

        try {

            c.setDefaultExecutor(MmTelFeature.this.mExecutor);

            listener.onIncomingCall(c.getServiceImpl(), extras);

        } catch (RemoteException e) {

            throw new RuntimeException(e);

telephony/java/android/telephony/ims/stub/ImsCallSessionImplBase.java

+3 −2
Original line number Diff line number Diff line
@@ -173,8 +173,9 @@ public class ImsCallSessionImplBase implements AutoCloseable { 


        @Override

        public void setListener(IImsCallSessionListener listener) {

            executeMethodAsync(() -> ImsCallSessionImplBase.this.setListener(

                    new ImsCallSessionListener(listener)), "setListener");

            ImsCallSessionListener iCSL = new ImsCallSessionListener(listener);

            iCSL.setDefaultExecutor(mExecutor);

            executeMethodAsync(() -> ImsCallSessionImplBase.this.setListener(iCSL), "setListener");

        }



        @Override