summaryrefslogtreecommitdiff
path: root/java/com/android/incallui/ActiveCallsCallListListener.java
diff options
context:
space:
mode:
authortwyen <twyen@google.com>2018-04-30 14:25:46 -0700
committerCopybara-Service <copybara-piper@google.com>2018-04-30 18:18:34 -0700
commit56f79ba6c6608f7041f5e65866b7164499ca7676 (patch)
tree7699f1e6bd1b2180d70144327d68f6df94dd3265 /java/com/android/incallui/ActiveCallsCallListListener.java
parent90a68377c36abccde8c4f1e5d3aa781944a1413c (diff)
Refactor PreferredAccountWorker to provide the dialog to be shown.
When dual SIM selection support is added to In Call UI it was a rush order and codes are duplicated. This CL moves the duplicated logic into PreferredAccountWorker so the same dialog can be shown for both. TEST=manual Bug: 69675796,72618783 Test: manual PiperOrigin-RevId: 194845320 Change-Id: Id283ca7616580b0efd4e8f02e63691c70ee7f93c
Diffstat (limited to 'java/com/android/incallui/ActiveCallsCallListListener.java')
-rw-r--r--java/com/android/incallui/ActiveCallsCallListListener.java4
1 files changed, 2 insertions, 2 deletions
diff --git a/java/com/android/incallui/ActiveCallsCallListListener.java b/java/com/android/incallui/ActiveCallsCallListListener.java
index 3e4cb9375..d94db8836 100644
--- a/java/com/android/incallui/ActiveCallsCallListListener.java
+++ b/java/com/android/incallui/ActiveCallsCallListListener.java
@@ -49,10 +49,10 @@ public class ActiveCallsCallListListener implements CallList.Listener {
public void onCallListChange(CallList callList) {
ImmutableList.Builder<ActiveCallInfo> activeCalls = ImmutableList.builder();
for (DialerCall call : callList.getAllCalls()) {
- if (call.getState() != DialerCallState.DISCONNECTED) {
+ if (call.getState() != DialerCallState.DISCONNECTED && call.getAccountHandle() != null) {
activeCalls.add(
ActiveCallInfo.builder()
- .setPhoneAccountHandle(Optional.fromNullable(call.getAccountHandle()))
+ .setPhoneAccountHandle(Optional.of(call.getAccountHandle()))
.build());
}
}