summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authortwyen <twyen@google.com>2018-04-30 18:15:02 -0700
committerCopybara-Service <copybara-piper@google.com>2018-05-02 16:36:19 -0700
commit4a65002a1619ec70157b3c274a501286134cc7d4 (patch)
tree9d0dcb6a4f977fdd8092f25e245370e2d11da2af
parentcccd332af6185505101df03ebee11923d7ee5fd4 (diff)
Select SIM when calling back from call log.
Previously the NUI call log call backs with whatever SIM the call was made/received in, which is inconsistent with the old UI. The Old UI behavior should be kept. TEST=TAP Bug: 78291136 Test: TAP PiperOrigin-RevId: 194878167 Change-Id: If9c5adcbed6a194c801d2b558abb45573b97d2ae
-rw-r--r--java/com/android/dialer/calllog/ui/menu/Modules.java16
-rw-r--r--java/com/android/dialer/calllogutils/CallLogRowActions.java4
2 files changed, 3 insertions, 17 deletions
diff --git a/java/com/android/dialer/calllog/ui/menu/Modules.java b/java/com/android/dialer/calllog/ui/menu/Modules.java
index 94da6ec99..2f6c1fb36 100644
--- a/java/com/android/dialer/calllog/ui/menu/Modules.java
+++ b/java/com/android/dialer/calllog/ui/menu/Modules.java
@@ -19,8 +19,6 @@ package com.android.dialer.calllog.ui.menu;
import android.content.Context;
import android.provider.CallLog.Calls;
import android.support.v4.os.BuildCompat;
-import android.telecom.PhoneAccount;
-import android.telecom.PhoneAccountHandle;
import android.text.TextUtils;
import com.android.dialer.blockreportspam.BlockReportSpamDialogInfo;
import com.android.dialer.calldetails.CallDetailsActivity;
@@ -41,7 +39,6 @@ import com.android.dialer.historyitemactions.IntentModule;
import com.android.dialer.historyitemactions.SharedModules;
import com.android.dialer.logging.ReportingLocation;
import com.android.dialer.phonenumberutil.PhoneNumberHelper;
-import com.android.dialer.telecom.TelecomUtil;
import com.android.dialer.util.CallUtil;
import com.google.common.base.Optional;
import java.util.ArrayList;
@@ -134,23 +131,16 @@ final class Modules {
.flattenToString()
.equals(row.getPhoneAccountComponentName());
- // Obtain a PhoneAccountHandle that will be used to start carrier voice/video calls.
- // If the row is for a Duo call, we should use the default phone account as the one included in
- // the row is for Duo only.
- PhoneAccountHandle phoneAccountHandle =
- isDuoCall
- ? TelecomUtil.getDefaultOutgoingPhoneAccount(context, PhoneAccount.SCHEME_TEL)
- : TelecomUtil.composePhoneAccountHandle(
- row.getPhoneAccountComponentName(), row.getPhoneAccountId());
-
List<HistoryItemActionModule> modules = new ArrayList<>();
// Add an audio call item
// TODO(zachh): Support post-dial digits; consider using DialerPhoneNumber.
CallIntentBuilder callIntentBuilder =
new CallIntentBuilder(normalizedNumber, CallInitiationType.Type.CALL_LOG)
- .setPhoneAccountHandle(phoneAccountHandle)
.setAllowAssistedDial(canSupportAssistedDialing(row));
+ // Leave PhoneAccountHandle blank so regular PreCall logic will be used. The account the call
+ // was made/received in should be ignored for audio and carrier video calls.
+ // TODO(a bug): figure out the correct video call behavior
modules.add(IntentModule.newCallModule(context, callIntentBuilder));
// If the call log entry is for a spam call, nothing more to be done.
diff --git a/java/com/android/dialer/calllogutils/CallLogRowActions.java b/java/com/android/dialer/calllogutils/CallLogRowActions.java
index 2090fc32f..d23a15f78 100644
--- a/java/com/android/dialer/calllogutils/CallLogRowActions.java
+++ b/java/com/android/dialer/calllogutils/CallLogRowActions.java
@@ -21,7 +21,6 @@ import com.android.dialer.callintent.CallInitiationType;
import com.android.dialer.callintent.CallIntentBuilder;
import com.android.dialer.calllog.model.CoalescedRow;
import com.android.dialer.precall.PreCall;
-import com.android.dialer.telecom.TelecomUtil;
/** Actions which can be performed on a call log row. */
public final class CallLogRowActions {
@@ -38,9 +37,6 @@ public final class CallLogRowActions {
activity,
new CallIntentBuilder(
row.getNumber().getNormalizedNumber(), CallInitiationType.Type.CALL_LOG)
- .setPhoneAccountHandle(
- TelecomUtil.composePhoneAccountHandle(
- row.getPhoneAccountComponentName(), row.getPhoneAccountId()))
.setIsVideoCall((row.getFeatures() & Calls.FEATURES_VIDEO) == Calls.FEATURES_VIDEO));
}
}