From 1ab339cf51698fe43344becf99f7c2500a2d5eaf Mon Sep 17 00:00:00 2001 From: linyuh Date: Mon, 23 Apr 2018 12:48:31 -0700 Subject: Implement CnapPhoneLookup Bug: 70988915 Test: CnapPhoneLookupTest PiperOrigin-RevId: 193968830 Change-Id: I7f0c8468472e831699e56e399060067286e3ea0d --- .../dialer/phonelookup/PhoneLookupModule.java | 3 + .../dialer/phonelookup/cnap/CnapPhoneLookup.java | 160 +++++++++++++++++++++ .../dialer/phonelookup/phone_lookup_info.proto | 11 +- 3 files changed, 173 insertions(+), 1 deletion(-) create mode 100644 java/com/android/dialer/phonelookup/cnap/CnapPhoneLookup.java (limited to 'java/com/android/dialer/phonelookup') diff --git a/java/com/android/dialer/phonelookup/PhoneLookupModule.java b/java/com/android/dialer/phonelookup/PhoneLookupModule.java index 273a2d092..86e6991c1 100644 --- a/java/com/android/dialer/phonelookup/PhoneLookupModule.java +++ b/java/com/android/dialer/phonelookup/PhoneLookupModule.java @@ -17,6 +17,7 @@ package com.android.dialer.phonelookup; import com.android.dialer.phonelookup.blockednumber.SystemBlockedNumberPhoneLookup; +import com.android.dialer.phonelookup.cnap.CnapPhoneLookup; import com.android.dialer.phonelookup.cp2.Cp2DefaultDirectoryPhoneLookup; import com.android.dialer.phonelookup.cp2.Cp2ExtendedDirectoryPhoneLookup; import com.android.dialer.phonelookup.spam.SpamPhoneLookup; @@ -31,11 +32,13 @@ public abstract class PhoneLookupModule { @Provides @SuppressWarnings({"unchecked", "rawtype"}) static ImmutableList providePhoneLookupList( + CnapPhoneLookup cnapPhoneLookup, Cp2DefaultDirectoryPhoneLookup cp2DefaultDirectoryPhoneLookup, Cp2ExtendedDirectoryPhoneLookup cp2ExtendedDirectoryPhoneLookup, SystemBlockedNumberPhoneLookup systemBlockedNumberPhoneLookup, SpamPhoneLookup spamPhoneLookup) { return ImmutableList.of( + cnapPhoneLookup, cp2DefaultDirectoryPhoneLookup, cp2ExtendedDirectoryPhoneLookup, systemBlockedNumberPhoneLookup, diff --git a/java/com/android/dialer/phonelookup/cnap/CnapPhoneLookup.java b/java/com/android/dialer/phonelookup/cnap/CnapPhoneLookup.java new file mode 100644 index 000000000..db7c210fc --- /dev/null +++ b/java/com/android/dialer/phonelookup/cnap/CnapPhoneLookup.java @@ -0,0 +1,160 @@ +/* + * Copyright (C) 2018 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License + */ + +package com.android.dialer.phonelookup.cnap; + +import android.content.Context; +import android.database.Cursor; +import android.telecom.Call; +import android.text.TextUtils; +import com.android.dialer.DialerPhoneNumber; +import com.android.dialer.common.Assert; +import com.android.dialer.common.LogUtil; +import com.android.dialer.common.concurrent.Annotations.BackgroundExecutor; +import com.android.dialer.common.database.Selection; +import com.android.dialer.inject.ApplicationContext; +import com.android.dialer.phonelookup.PhoneLookup; +import com.android.dialer.phonelookup.PhoneLookupInfo; +import com.android.dialer.phonelookup.PhoneLookupInfo.CnapInfo; +import com.android.dialer.phonelookup.database.contract.PhoneLookupHistoryContract.PhoneLookupHistory; +import com.google.common.collect.ImmutableMap; +import com.google.common.collect.ImmutableSet; +import com.google.common.util.concurrent.Futures; +import com.google.common.util.concurrent.ListenableFuture; +import com.google.common.util.concurrent.ListeningExecutorService; +import com.google.protobuf.InvalidProtocolBufferException; +import javax.inject.Inject; + +/** PhoneLookup implementation for CNAP info. */ +public final class CnapPhoneLookup implements PhoneLookup { + + private final Context appContext; + private final ListeningExecutorService backgroundExecutorService; + + @Inject + CnapPhoneLookup( + @ApplicationContext Context appContext, + @BackgroundExecutor ListeningExecutorService backgroundExecutorService) { + this.appContext = appContext; + this.backgroundExecutorService = backgroundExecutorService; + } + + /** + * Override the default implementation in {@link PhoneLookup#lookup(Context, Call)} as CNAP info + * is in the provided {@link Call}. + */ + @Override + public ListenableFuture lookup(Context appContext, Call call) { + String callerDisplayName = call.getDetails().getCallerDisplayName(); + return Futures.immediateFuture( + TextUtils.isEmpty(callerDisplayName) + ? CnapInfo.getDefaultInstance() + : CnapInfo.newBuilder().setName(callerDisplayName).build()); + } + + /** + * CNAP info cannot be retrieved when all we have is a number. The best we can do is returning the + * existing info in {@link PhoneLookupHistory}. + */ + @Override + public ListenableFuture lookup(DialerPhoneNumber dialerPhoneNumber) { + return backgroundExecutorService.submit( + () -> { + Selection selection = + Selection.builder() + .and( + Selection.column(PhoneLookupHistory.NORMALIZED_NUMBER) + .is("=", dialerPhoneNumber.getNormalizedNumber())) + .build(); + + try (Cursor cursor = + appContext + .getContentResolver() + .query( + PhoneLookupHistory.CONTENT_URI, + new String[] {PhoneLookupHistory.PHONE_LOOKUP_INFO}, + selection.getSelection(), + selection.getSelectionArgs(), + /* sortOrder = */ null)) { + if (cursor == null) { + LogUtil.e("CnapPhoneLookup.lookup", "null cursor"); + return CnapInfo.getDefaultInstance(); + } + + if (!cursor.moveToFirst()) { + LogUtil.i("CnapPhoneLookup.lookup", "empty cursor"); + return CnapInfo.getDefaultInstance(); + } + + // At ths point, we expect only one row in the cursor as + // PhoneLookupHistory.NORMALIZED_NUMBER is the primary key of table PhoneLookupHistory. + Assert.checkState(cursor.getCount() == 1); + + int phoneLookupInfoColumn = + cursor.getColumnIndexOrThrow(PhoneLookupHistory.PHONE_LOOKUP_INFO); + PhoneLookupInfo phoneLookupInfo; + try { + phoneLookupInfo = PhoneLookupInfo.parseFrom(cursor.getBlob(phoneLookupInfoColumn)); + } catch (InvalidProtocolBufferException e) { + throw new IllegalStateException(e); + } + + return phoneLookupInfo.getCnapInfo(); + } + }); + } + + @Override + public ListenableFuture isDirty(ImmutableSet phoneNumbers) { + return Futures.immediateFuture(false); + } + + @Override + public ListenableFuture> getMostRecentInfo( + ImmutableMap existingInfoMap) { + return Futures.immediateFuture(existingInfoMap); + } + + @Override + public void setSubMessage(PhoneLookupInfo.Builder destination, CnapInfo subMessage) { + destination.setCnapInfo(subMessage); + } + + @Override + public CnapInfo getSubMessage(PhoneLookupInfo phoneLookupInfo) { + return phoneLookupInfo.getCnapInfo(); + } + + @Override + public ListenableFuture onSuccessfulBulkUpdate() { + return Futures.immediateFuture(null); + } + + @Override + public void registerContentObservers() { + // No content observers for CNAP info. + } + + @Override + public void unregisterContentObservers() { + // No content observers for CNAP info. + } + + @Override + public ListenableFuture clearData() { + return Futures.immediateFuture(null); + } +} diff --git a/java/com/android/dialer/phonelookup/phone_lookup_info.proto b/java/com/android/dialer/phonelookup/phone_lookup_info.proto index 96121885f..1beff6ca9 100644 --- a/java/com/android/dialer/phonelookup/phone_lookup_info.proto +++ b/java/com/android/dialer/phonelookup/phone_lookup_info.proto @@ -13,7 +13,8 @@ package com.android.dialer.phonelookup; // to an implementation of PhoneLookup. For example, field // "cp2_info_in_default_directory" corresponds to class // Cp2DefaultDirectoryPhoneLookup, and class Cp2DefaultDirectoryPhoneLookup -// alone is responsible for populating it. Next ID: 7 +// alone is responsible for populating it. +// Next ID: 8 message PhoneLookupInfo { // Information about a PhoneNumber retrieved from CP2. message Cp2Info { @@ -154,4 +155,12 @@ message PhoneLookupInfo { optional BlockedState blocked_state = 1; } optional SystemBlockedNumberInfo system_blocked_number_info = 4; + + // Information obtained via CNAP + // (https://en.wikipedia.org/wiki/Calling_Name_Presentation) + // Next ID: 2 + message CnapInfo { + optional string name = 1; + } + optional CnapInfo cnap_info = 7; } \ No newline at end of file -- cgit v1.2.3