From 3eb6fd18fe64c02f0433de75bec7cfca9d231e62 Mon Sep 17 00:00:00 2001 From: calderwoodra Date: Tue, 1 May 2018 11:01:37 -0700 Subject: Fixed speed dial remove. App would crash when users attempted to remove a starred contact because we try to update Phone.CONTENT_URI, which is not allowed. Test assumed that we could update PHone.CONTENT_URI, which is why this wasn't caught. Updated the test to verify the contact in Contacts.CONTENT_URI get updated to starred = 0. Unforunately, since fakeContentProvider doesn't work exactly like cp2, the update won't propogate to Phone.CONTENT_URI, so out test is less than ideal, but it should cover our bases. Also updated the image and text for starring a suggestion. Bug: 78923466 Test: tap PiperOrigin-RevId: 194962080 Change-Id: I9f8f71d70ba83510682890917081380a3c51cd7a --- java/com/android/dialer/speeddial/SpeedDialFragment.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'java/com/android/dialer/speeddial/SpeedDialFragment.java') diff --git a/java/com/android/dialer/speeddial/SpeedDialFragment.java b/java/com/android/dialer/speeddial/SpeedDialFragment.java index 17591aa6c..124470617 100644 --- a/java/com/android/dialer/speeddial/SpeedDialFragment.java +++ b/java/com/android/dialer/speeddial/SpeedDialFragment.java @@ -399,7 +399,7 @@ public class SpeedDialFragment extends Fragment { @Override public int getDrawableId() { - return R.drawable.context_menu_contact_icon; + return R.drawable.quantum_ic_star_vd_theme_24; } @Override -- cgit v1.2.3