From 3c97693cfb097016282c9a215e4a40c123f88c80 Mon Sep 17 00:00:00 2001 From: Malcolm Chen Date: Wed, 3 Apr 2019 18:39:21 -0700 Subject: [PATCH] Add comments in grouping APIs to clarify possible exceptions. Bug: 127408608 Test: cts, unittest, TelephonyManagerTestApp Change-Id: I3671e366be7e8c3a75b5ee23b7cf88a33ecd3a4f Merged-In: I3671e366be7e8c3a75b5ee23b7cf88a33ecd3a4f --- telephony/java/android/telephony/SubscriptionManager.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/telephony/java/android/telephony/SubscriptionManager.java b/telephony/java/android/telephony/SubscriptionManager.java index b0fa9d526b16d..78f38f645f7d1 100644 --- a/telephony/java/android/telephony/SubscriptionManager.java +++ b/telephony/java/android/telephony/SubscriptionManager.java @@ -2733,6 +2733,8 @@ public class SubscriptionManager { * * @throws SecurityException if the caller doesn't meet the requirements * outlined above. + * @throws IllegalArgumentException if any of the subscriptions in the list doesn't exist. + * @throws IllegalStateException if Telephony service is in bad state. * * @param subIdList list of subId that will be in the same group * @return groupUUID a UUID assigned to the subscription group. @@ -2781,6 +2783,7 @@ public class SubscriptionManager { * outlined above. * @throws IllegalArgumentException if the some subscriptions in the list doesn't exist, * or the groupUuid doesn't exist. + * @throws IllegalStateException if Telephony service is in bad state. * * @param subIdList list of subId that need adding into the group * @param groupUuid the groupUuid the subscriptions are being added to. @@ -2833,6 +2836,7 @@ public class SubscriptionManager { * outlined above. * @throws IllegalArgumentException if the some subscriptions in the list doesn't belong * the specified group. + * @throws IllegalStateException if Telephony service is in bad state. * * @param subIdList list of subId that need removing from their groups. *