diff options
author | virkumar <virkumar@google.com> | 2022-04-21 09:31:35 +0000 |
---|---|---|
committer | Virkumar Karavate <virkumar@google.com> | 2022-04-22 10:22:40 +0000 |
commit | eb40db77e8b13c9aefddc8e3c64e2ae93df1ddd5 (patch) | |
tree | 5f2edb6b2844f7fa75ee03a8668440156a295119 | |
parent | 07709d2168cc3a4ce46c0e72c88759509d74a21a (diff) | |
download | ims-eb40db77e8b13c9aefddc8e3c64e2ae93df1ddd5.tar.gz |
(Increasing logging) Adding a new case for IllegalStateException
Handle IllegalStateException for increasing logging incase such excpetion occures.
Test: atest CtsTelephonyTestCases:ImsCallingTest
Bug: 229756822
Merged-In: Iea42ae7c41eb424978bf4390aa8cea63fbd81fce
Change-Id: Iea42ae7c41eb424978bf4390aa8cea63fbd81fce
(cherry picked from commit 4bacd9b15a36491fd5727c836258de19905c6ae7)
-rw-r--r-- | src/java/com/android/ims/MmTelFeatureConnection.java | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/src/java/com/android/ims/MmTelFeatureConnection.java b/src/java/com/android/ims/MmTelFeatureConnection.java index cd8c3f15..3170d413 100644 --- a/src/java/com/android/ims/MmTelFeatureConnection.java +++ b/src/java/com/android/ims/MmTelFeatureConnection.java @@ -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()); } } } |