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

Commit 4bacd9b1 authored by virkumar's avatar virkumar
Browse files

(Increasing logging) Adding a new case for IllegalStateException 

Handle IllegalStateException for increasing logging incase such excpetion occures.

Test: atest CtsTelephonyTestCases:ImsCallingTest
Bug: 229756822
Change-Id: Iea42ae7c41eb424978bf4390aa8cea63fbd81fce
parent 628ab5a7

src/java/com/android/ims/MmTelFeatureConnection.java

+10 −8
Original line number Diff line number Diff line
@@ -87,9 +87,10 @@ public class MmTelFeatureConnection extends FeatureConnection { 
            if (imsRegistration != null) {

                try {

                    imsRegistration.removeRegistrationCallback(localCallback);

                } catch (RemoteException e) {

                } catch (RemoteException | IllegalStateException e) {

                    Log.w(TAG + " [" + mSlotId + "]", "ImsRegistrationCallbackAdapter -"

                            + " unregisterCallback: couldn't remove registration callback");

                            + " unregisterCallback: couldn't remove registration callback"

                            + " Exception: " + e.getMessage());

                }

            } else {

                Log.e(TAG + " [" + mSlotId + "]", "ImsRegistrationCallbackAdapter: ImsRegistration"
@@ -145,9 +146,9 @@ public class MmTelFeatureConnection extends FeatureConnection { 
            if (binder != null) {

                try {

                    binder.removeCapabilityCallback(localCallback);

                } catch (RemoteException e) {

                } catch (RemoteException | IllegalStateException e) {

                    Log.w(TAG + " [" + mSlotId + "]", "CapabilityCallbackManager, unregister:"

                            + " Binder is dead.");

                            + " Binder is dead. Exception: " + e.getMessage());

                }

            } else {

                Log.w(TAG + " [" + mSlotId + "]", "CapabilityCallbackManager, unregister:"
@@ -187,9 +188,9 @@ public class MmTelFeatureConnection extends FeatureConnection { 
            }

            try {

                binder.removeImsConfigCallback(localCallback);

            } catch (RemoteException e) {

            } catch (RemoteException | IllegalStateException e) {

                Log.w(TAG + " [" + mSlotId + "]", "ProvisioningCallbackManager - couldn't"

                        + " unregister, binder is dead.");

                        + " unregister, binder is dead. Exception: " + e.getMessage());

            }

        }

    }
@@ -322,8 +323,9 @@ public class MmTelFeatureConnection extends FeatureConnection { 
                    mMultiEndpoint.getInterface().setExternalCallStateListener(null);

                    mMultiEndpoint = new BinderAccessState<>(BinderAccessState.STATE_NOT_SET);

                }

            } catch (RemoteException e) {

                Log.w(TAG + " [" + mSlotId + "]", "closeConnection: couldn't remove listeners!");

            } catch (RemoteException | IllegalStateException e) {

                Log.w(TAG + " [" + mSlotId + "]", "closeConnection: couldn't remove listeners!" +

                        " Exception: " + e.getMessage());

            }

        }

    }