From b29f58262aef174c0b51c14c7b764e5ee934c67f Mon Sep 17 00:00:00 2001 From: zachh Date: Mon, 4 Dec 2017 17:53:27 -0800 Subject: Added PhoneLookupHistoryRecorder. When a call is added in InCallUi, it fetches the current PhoneLookupInfo for the call and writes it to PhoneLookupHistory. Required updating PhoneLookupHistoryContentProvider#update to use "replace" to (presumably) atomically insert when a row is missing. Bug: 34672501 Test: unit PiperOrigin-RevId: 177896892 Change-Id: I43f9ded240a81156722be816a9635d586de642a1 --- java/com/android/incallui/InCallPresenter.java | 2 + .../incallui/PhoneLookupHistoryRecorder.java | 116 +++++++++++++++++++++ 2 files changed, 118 insertions(+) create mode 100644 java/com/android/incallui/PhoneLookupHistoryRecorder.java (limited to 'java/com/android/incallui') diff --git a/java/com/android/incallui/InCallPresenter.java b/java/com/android/incallui/InCallPresenter.java index c5310b969..f8605ae7c 100644 --- a/java/com/android/incallui/InCallPresenter.java +++ b/java/com/android/incallui/InCallPresenter.java @@ -545,6 +545,8 @@ public class InCallPresenter implements CallList.Listener, AudioModeProvider.Aud // Since a call has been added we are no longer waiting for Telecom to send us a call. setBoundAndWaitingForOutgoingCall(false, null); call.registerCallback(mCallCallback); + // TODO(maxwelb): Return the future in recordPhoneLookupInfo and propagate. + PhoneLookupHistoryRecorder.recordPhoneLookupInfo(mContext.getApplicationContext(), call); Trace.endSection(); } diff --git a/java/com/android/incallui/PhoneLookupHistoryRecorder.java b/java/com/android/incallui/PhoneLookupHistoryRecorder.java new file mode 100644 index 000000000..2632e6515 --- /dev/null +++ b/java/com/android/incallui/PhoneLookupHistoryRecorder.java @@ -0,0 +1,116 @@ +/* + * Copyright (C) 2017 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.incallui; + +import android.content.ContentValues; +import android.content.Context; +import android.support.annotation.Nullable; +import android.telecom.Call; +import android.telecom.PhoneAccountHandle; +import android.telephony.PhoneNumberUtils; +import android.telephony.SubscriptionInfo; +import android.text.TextUtils; +import com.android.dialer.buildtype.BuildType; +import com.android.dialer.common.Assert; +import com.android.dialer.common.LogUtil; +import com.android.dialer.common.concurrent.DialerExecutors; +import com.android.dialer.location.CountryDetector; +import com.android.dialer.phonelookup.PhoneLookupComponent; +import com.android.dialer.phonelookup.PhoneLookupInfo; +import com.android.dialer.phonelookup.database.contract.PhoneLookupHistoryContract.PhoneLookupHistory; +import com.android.dialer.telecom.TelecomUtil; +import com.android.incallui.util.TelecomCallUtil; +import com.google.common.base.Optional; +import com.google.common.util.concurrent.FutureCallback; +import com.google.common.util.concurrent.Futures; +import com.google.common.util.concurrent.ListenableFuture; +import java.util.Locale; + +/** + * Fetches the current {@link PhoneLookupInfo} for the provided call and writes it to the + * PhoneLookupHistory. + */ +final class PhoneLookupHistoryRecorder { + + /** + * If the new UI is enabled, fetches the current {@link PhoneLookupInfo} for the provided call and + * writes it to the PhoneLookupHistory. Otherwise does nothing. + */ + static void recordPhoneLookupInfo(Context appContext, Call call) { + if (!(BuildType.get() == BuildType.BUGFOOD || LogUtil.isDebugEnabled())) { + return; + } + ListenableFuture future = + PhoneLookupComponent.get(appContext).phoneLookup().lookup(call); + Futures.addCallback( + future, + new FutureCallback() { + @Override + public void onSuccess(@Nullable PhoneLookupInfo result) { + Assert.checkArgument(result != null); + Optional normalizedNumber = getNormalizedNumber(appContext, call); + if (!normalizedNumber.isPresent()) { + LogUtil.w("PhoneLookupHistoryRecorder.onSuccess", "couldn't get a number"); + return; + } + ContentValues contentValues = new ContentValues(); + contentValues.put(PhoneLookupHistory.PHONE_LOOKUP_INFO, result.toByteArray()); + contentValues.put(PhoneLookupHistory.LAST_MODIFIED, System.currentTimeMillis()); + appContext + .getContentResolver() + .update( + PhoneLookupHistory.CONTENT_URI + .buildUpon() + .appendEncodedPath(normalizedNumber.get()) + .build(), + contentValues, + null, + null); + } + + @Override + public void onFailure(Throwable t) { + // TODO(zachh): Consider how to best handle this; take measures to repair call log? + LogUtil.w( + "PhoneLookupHistoryRecorder.onFailure", "could not write PhoneLookupHistory", t); + } + }, + DialerExecutors.getLowPriorityThreadPool(appContext)); + } + + private static Optional getNormalizedNumber(Context appContext, Call call) { + PhoneAccountHandle phoneAccountHandle = call.getDetails().getAccountHandle(); + Optional subscriptionInfo = + TelecomUtil.getSubscriptionInfo(appContext, phoneAccountHandle); + String countryCode = + subscriptionInfo.isPresent() + ? subscriptionInfo.get().getCountryIso() + : CountryDetector.getInstance(appContext).getCurrentCountryIso(); + if (countryCode == null) { + LogUtil.w( + "PhoneLookupHistoryRecorder.getNormalizedNumber", + "couldn't find a country code for call"); + countryCode = "US"; + } + String rawNumber = TelecomCallUtil.getNumber(call); + if (TextUtils.isEmpty(rawNumber)) { + return Optional.absent(); + } + String normalizedNumber = + PhoneNumberUtils.formatNumberToE164(rawNumber, countryCode.toUpperCase(Locale.US)); + return normalizedNumber == null ? Optional.of(rawNumber) : Optional.of(normalizedNumber); + } +} -- cgit v1.2.3