summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorandroid-build-prod (mdb) <android-build-team-robot@google.com>2018-05-04 03:03:38 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2018-05-04 03:03:38 +0000
commit6e1516d52539f2d41e57443249ca543d2481cce3 (patch)
tree85c68bb5a034732405f2036ba1e4988d943b4766
parent2a82028eebeaa37bc5b2ec975585d7c920e95321 (diff)
parent3eb6fd18fe64c02f0433de75bec7cfca9d231e62 (diff)
Merge "Fixed speed dial remove."
-rw-r--r--java/com/android/dialer/speeddial/SpeedDialFragment.java2
-rw-r--r--java/com/android/dialer/speeddial/loader/SpeedDialUiItemMutator.java18
-rw-r--r--java/com/android/dialer/speeddial/res/values/strings.xml2
3 files changed, 6 insertions, 16 deletions
diff --git a/java/com/android/dialer/speeddial/SpeedDialFragment.java b/java/com/android/dialer/speeddial/SpeedDialFragment.java
index c1745990d..702472c6a 100644
--- a/java/com/android/dialer/speeddial/SpeedDialFragment.java
+++ b/java/com/android/dialer/speeddial/SpeedDialFragment.java
@@ -402,7 +402,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
diff --git a/java/com/android/dialer/speeddial/loader/SpeedDialUiItemMutator.java b/java/com/android/dialer/speeddial/loader/SpeedDialUiItemMutator.java
index e8892c431..0a2a241b3 100644
--- a/java/com/android/dialer/speeddial/loader/SpeedDialUiItemMutator.java
+++ b/java/com/android/dialer/speeddial/loader/SpeedDialUiItemMutator.java
@@ -130,11 +130,8 @@ public final class SpeedDialUiItemMutator {
private ImmutableList<SpeedDialUiItem> removeSpeedDialUiItemInternal(
SpeedDialUiItem speedDialUiItem) {
Assert.isWorkerThread();
- if (speedDialUiItem.isStarred()) {
- removeStarredSpeedDialUiItem(speedDialUiItem);
- } else {
- removeSuggestedSpeedDialUiItem(speedDialUiItem);
- }
+ Assert.checkArgument(speedDialUiItem.isStarred());
+ removeStarredSpeedDialUiItem(speedDialUiItem);
return loadSpeedDialUiItemsInternal();
}
@@ -175,19 +172,12 @@ public final class SpeedDialUiItemMutator {
appContext
.getContentResolver()
.update(
- Phone.CONTENT_URI,
+ Contacts.CONTENT_URI,
contentValues,
- Phone.CONTACT_ID + " = ?",
+ Contacts._ID + " = ?",
new String[] {Long.toString(speedDialUiItem.contactId())});
}
- @WorkerThread
- @SuppressWarnings("unused")
- private void removeSuggestedSpeedDialUiItem(SpeedDialUiItem speedDialUiItem) {
- Assert.isWorkerThread();
- // TODO(calderwoodra): remove strequent contact
- }
-
/**
* Takes a contact uri from {@link Phone#CONTENT_URI} and updates {@link Phone#STARRED} to be
* true, if it isn't already or Inserts the contact into the {@link SpeedDialEntryDatabaseHelper}
diff --git a/java/com/android/dialer/speeddial/res/values/strings.xml b/java/com/android/dialer/speeddial/res/values/strings.xml
index 028f6a90c..d2f5371b8 100644
--- a/java/com/android/dialer/speeddial/res/values/strings.xml
+++ b/java/com/android/dialer/speeddial/res/values/strings.xml
@@ -57,5 +57,5 @@
<string name="contact_menu_contact_info">Contact info</string>
<!-- Text shown to the user in a button to prompt them to mark a contact as a favorite contact [CHAR LIMIT=30]. -->
- <string name="suggested_contact_bottom_sheet_add_favorite_option">Add to favorites</string>
+ <string name="suggested_contact_bottom_sheet_add_favorite_option">Add favorite</string>
</resources> \ No newline at end of file