From 5dd30438fd3e4384b57cef3c7606ec20fad9b50d Mon Sep 17 00:00:00 2001 From: zachh Date: Thu, 11 Jan 2018 13:43:45 -0800 Subject: Minor polish and bugfixes for new call log. -Reuse the adapter in NewCallLogFragment to maintain position in call log when returning from call details -Convert empty values to null when invoking ContactPhotoManager to make photos appear correctly in new call log bottom sheets -Added CallLogContactTypes to be shared between proper call log and bottom sheets -Fixed vertical alignment in bottom sheet when there was no secondary text -Added some verbose logging Test: unit and manual PiperOrigin-RevId: 181655715 Change-Id: I9c1a42ad8d529ee2327e15fcc1d32b7b83982014 --- .../datasources/phonelookup/PhoneLookupDataSource.java | 13 +++++++++++++ 1 file changed, 13 insertions(+) (limited to 'java/com/android/dialer/calllog/datasources/phonelookup') diff --git a/java/com/android/dialer/calllog/datasources/phonelookup/PhoneLookupDataSource.java b/java/com/android/dialer/calllog/datasources/phonelookup/PhoneLookupDataSource.java index 508191cde..dca992789 100644 --- a/java/com/android/dialer/calllog/datasources/phonelookup/PhoneLookupDataSource.java +++ b/java/com/android/dialer/calllog/datasources/phonelookup/PhoneLookupDataSource.java @@ -141,6 +141,13 @@ public final class PhoneLookupDataSource */ @Override public ListenableFuture fill(Context appContext, CallLogMutations mutations) { + LogUtil.v( + "PhoneLookupDataSource.fill", + "processing mutations (inserts: %d, updates: %d, deletes: %d)", + mutations.getInserts().size(), + mutations.getUpdates().size(), + mutations.getDeletes().size()); + // Clear state saved since the last call to fill. This is necessary in case fill is called but // onSuccessfulFill is not called during a previous flow. phoneLookupHistoryRowsToUpdate.clear(); @@ -224,6 +231,12 @@ public final class PhoneLookupDataSource rowsToUpdateFuture, rowsToUpdate -> { updateMutations(rowsToUpdate, mutations); + LogUtil.v( + "PhoneLookupDataSource.fill", + "updated mutations (inserts: %d, updates: %d, deletes: %d)", + mutations.getInserts().size(), + mutations.getUpdates().size(), + mutations.getDeletes().size()); return null; }, lightweightExecutorService); -- cgit v1.2.3