From 69cf3f0f03559e80d2606e8076df5f0ba406d89f Mon Sep 17 00:00:00 2001
From: Amit Mahajan <amitmahajan@google.com>
Date: Tue, 3 Mar 2020 15:49:58 -0800
Subject: [PATCH] Update API documentation per api-council feedback.

Test: SMS/MMS sending/receiving sanity (manual)
Bug: 149236716
Merged--in: I59b9d468fa6e1a5843871abf1a330962aef64505
Change-Id: I59b9d468fa6e1a5843871abf1a330962aef64505
(cherry picked from commit d42340b66cab46f59902bc9f2c57131361612c50)
---
 telephony/java/android/telephony/SmsManager.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/telephony/java/android/telephony/SmsManager.java b/telephony/java/android/telephony/SmsManager.java
index 0f52ba7935ab8..7f5b4226185c6 100644
--- a/telephony/java/android/telephony/SmsManager.java
+++ b/telephony/java/android/telephony/SmsManager.java
@@ -2768,7 +2768,7 @@ public final class SmsManager {
                         getSubscriptionId(), null);
             }
         } catch (RemoteException ex) {
-            // ignore it
+            throw new RuntimeException(ex);
         }
         return smsc;
     }
@@ -2790,7 +2790,7 @@ public final class SmsManager {
      * </p>
      *
      * @param smsc the SMSC address string.
-     * @return true for success, false otherwise.
+     * @return true for success, false otherwise. Failure can be due modem returning an error.
      */
     @SuppressAutoDoc // for carrier privileges and default SMS application.
     @RequiresPermission(android.Manifest.permission.MODIFY_PHONE_STATE)
@@ -2802,7 +2802,7 @@ public final class SmsManager {
                         smsc, getSubscriptionId(), null);
             }
         } catch (RemoteException ex) {
-            // ignore it
+            throw new RuntimeException(ex);
         }
         return false;
     }