diff options
author | twyen <twyen@google.com> | 2018-06-05 11:18:37 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-06-05 12:57:09 -0700 |
commit | ef7ac4084e9d396e2b96a07faaccd232eca15648 (patch) | |
tree | 5ae162bfee7c99672f47a09a0969808409e9c31a /java/com/android/dialer/dialpadview | |
parent | 9337f657f6c2890fd3d043b943da3234a7fa2d1f (diff) |
Handle null carrier config for emergency call hint
getCarrierConfig() is @Nullable. While the default implementation always return a bundle, this is not true for other carrier apps.
ShadowTelephonyManager.getCarrierConfig() now return null as default to catch this behavior.
TEST=TAP
Bug: 80629471
Test: TAP
PiperOrigin-RevId: 199327865
Change-Id: I7416bee3d5f5307919c6825fc74c26f25b6b001f
Diffstat (limited to 'java/com/android/dialer/dialpadview')
-rw-r--r-- | java/com/android/dialer/dialpadview/DialpadFragment.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/java/com/android/dialer/dialpadview/DialpadFragment.java b/java/com/android/dialer/dialpadview/DialpadFragment.java index 8fb7533f4..2d0eff543 100644 --- a/java/com/android/dialer/dialpadview/DialpadFragment.java +++ b/java/com/android/dialer/dialpadview/DialpadFragment.java @@ -38,6 +38,7 @@ import android.net.Uri; import android.os.Build.VERSION; import android.os.Build.VERSION_CODES; import android.os.Bundle; +import android.os.PersistableBundle; import android.os.Trace; import android.provider.Contacts.People; import android.provider.Contacts.Phones; @@ -501,9 +502,9 @@ public class DialpadFragment extends Fragment return false; } TelephonyManager telephonyManager = context.getSystemService(TelephonyManager.class); + PersistableBundle config = telephonyManager.getCarrierConfig(); // A delay of -1 means wifi emergency call is available/the warning is not required. - if (telephonyManager.getCarrierConfig().getInt(KEY_EMERGENCY_NOTIFICATION_DELAY_INT, -1) - == -1) { + if (config == null || config.getInt(KEY_EMERGENCY_NOTIFICATION_DELAY_INT, -1) == -1) { return false; } |