diff options
author | twyen <twyen@google.com> | 2018-04-30 14:25:46 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-04-30 18:18:34 -0700 |
commit | 56f79ba6c6608f7041f5e65866b7164499ca7676 (patch) | |
tree | 7699f1e6bd1b2180d70144327d68f6df94dd3265 /java/com/android/incallui/ActiveCallsCallListListener.java | |
parent | 90a68377c36abccde8c4f1e5d3aa781944a1413c (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.java | 4 |
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()); } } |