commit 60b3b4ab1935447e825747a34fdd7b04c954ce23
author PODISHETTY KUMAR <podishettyk@google.com> Wed Jan 24 10:56:29 2024 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Wed Jan 24 10:56:34 2024 +0000
tree 20e342275927285848d4c8edf73311216a42383d
parent f0c02261966126a19fcbd987cb23dcecbf3b52c4

Revert "reflect the refactoring for satellite API"

Revert submission 25876830-306111250

Reason for revert: <Reason for revert: Potential culprit for b/322132989  - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.>

Reverted changes: /q/submissionid:25876830-306111250

Change-Id: I9334f4a9c1cae70094e42f51a4014db4f5a10928

testapps/TestSatelliteApp/src/com/android/phone/testapps/satellitetestapp/TestSatelliteWrapper.java

diff --git a/testapps/TestSatelliteApp/src/com/android/phone/testapps/satellitetestapp/TestSatelliteWrapper.java b/testapps/TestSatelliteApp/src/com/android/phone/testapps/satellitetestapp/TestSatelliteWrapper.java
index cbbd621..792c984 100644
--- a/testapps/TestSatelliteApp/src/com/android/phone/testapps/satellitetestapp/TestSatelliteWrapper.java
+++ b/testapps/TestSatelliteApp/src/com/android/phone/testapps/satellitetestapp/TestSatelliteWrapper.java
@@ -23,6 +23,7 @@
 import android.os.OutcomeReceiver;
 import android.telephony.SubscriptionInfo;
 import android.telephony.SubscriptionManager;
+import android.telephony.satellite.SatelliteManager;
 import android.telephony.satellite.wrapper.NtnSignalStrengthCallbackWrapper;
 import android.telephony.satellite.wrapper.NtnSignalStrengthWrapper;
 import android.telephony.satellite.wrapper.SatelliteCapabilitiesCallbackWrapper;
@@ -40,6 +41,7 @@
 import java.util.concurrent.Executors;
 import java.util.stream.Collectors;
 
+
 /**
  * Activity related to SatelliteControl APIs for satellite.
  */
@@ -146,7 +148,7 @@
 
         try {
             mSatelliteManagerWrapper.requestNtnSignalStrength(mExecutor, receiver);
-        } catch (SecurityException ex) {
+        } catch (SecurityException | IllegalStateException ex) {
             String errorMessage = "requestNtnSignalStrength: " + ex.getMessage();
             Log.d(TAG, errorMessage);
             addLogMessage(errorMessage);
@@ -165,10 +167,18 @@
             mSatelliteManagerWrapper.registerForNtnSignalStrengthChanged(mExecutor,
                     mNtnSignalStrengthCallback);
         } catch (Exception ex) {
-            String errorMessage = "registerForNtnSignalStrengthChanged: " + ex.getMessage();
+            String errorMessage;
+            if (ex instanceof SatelliteManager.SatelliteException) {
+                errorMessage =
+                        "registerForNtnSignalStrengthChanged: " + translateResultCodeToString(
+                                ((SatelliteManager.SatelliteException) ex).getErrorCode());
+            } else {
+                errorMessage = "registerForNtnSignalStrengthChanged: " + ex.getMessage();
+            }
             Log.d(TAG, errorMessage);
             addLogMessage(errorMessage);
             mNtnSignalStrengthCallback = null;
+
         }
     }
 
@@ -307,8 +317,6 @@
                 return "SATELLITE_RESULT_REQUEST_IN_PROGRESS";
             case SatelliteManagerWrapper.SATELLITE_RESULT_MODEM_BUSY:
                 return "SATELLITE_RESULT_MODEM_BUSY";
-            case SatelliteManagerWrapper.SATELLITE_RESULT_ILLEGAL_STATE:
-                return "SATELLITE_RESULT_ILLEGAL_STATE";
             default:
                 return "INVALID CODE: " + result;
         }