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

1 files changed, 13 insertions, 0 deletions

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<Void> 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);