diff options
author | Linux Build Service Account <lnxbuild@localhost> | 2015-03-16 02:45:59 -0700 |
---|---|---|
committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2015-03-16 02:45:59 -0700 |
commit | 075bfe6f63c6d20735bed69049121d6029947c9c (patch) | |
tree | 12053e4fccf8ee5d6118ab8b4ffe5f1e09364669 | |
parent | 294ad96d5af7849dba01f49f233ae589fc3080de (diff) | |
parent | eedd00782207267f45f364470e7bd8808fbe0938 (diff) | |
download | android_packages_apps_Dialer-075bfe6f63c6d20735bed69049121d6029947c9c.tar.gz android_packages_apps_Dialer-075bfe6f63c6d20735bed69049121d6029947c9c.tar.bz2 android_packages_apps_Dialer-075bfe6f63c6d20735bed69049121d6029947c9c.zip |
Merge "Fix potential NPE when generating accessibility text."
-rw-r--r-- | src/com/android/dialer/calllog/CallLogListItemHelper.java | 19 |
1 files changed, 15 insertions, 4 deletions
diff --git a/src/com/android/dialer/calllog/CallLogListItemHelper.java b/src/com/android/dialer/calllog/CallLogListItemHelper.java index f4ea07e84..4e30cfdef 100644 --- a/src/com/android/dialer/calllog/CallLogListItemHelper.java +++ b/src/com/android/dialer/calllog/CallLogListItemHelper.java @@ -20,6 +20,7 @@ import android.content.Context; import android.content.res.Resources; import android.provider.CallLog.Calls; import android.text.TextUtils; +import android.util.Log; import com.android.contacts.common.CallUtil; import com.android.dialer.PhoneCallDetails; @@ -30,6 +31,8 @@ import com.android.dialer.R; * Helper class to fill in the views of a call log entry. */ /* package */class CallLogListItemHelper { + private static final String TAG = "CallLogListItemHelper"; + /** Helper for populating the details of a phone call. */ private final PhoneCallDetailsHelper mPhoneCallDetailsHelper; /** Helper for handling phone numbers. */ @@ -78,17 +81,25 @@ import com.android.dialer.R; * @param views The views associated with the current call log entry. */ public void setActionContentDescriptions(CallLogListItemViews views) { + if (views.nameOrNumber == null) { + Log.e(TAG, "setActionContentDescriptions; name or number is null."); + } + + // Calling expandTemplate with a null parameter will cause a NullPointerException. + // Although we don't expect a null name or number, it is best to protect against it. + CharSequence nameOrNumber = views.nameOrNumber == null ? "" : views.nameOrNumber; + views.callBackButtonView.setContentDescription( - mResources.getString(R.string.description_call_back_action, views.nameOrNumber)); + mResources.getString(R.string.description_call_back_action, nameOrNumber)); views.videoCallButtonView.setContentDescription( - mResources.getString(R.string.description_video_call_action, views.nameOrNumber)); + mResources.getString(R.string.description_video_call_action, nameOrNumber)); views.voicemailButtonView.setContentDescription( - mResources.getString(R.string.description_voicemail_action, views.nameOrNumber)); + mResources.getString(R.string.description_voicemail_action, nameOrNumber)); views.detailsButtonView.setContentDescription( - mResources.getString(R.string.description_details_action, views.nameOrNumber)); + mResources.getString(R.string.description_details_action, nameOrNumber)); } /** |