diff options
author | Nancy Chen <nancychen@google.com> | 2016-01-15 02:55:02 +0000 |
---|---|---|
committer | Nancy Chen <nancychen@google.com> | 2016-01-15 02:55:02 +0000 |
commit | f7e10f2ac8f2fd8999113e6d3bcb731c9a894b86 (patch) | |
tree | d6478e553604cbd844704d03d80708c0951bd23b | |
parent | 201e021fe24e36ae2d94eadd4701933dd099d309 (diff) |
Revert "Use telephony voicemail notification settings for vvm."
This reverts commit 201e021fe24e36ae2d94eadd4701933dd099d309.
Change-Id: I9b8e9bc74d23cea7c668166d275033c8fddf52cf
3 files changed, 1 insertions, 94 deletions
diff --git a/src-N/com/android/dialer/calllog/VoicemailNotificationSettingsLookup.java b/src-N/com/android/dialer/calllog/VoicemailNotificationSettingsLookup.java deleted file mode 100644 index 4ca0c65e6..000000000 --- a/src-N/com/android/dialer/calllog/VoicemailNotificationSettingsLookup.java +++ /dev/null @@ -1,44 +0,0 @@ -/* - * Copyright (C) 2016 The Android Open Source Project - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License - */ -package com.android.dialer.calllog; - -import android.app.Notification; -import android.content.ComponentName; -import android.content.Context; -import android.net.Uri; -import android.telecom.PhoneAccountHandle; -import android.telephony.TelephonyManager; - -import com.android.dialer.calllog.DefaultVoicemailNotifier.NewCall; - -/** - * Class to do lookup for voicemail ringtone and vibration. On N and above, this will look up - * voicemail notification settings from Telephony. - */ -public class VoicemailNotificationSettingsLookup { - public static Uri getVoicemailRingtoneUri(Context context, PhoneAccountHandle accountHandle) { - return getTelephonyManager(context).getVoicemailRingtoneUri(accountHandle); - } - - public static int getNotificationDefaults(Context context, PhoneAccountHandle accountHandle) { - return getTelephonyManager(context).isVoicemailVibrationEnabled(accountHandle) - ? Notification.DEFAULT_VIBRATE : 0; - } - - private static TelephonyManager getTelephonyManager(Context context) { - return (TelephonyManager) context.getSystemService(Context.TELEPHONY_SERVICE); - } -} diff --git a/src-pre-N/com/android/dialer/calllog/VoicemailNotificationSettingsLookup.java b/src-pre-N/com/android/dialer/calllog/VoicemailNotificationSettingsLookup.java deleted file mode 100644 index 5e161f2ba..000000000 --- a/src-pre-N/com/android/dialer/calllog/VoicemailNotificationSettingsLookup.java +++ /dev/null @@ -1,34 +0,0 @@ -/* - * Copyright (C) 2016 The Android Open Source Project - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License - */ -package com.android.dialer.calllog; - -import android.app.Notification; -import android.content.Context; -import android.net.Uri; -import android.telecom.PhoneAccountHandle; - -/** - * Class to do lookup for voicemail ringtone. On pre-N, this is just the system defaults. - */ -public class VoicemailNotificationSettingsLookup { - public static Uri getVoicemailRingtoneUri(Context context, PhoneAccountHandle accountHandle) { - return null; - } - - public static int getNotificationDefaults(Context context, PhoneAccountHandle accountHandle) { - return Notification.DEFAULT_ALL; - } -} diff --git a/src/com/android/dialer/calllog/DefaultVoicemailNotifier.java b/src/com/android/dialer/calllog/DefaultVoicemailNotifier.java index d0156cf7f..2b894a12d 100644 --- a/src/com/android/dialer/calllog/DefaultVoicemailNotifier.java +++ b/src/com/android/dialer/calllog/DefaultVoicemailNotifier.java @@ -24,7 +24,6 @@ import com.google.common.collect.Maps; import android.app.Notification; import android.app.NotificationManager; import android.app.PendingIntent; -import android.content.ComponentName; import android.content.ContentResolver; import android.content.ContentUris; import android.content.Context; @@ -34,7 +33,6 @@ import android.database.Cursor; import android.net.Uri; import android.provider.CallLog.Calls; import android.provider.ContactsContract.PhoneLookup; -import android.telecom.PhoneAccountHandle; import android.telephony.PhoneNumberUtils; import android.text.TextUtils; import android.util.Log; @@ -196,26 +194,13 @@ public class DefaultVoicemailNotifier { // TODO: Use the photo of contact if all calls are from the same person. final int icon = android.R.drawable.stat_notify_voicemail; - Uri ringtoneUri = null; - int notificationDefaults = 0; - if (callToNotify != null) { - PhoneAccountHandle accountHandle = new PhoneAccountHandle( - ComponentName.unflattenFromString(callToNotify.accountComponentName), - callToNotify.accountId); - ringtoneUri = VoicemailNotificationSettingsLookup - .getVoicemailRingtoneUri(mContext, accountHandle); - notificationDefaults = VoicemailNotificationSettingsLookup - .getNotificationDefaults(mContext, accountHandle); - } - Notification.Builder notificationBuilder = new Notification.Builder(mContext) .setSmallIcon(icon) .setContentTitle(title) .setContentText(callers) .setStyle(new Notification.BigTextStyle().bigText(transcription)) .setColor(resources.getColor(R.color.dialer_theme_color)) - .setSound(ringtoneUri) - .setDefaults(notificationDefaults) + .setDefaults(callToNotify != null ? Notification.DEFAULT_ALL : 0) .setDeleteIntent(createMarkNewVoicemailsAsOldIntent()) .setAutoCancel(true); |