summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Madhav <madhavadas@google.com> 2025-02-28 01:02:52 +0000
committer Madhav <madhavadas@google.com> 2025-03-11 19:52:29 +0000
commit06580ed2adb786e8007c7a4db3106456680a172d (patch)
treefd1ded31dad543505eaa945e1e227b74e5ad05c0
parent3650f2c5051fe0489cf612e1ffe29dd3da4910e6 (diff)
Update getEmergencyNumberList() feature flag dependency
Given that getEmergencyNumberList() functionality is utilized by both calling and messaging modules, and messaging can be enabled without calling, the function's access control should be based on FEATURE_TELEPHONY_CALLING or FEATURE_TELEPHONY_MESSAGING. Bug: 395178686 Flag: EXEMPT bug fix Test: atest CtsTelephonyTestCases --no-bazel-mode Change-Id: I8f33a5e58a7571dc9baa379efc4c1e7bcd3166ca
-rw-r--r--telephony/java/android/telephony/TelephonyManager.java16
1 files changed, 12 insertions, 4 deletions
diff --git a/telephony/java/android/telephony/TelephonyManager.java b/telephony/java/android/telephony/TelephonyManager.java
index 531fe1adcadb..f8797e3904a5 100644
--- a/telephony/java/android/telephony/TelephonyManager.java
+++ b/telephony/java/android/telephony/TelephonyManager.java
@@ -15490,11 +15490,15 @@ public class TelephonyManager {
* or throw a SecurityException if the caller does not have the permission.
*
* @throws UnsupportedOperationException If the device does not have
- * {@link PackageManager#FEATURE_TELEPHONY_CALLING}.
+ * {@link PackageManager#FEATURE_TELEPHONY_CALLING} or
+ * {@link PackageManager#FEATURE_TELEPHONY_MESSAGING}.
*/
@RequiresPermission(android.Manifest.permission.READ_PHONE_STATE)
@NonNull
- @RequiresFeature(PackageManager.FEATURE_TELEPHONY_CALLING)
+ @RequiresFeature(anyOf = {
+ PackageManager.FEATURE_TELEPHONY_CALLING,
+ PackageManager.FEATURE_TELEPHONY_MESSAGING
+ })
public Map<Integer, List<EmergencyNumber>> getEmergencyNumberList() {
Map<Integer, List<EmergencyNumber>> emergencyNumberList = new HashMap<>();
try {
@@ -15548,11 +15552,15 @@ public class TelephonyManager {
* or throw a SecurityException if the caller does not have the permission.
* @throws IllegalStateException if the Telephony process is not currently available.
* @throws UnsupportedOperationException If the device does not have
- * {@link PackageManager#FEATURE_TELEPHONY_CALLING}.
+ * {@link PackageManager#FEATURE_TELEPHONY_CALLING} or
+ * {@link PackageManager#FEATURE_TELEPHONY_MESSAGING}.
*/
@RequiresPermission(android.Manifest.permission.READ_PHONE_STATE)
@NonNull
- @RequiresFeature(PackageManager.FEATURE_TELEPHONY_CALLING)
+ @RequiresFeature(anyOf = {
+ PackageManager.FEATURE_TELEPHONY_CALLING,
+ PackageManager.FEATURE_TELEPHONY_MESSAGING
+ })
public Map<Integer, List<EmergencyNumber>> getEmergencyNumberList(
@EmergencyServiceCategories int categories) {
Map<Integer, List<EmergencyNumber>> emergencyNumberListForCategories = new HashMap<>();