diff options
author | Treehugger Robot <treehugger-gerrit@google.com> | 2018-01-11 23:52:26 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2018-01-11 23:52:26 +0000 |
commit | 58bade2ace277dff4db3bbbfa2e07015f0f7c6ad (patch) | |
tree | 88259dfd2edf1eefdbb367cd93bc7dfd6a7db548 /java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java | |
parent | 85dcd85f0c90d3a7ff45dcbc8f18c1b0da1ebc65 (diff) | |
parent | 4ee24a9492c7c83952f59ecc54071c31aa68fa86 (diff) |
Merge "Fixed bug in handling of empty numbers in new call log."
Diffstat (limited to 'java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java')
-rw-r--r-- | java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java b/java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java index 34f35311f..68695b7e9 100644 --- a/java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java +++ b/java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java @@ -34,6 +34,7 @@ import com.google.common.collect.Maps; import com.google.common.util.concurrent.Futures; import com.google.common.util.concurrent.ListenableFuture; import com.google.common.util.concurrent.ListeningExecutorService; +import com.google.common.util.concurrent.MoreExecutors; import java.util.ArrayList; import java.util.List; import java.util.Map; @@ -88,7 +89,18 @@ public final class CompositePhoneLookup implements PhoneLookup<PhoneLookupInfo> public ListenableFuture<Boolean> isDirty(ImmutableSet<DialerPhoneNumber> phoneNumbers) { List<ListenableFuture<Boolean>> futures = new ArrayList<>(); for (PhoneLookup<?> phoneLookup : phoneLookups) { - futures.add(phoneLookup.isDirty(phoneNumbers)); + futures.add( + Futures.transform( + phoneLookup.isDirty(phoneNumbers), + isDirty -> { + LogUtil.v( + "CompositePhoneLookup.isDirty", + "isDirty for %s: %b", + phoneLookup.getClass().getSimpleName(), + isDirty); + return isDirty; + }, + MoreExecutors.directExecutor())); } // Executes all child lookups (possibly in parallel), completing when the first composite lookup // which returns "true" completes, and cancels the others. |