diff options
| author | 2023-08-11 22:00:07 +0000 | |
|---|---|---|
| committer | 2023-08-17 20:57:03 +0000 | |
| commit | 96d2e1b74dd5d27c62d1e89c84e871541fc2e3be (patch) | |
| tree | e266afb5dac9229610c25a8c6caf4014041530d4 | |
| parent | 91e32fa42653150e274f59385e2212fbeaa4b959 (diff) | |
Remove pinned tasks when starting locked task w/ intent
When a task is started w/ flags from an intent, it directly calls into
LockTaskController#startLockTaskMode. This misses the other code called
normally in ActivityTaskManagerService#startLockTaskMode such as closing
any pinned apps.
We fix this by moving this logic down a level to the common code path
shared by both
Bug: 295207389
Test: repro in bug
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:6b6d5b9ab3a7623af264852c47136b5ca5fe7cbc)
Merged-In: Id7386eae96cd014bf41ab3c4184ba9f3a79bdf10
Change-Id: Id7386eae96cd014bf41ab3c4184ba9f3a79bdf10
| -rw-r--r-- | services/core/java/com/android/server/wm/ActivityTaskManagerService.java | 3 | ||||
| -rw-r--r-- | services/core/java/com/android/server/wm/LockTaskController.java | 4 |
2 files changed, 4 insertions, 3 deletions
diff --git a/services/core/java/com/android/server/wm/ActivityTaskManagerService.java b/services/core/java/com/android/server/wm/ActivityTaskManagerService.java index fcf65872e0af..ff2c71999188 100644 --- a/services/core/java/com/android/server/wm/ActivityTaskManagerService.java +++ b/services/core/java/com/android/server/wm/ActivityTaskManagerService.java @@ -2596,9 +2596,6 @@ public class ActivityTaskManagerService extends IActivityTaskManager.Stub { final int callingUid = Binder.getCallingUid(); final long ident = Binder.clearCallingIdentity(); try { - // When a task is locked, dismiss the root pinned task if it exists - mRootWindowContainer.removeRootTasksInWindowingModes(WINDOWING_MODE_PINNED); - getLockTaskController().startLockTaskMode(task, isSystemCaller, callingUid); } finally { Binder.restoreCallingIdentity(ident); diff --git a/services/core/java/com/android/server/wm/LockTaskController.java b/services/core/java/com/android/server/wm/LockTaskController.java index 0c98fb5000d5..830f7859b2f5 100644 --- a/services/core/java/com/android/server/wm/LockTaskController.java +++ b/services/core/java/com/android/server/wm/LockTaskController.java @@ -19,6 +19,7 @@ package com.android.server.wm; import static android.app.ActivityManager.LOCK_TASK_MODE_LOCKED; import static android.app.ActivityManager.LOCK_TASK_MODE_NONE; import static android.app.ActivityManager.LOCK_TASK_MODE_PINNED; +import static android.app.WindowConfiguration.WINDOWING_MODE_PINNED; import static android.app.WindowConfiguration.WINDOWING_MODE_UNDEFINED; import static android.content.Context.DEVICE_POLICY_SERVICE; import static android.content.Context.STATUS_BAR_SERVICE; @@ -669,6 +670,9 @@ public class LockTaskController { } } + // When a task is locked, dismiss the root pinned task if it exists + mSupervisor.mRootWindowContainer.removeRootTasksInWindowingModes(WINDOWING_MODE_PINNED); + // System can only initiate screen pinning, not full lock task mode ProtoLog.w(WM_DEBUG_LOCKTASK, "%s", isSystemCaller ? "Locking pinned" : "Locking fully"); setLockTaskMode(task, isSystemCaller ? LOCK_TASK_MODE_PINNED : LOCK_TASK_MODE_LOCKED, |