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

Commit 7e8765d3 authored by Brad Ebinger's avatar Brad Ebinger Committed by Automerger Merge Worker
Browse files

Merge "Throw NPE instead of IllegalArgumentException in SipDelegateManager"... 

Merge "Throw NPE instead of IllegalArgumentException in SipDelegateManager" am: 5733e616 am: aff57e78 am: 9a880b4f

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1675740

Change-Id: Ib1daec20b74e0225645a53588e72b4f57c78d1cf
parents 6e1db3d5 9a880b4f

telephony/java/android/telephony/ims/SipDelegateManager.java

+7 −10
Original line number Diff line number Diff line
@@ -39,6 +39,7 @@ import com.android.internal.annotations.VisibleForTesting; 


import java.lang.annotation.Retention;

import java.lang.annotation.RetentionPolicy;

import java.util.Objects;

import java.util.concurrent.Executor;



/**
@@ -322,9 +323,10 @@ public class SipDelegateManager { 
    public void createSipDelegate(@NonNull DelegateRequest request, @NonNull Executor executor,

            @NonNull DelegateConnectionStateCallback dc,

            @NonNull DelegateConnectionMessageCallback mc) throws ImsException {

        if (request == null || executor == null || dc == null || mc == null) {

            throw new IllegalArgumentException("Invalid arguments passed into createSipDelegate");

        }

        Objects.requireNonNull(request, "The DelegateRequest must not be null.");

        Objects.requireNonNull(executor, "The Executor must not be null.");

        Objects.requireNonNull(dc, "The DelegateConnectionStateCallback must not be null.");

        Objects.requireNonNull(mc, "The DelegateConnectionMessageCallback must not be null.");

        try {

            SipDelegateConnectionAidlWrapper wrapper =

                    new SipDelegateConnectionAidlWrapper(executor, dc, mc);
@@ -355,10 +357,7 @@ public class SipDelegateManager { 
    @RequiresPermission(Manifest.permission.PERFORM_IMS_SINGLE_REGISTRATION)

    public void destroySipDelegate(@NonNull SipDelegateConnection delegateConnection,

            @SipDelegateDestroyReason int reason) {



        if (delegateConnection == null) {

            throw new IllegalArgumentException("invalid argument passed into destroySipDelegate");

        }

        Objects.requireNonNull(delegateConnection, "SipDelegateConnection can not be null.");

        if (delegateConnection instanceof SipDelegateConnectionAidlWrapper) {

            SipDelegateConnectionAidlWrapper w =

                    (SipDelegateConnectionAidlWrapper) delegateConnection;
@@ -396,9 +395,7 @@ public class SipDelegateManager { 
    @RequiresPermission(Manifest.permission.PERFORM_IMS_SINGLE_REGISTRATION)

    public void triggerFullNetworkRegistration(@NonNull SipDelegateConnection connection,

            @IntRange(from = 100, to = 699) int sipCode, @Nullable String sipReason) {

        if (connection == null) {

            throw new IllegalArgumentException("invalid connection.");

        }

        Objects.requireNonNull(connection, "SipDelegateConnection can not be null.");

        if (connection instanceof SipDelegateConnectionAidlWrapper) {

            SipDelegateConnectionAidlWrapper w = (SipDelegateConnectionAidlWrapper) connection;

            try {