From b494ce8c015cf54a28a47f44791dc95ea6c25aad Mon Sep 17 00:00:00 2001 From: zachh Date: Wed, 17 Jan 2018 15:32:03 -0800 Subject: Support deleting PhoneLookupHistory rows with empty phone numbers. Test: unit PiperOrigin-RevId: 182280850 Change-Id: I51a89f50d56af344425d5f67c6bcaa3928b457e9 --- .../phonelookup/database/PhoneLookupHistoryContentProvider.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'java/com/android/dialer/phonelookup') diff --git a/java/com/android/dialer/phonelookup/database/PhoneLookupHistoryContentProvider.java b/java/com/android/dialer/phonelookup/database/PhoneLookupHistoryContentProvider.java index e20e81369..1c20b10e9 100644 --- a/java/com/android/dialer/phonelookup/database/PhoneLookupHistoryContentProvider.java +++ b/java/com/android/dialer/phonelookup/database/PhoneLookupHistoryContentProvider.java @@ -30,7 +30,6 @@ import android.net.Uri; import android.support.annotation.IntDef; import android.support.annotation.NonNull; import android.support.annotation.Nullable; -import android.text.TextUtils; import com.android.dialer.common.Assert; import com.android.dialer.common.LogUtil; import com.android.dialer.phonelookup.database.contract.PhoneLookupHistoryContract; @@ -190,7 +189,8 @@ public class PhoneLookupHistoryContentProvider extends ContentProvider { Assert.checkArgument( selectionArgs == null, "Do not specify selection args when deleting by number"); String number = Uri.decode(uri.getQueryParameter(PhoneLookupHistory.NUMBER_QUERY_PARAM)); - Assert.checkArgument(!TextUtils.isEmpty(number), "error parsing number from uri: %s", uri); + Assert.checkArgument( + number != null, "error parsing number from uri: %s", LogUtil.sanitizePii(uri)); selection = PhoneLookupHistory.NORMALIZED_NUMBER + "= ?"; selectionArgs = new String[] {number}; break; -- cgit v1.2.3