diff options
author | twyen <twyen@google.com> | 2018-05-21 13:05:30 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-05-21 14:17:19 -0700 |
commit | 857de2fe89bd9daa3fa836998568208d7c8ce575 (patch) | |
tree | 90263793ac167c4588f8d462d431d2a8bcfc1392 /java/com/android/dialer/telecom | |
parent | 4791039e1e62ff6fc22f063dbf363fb6864c87ad (diff) |
Check READ_PHONE_STATE for isVoicemailAvailable()
TelecomUtil.has*Permission() pass if the dialer is the default dialer. In reality most API does not allow the default dialer do whatever they want. PermissionUtil should be used instead.
TEST=uabdullah please fix your "Temporary" @Ignore
Bug: 77907588
Test: uabdullah please fix your "Temporary" @Ignore
PiperOrigin-RevId: 197442842
Change-Id: I1e6f275e2b8e6d3cfa54ddb8cd24f7d47f9462ca
Diffstat (limited to 'java/com/android/dialer/telecom')
-rw-r--r-- | java/com/android/dialer/telecom/TelecomUtil.java | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/java/com/android/dialer/telecom/TelecomUtil.java b/java/com/android/dialer/telecom/TelecomUtil.java index 8c0090928..c08814688 100644 --- a/java/com/android/dialer/telecom/TelecomUtil.java +++ b/java/com/android/dialer/telecom/TelecomUtil.java @@ -275,15 +275,21 @@ public abstract class TelecomUtil { && hasPermission(context, Manifest.permission.WRITE_VOICEMAIL)); } + /** @deprecated use {@link com.android.dialer.util.PermissionsUtil} */ + @Deprecated public static boolean hasModifyPhoneStatePermission(Context context) { return isDefaultDialer(context) || hasPermission(context, Manifest.permission.MODIFY_PHONE_STATE); } + /** @deprecated use {@link com.android.dialer.util.PermissionsUtil} */ + @Deprecated public static boolean hasReadPhoneStatePermission(Context context) { return isDefaultDialer(context) || hasPermission(context, Manifest.permission.READ_PHONE_STATE); } + /** @deprecated use {@link com.android.dialer.util.PermissionsUtil} */ + @Deprecated public static boolean hasCallPhonePermission(Context context) { return isDefaultDialer(context) || hasPermission(context, Manifest.permission.CALL_PHONE); } |