commit 558ace6acae5c522cab6f023c136278d23d06786
author Matt Pietal <mpietal@google.com> Wed Apr 28 10:01:54 2021 -0400
committer Matt Pietal <mpietal@google.com> Wed Apr 28 13:29:29 2021 -0400
tree 377f4daa5ccd644120c9f15c4c4545ebc990a373
parent d3de9a135231d92dd9b89db2f7d55bbcb6dfcb0b

SUW - Apply theme to FRP PIN pages

It is safe to always attempt to copy SUW intent extras, as they will
only be applied if they exist.

Fixes: 171950236
Fixes: 181212237
Fixes: 183711331
Test: SUW FRP verify, settings confirm existing PIN
Change-Id: I6d35683abdc864aea7b1ed0190d6776a75b3e116

src/com/android/settings/password/ChooseLockSettingsHelper.java

diff --git a/src/com/android/settings/password/ChooseLockSettingsHelper.java b/src/com/android/settings/password/ChooseLockSettingsHelper.java
index 29330f5..5ed43fa 100644
--- a/src/com/android/settings/password/ChooseLockSettingsHelper.java
+++ b/src/com/android/settings/password/ChooseLockSettingsHelper.java
@@ -359,21 +359,22 @@
                 requestGatekeeperPasswordHandle);
 
         intent.setClassName(SETTINGS_PACKAGE_NAME, activityClass.getName());
+
+        Intent inIntent = mFragment != null ? mFragment.getActivity().getIntent() :
+                mActivity.getIntent();
+        copyInternalExtras(inIntent, intent);
         if (external) {
             intent.addFlags(Intent.FLAG_ACTIVITY_FORWARD_RESULT);
+            copyOptionalExtras(inIntent, intent);
             if (mFragment != null) {
-                copyOptionalExtras(mFragment.getActivity().getIntent(), intent);
                 mFragment.startActivity(intent);
             } else {
-                copyOptionalExtras(mActivity.getIntent(), intent);
                 mActivity.startActivity(intent);
             }
         } else {
             if (mFragment != null) {
-                copyInternalExtras(mFragment.getActivity().getIntent(), intent);
                 mFragment.startActivityForResult(intent, request);
             } else {
-                copyInternalExtras(mActivity.getIntent(), intent);
                 mActivity.startActivityForResult(intent, request);
             }
         }

src/com/android/settings/password/ConfirmDeviceCredentialBaseActivity.java

diff --git a/src/com/android/settings/password/ConfirmDeviceCredentialBaseActivity.java b/src/com/android/settings/password/ConfirmDeviceCredentialBaseActivity.java
index f0b50a1..4029755 100644
--- a/src/com/android/settings/password/ConfirmDeviceCredentialBaseActivity.java
+++ b/src/com/android/settings/password/ConfirmDeviceCredentialBaseActivity.java
@@ -30,6 +30,8 @@
 import com.android.settings.SetupWizardUtils;
 import com.android.settings.Utils;
 
+import com.google.android.setupdesign.util.ThemeHelper;
+
 public abstract class ConfirmDeviceCredentialBaseActivity extends SettingsActivity {
 
     private static final String STATE_IS_KEYGUARD_LOCKED = "STATE_IS_KEYGUARD_LOCKED";
@@ -74,6 +76,7 @@
             setTheme(SetupWizardUtils.getTheme(this, getIntent()));
             mConfirmCredentialTheme = ConfirmCredentialTheme.NORMAL;
         }
+        ThemeHelper.trySetDynamicColor(this);
         super.onCreate(savedState);
 
         if (mConfirmCredentialTheme == ConfirmCredentialTheme.NORMAL) {