From 99817db9f0369747c22a4e03ad1f1db26c5d81bb Mon Sep 17 00:00:00 2001 From: linyuh Date: Tue, 9 Jan 2018 18:26:39 -0800 Subject: Rename Cp2PhoneLookup as Cp2LocalPhoneLookup and PhoneLookupInfo.cp2_info as PhoneLookupInfo.cp2_local_info. To support remote CP2 contacts, there will be a new PhoneLookup ("Cp2RemotePhoneLookup") and a new field in proto PhoneLookupInfo ("cp2_remote_info"). In proto PhoneLookupInfo, cp2_local_info and cp2_remote_info will be of the same type ("Cp2Info"). Bug: 71763594 Test: Existing tests PiperOrigin-RevId: 181405798 Change-Id: I6c43b486229d4e9ae7b55c579d9c9997a2884c80 --- .../phonelookup/PhoneLookupDataSource.java | 2 +- .../dialer/calllog/ui/RealtimeRowProcessor.java | 20 ++++++++++---------- 2 files changed, 11 insertions(+), 11 deletions(-) (limited to 'java/com/android/dialer/calllog') diff --git a/java/com/android/dialer/calllog/datasources/phonelookup/PhoneLookupDataSource.java b/java/com/android/dialer/calllog/datasources/phonelookup/PhoneLookupDataSource.java index 935ea7406..cdde638ee 100644 --- a/java/com/android/dialer/calllog/datasources/phonelookup/PhoneLookupDataSource.java +++ b/java/com/android/dialer/calllog/datasources/phonelookup/PhoneLookupDataSource.java @@ -584,7 +584,7 @@ public final class PhoneLookupDataSource AnnotatedCallLog.CAN_REPORT_AS_INVALID_NUMBER, PhoneLookupSelector.canReportAsInvalidNumber(phoneLookupInfo)); contentValues.put( - AnnotatedCallLog.CP2_INFO_INCOMPLETE, phoneLookupInfo.getCp2Info().getIsIncomplete()); + AnnotatedCallLog.CP2_INFO_INCOMPLETE, phoneLookupInfo.getCp2LocalInfo().getIsIncomplete()); } private static Uri numberUri(String number) { diff --git a/java/com/android/dialer/calllog/ui/RealtimeRowProcessor.java b/java/com/android/dialer/calllog/ui/RealtimeRowProcessor.java index 814efc779..74b7defb0 100644 --- a/java/com/android/dialer/calllog/ui/RealtimeRowProcessor.java +++ b/java/com/android/dialer/calllog/ui/RealtimeRowProcessor.java @@ -23,7 +23,7 @@ import com.android.dialer.calllog.model.CoalescedRow; import com.android.dialer.common.concurrent.Annotations.Ui; import com.android.dialer.phonelookup.PhoneLookupInfo; import com.android.dialer.phonelookup.PhoneLookupInfo.Cp2Info; -import com.android.dialer.phonelookup.cp2.Cp2PhoneLookup; +import com.android.dialer.phonelookup.cp2.Cp2LocalPhoneLookup; import com.android.dialer.phonelookup.selector.PhoneLookupSelector; import com.google.common.base.Optional; import com.google.common.util.concurrent.Futures; @@ -46,7 +46,7 @@ import javax.inject.Inject; public final class RealtimeRowProcessor { private final ListeningExecutorService uiExecutor; - private final Cp2PhoneLookup cp2PhoneLookup; + private final Cp2LocalPhoneLookup cp2LocalPhoneLookup; private final PhoneLookupSelector phoneLookupSelector; private final Map cache = new ArrayMap<>(); @@ -54,10 +54,10 @@ public final class RealtimeRowProcessor { @Inject RealtimeRowProcessor( @Ui ListeningExecutorService uiExecutor, - Cp2PhoneLookup cp2PhoneLookup, + Cp2LocalPhoneLookup cp2LocalPhoneLookup, PhoneLookupSelector phoneLookupSelector) { this.uiExecutor = uiExecutor; - this.cp2PhoneLookup = cp2PhoneLookup; + this.cp2LocalPhoneLookup = cp2LocalPhoneLookup; this.phoneLookupSelector = phoneLookupSelector; } @@ -67,7 +67,7 @@ public final class RealtimeRowProcessor { */ @MainThread ListenableFuture> applyRealtimeProcessing(final CoalescedRow row) { - // Cp2PhoneLookup can not always efficiently process all rows. + // Cp2LocalPhoneLookup can not always efficiently process all rows. if (!row.cp2InfoIncomplete()) { return Futures.immediateFuture(Optional.absent()); } @@ -77,24 +77,24 @@ public final class RealtimeRowProcessor { if (cachedCp2Info.equals(Cp2Info.getDefaultInstance())) { return Futures.immediateFuture(Optional.absent()); } - return Futures.immediateFuture(Optional.of(applyCp2InfoToRow(cachedCp2Info, row))); + return Futures.immediateFuture(Optional.of(applyCp2LocalInfoToRow(cachedCp2Info, row))); } - ListenableFuture cp2InfoFuture = cp2PhoneLookup.lookupByNumber(row.number()); + ListenableFuture cp2InfoFuture = cp2LocalPhoneLookup.lookupByNumber(row.number()); return Futures.transform( cp2InfoFuture, cp2Info -> { cache.put(row.number(), cp2Info); if (!cp2Info.equals(Cp2Info.getDefaultInstance())) { - return Optional.of(applyCp2InfoToRow(cp2Info, row)); + return Optional.of(applyCp2LocalInfoToRow(cp2Info, row)); } return Optional.absent(); }, uiExecutor /* ensures the cache is updated on a single thread */); } - private CoalescedRow applyCp2InfoToRow(Cp2Info cp2Info, CoalescedRow row) { - PhoneLookupInfo phoneLookupInfo = PhoneLookupInfo.newBuilder().setCp2Info(cp2Info).build(); + private CoalescedRow applyCp2LocalInfoToRow(Cp2Info cp2Info, CoalescedRow row) { + PhoneLookupInfo phoneLookupInfo = PhoneLookupInfo.newBuilder().setCp2LocalInfo(cp2Info).build(); // It is safe to overwrite any existing data because CP2 always has highest priority. return row.toBuilder() .setName(phoneLookupSelector.selectName(phoneLookupInfo)) -- cgit v1.2.3