commit 3c8ab7ed43cbda8b0c9620b9edad48e49e61f1be
author twyen <twyen@google.com> Mon Nov 20 12:21:00 2017 -0800
committer Copybara-Service <copybara-piper@google.com> Mon Nov 20 12:32:20 2017 -0800
tree c2927e4259bac80faffdf4ebea06445ff88a14da
parent 26dbd5e7eeabe7929e5ac639faa2969272f6f202

Add config flag for CallingAccountSelector

Since CallingAccountSelector is currently the only action that requires a UI, disabling it will effectively revert the PreCall behavior back to pre-PreCall era, sending intents to telecom directly.

Bug: 68953196
Test: CallingAccountSelectorTest
PiperOrigin-RevId: 176398440
Change-Id: I612a56bc66f32cf72779bd7c507de6a0784f1298

java/com/android/dialer/precall/impl/CallingAccountSelector.java

diff --git a/java/com/android/dialer/precall/impl/CallingAccountSelector.java b/java/com/android/dialer/precall/impl/CallingAccountSelector.java
index e0fe0c4..d46e317 100644
--- a/java/com/android/dialer/precall/impl/CallingAccountSelector.java
+++ b/java/com/android/dialer/precall/impl/CallingAccountSelector.java
@@ -39,6 +39,7 @@
 import com.android.dialer.common.LogUtil;
 import com.android.dialer.common.concurrent.DialerExecutor.Worker;
 import com.android.dialer.common.concurrent.DialerExecutorComponent;
+import com.android.dialer.configprovider.ConfigProviderBindings;
 import com.android.dialer.precall.PreCallAction;
 import com.android.dialer.precall.PreCallCoordinator;
 import com.android.dialer.precall.PreCallCoordinator.PendingAction;
@@ -62,6 +63,11 @@
 
   @Override
   public boolean requiresUi(Context context, CallIntentBuilder builder) {
+    if (!ConfigProviderBindings.get(context)
+        .getBoolean("precall_calling_account_selector_enabled", true)) {
+      return false;
+    }
+
     if (builder.getPhoneAccountHandle() != null) {
       return false;
     }