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/main | |
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/main')
-rw-r--r-- | java/com/android/dialer/main/impl/OldMainActivityPeer.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/java/com/android/dialer/main/impl/OldMainActivityPeer.java b/java/com/android/dialer/main/impl/OldMainActivityPeer.java index 402edb3d5..a0e92a8cb 100644 --- a/java/com/android/dialer/main/impl/OldMainActivityPeer.java +++ b/java/com/android/dialer/main/impl/OldMainActivityPeer.java @@ -372,7 +372,7 @@ public class OldMainActivityPeer implements MainActivityPeer, FragmentUtilListen private static boolean isVoicemailAvailable( Context context, PhoneAccountHandle defaultUserSelectedAccount) { - if (!TelecomUtil.hasReadPhoneStatePermission(context)) { + if (!PermissionsUtil.hasReadPhoneStatePermissions(context)) { LogUtil.i( "OldMainActivityPeer.isVoicemailAvailable", "No read phone permisison or not the default dialer."); |