diff options
author | calderwoodra <calderwoodra@google.com> | 2018-05-01 11:01:37 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-05-02 16:36:45 -0700 |
commit | 3eb6fd18fe64c02f0433de75bec7cfca9d231e62 (patch) | |
tree | 85d192a03bc5d506267c39653866954550a437dc /java/com/android/dialer/speeddial/SpeedDialFragment.java | |
parent | 4a65002a1619ec70157b3c274a501286134cc7d4 (diff) |
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
Diffstat (limited to 'java/com/android/dialer/speeddial/SpeedDialFragment.java')
-rw-r--r-- | java/com/android/dialer/speeddial/SpeedDialFragment.java | 2 |
1 files changed, 1 insertions, 1 deletions
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 |