summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorlinyuh <linyuh@google.com>2018-06-05 17:42:51 -0700
committerCopybara-Service <copybara-piper@google.com>2018-06-05 18:55:17 -0700
commitb4d6e35844ef2ee5b77b34b487e019c802e27b29 (patch)
tree4de56c22b668b3f87e9515dbb2b9710409aa0350
parent87300f1c76e30f6086f7fd1f44ddd7faaed2e1a7 (diff)
Allow CallingAccountSelector to be shown when the device is locked.
Bug: 80408180 Test: PreCallActivityTest + Manual testing on a Moto device PiperOrigin-RevId: 199387969 Change-Id: I81d19e22f5a26676eb725b32148c43afd43d3119
-rw-r--r--java/com/android/dialer/precall/impl/PreCallActivity.java31
1 files changed, 26 insertions, 5 deletions
diff --git a/java/com/android/dialer/precall/impl/PreCallActivity.java b/java/com/android/dialer/precall/impl/PreCallActivity.java
index ee417b968..7b27945e5 100644
--- a/java/com/android/dialer/precall/impl/PreCallActivity.java
+++ b/java/com/android/dialer/precall/impl/PreCallActivity.java
@@ -16,11 +16,8 @@
package com.android.dialer.precall.impl;
-import android.annotation.TargetApi;
import android.app.Activity;
import android.app.KeyguardManager;
-import android.os.Build.VERSION;
-import android.os.Build.VERSION_CODES;
import android.os.Bundle;
import android.support.annotation.Nullable;
import android.view.WindowManager.LayoutParams;
@@ -36,8 +33,32 @@ public class PreCallActivity extends Activity {
preCallCoordinator = new PreCallCoordinatorImpl(this);
preCallCoordinator.onCreate(getIntent(), savedInstanceState);
if (getSystemService(KeyguardManager.class).isKeyguardLocked()) {
- getWindow().addFlags(LayoutParams.FLAG_SHOW_WHEN_LOCKED);
-
+ // Note:
+ //
+ // Flag LayoutParams.FLAG_TURN_SCREEN_ON was deprecated in O_MR1, but calling the new API
+ // setTurnScreenOn(true) doesn't give us the expected behavior.
+ //
+ // Calling setTurnScreenOn(true) alone doesn't turn on the screen when the device is locked.
+ // We must also call KeyguardManager#requestDismissKeyguard, which will bring up the lock
+ // screen for the user to enter their credentials.
+ //
+ // If the Keyguard is not secure or the device is currently in a trusted state, calling
+ // requestDismissKeyguard will immediately dismiss the Keyguard without any user interaction.
+ // However, the lock screen will still pop up before it quickly disappears.
+ //
+ // If the Keyguard is secure and the device is not in a trusted state, the device will show
+ // the lock screen and wait for the user's credentials.
+ //
+ // Therefore, to avoid showing the lock screen, we will continue using the deprecated flag in
+ // O_MR1 and later Android versions.
+ //
+ // Flag LayoutParams.FLAG_SHOW_WHEN_LOCKED was also deprecated in O_MR1, and the new API
+ // setShowWhenLocked(boolean) works. However, as the purpose of the two new APIs is to prevent
+ // an unintentional double life-cycle event, only using one is ineffective.
+ //
+ // Therefore, to simplify code and make testing easier, we will also keep using
+ // LayoutParams.FLAG_SHOW_WHEN_LOCKED.
+ getWindow().addFlags(LayoutParams.FLAG_SHOW_WHEN_LOCKED | LayoutParams.FLAG_TURN_SCREEN_ON);
}
}