Remove workaround of delaying remove task
The timing doesn't work on the latest code base. The onStop
of activity still almost has no chance to be called after
swiping the task to kill process.
And the friendly activity lifecycle will be controlled by
removeTask since Id69a8ba77da16a13f7a3413bb004fc2206f4b4e8.
Bug: 184899234
Bug: 213583589
Test: No behavior change after swiping task out from recents
Change-Id: If4110069834964571df7128b119fc6c782a03b34
diff --git a/quickstep/src/com/android/quickstep/views/RecentsView.java b/quickstep/src/com/android/quickstep/views/RecentsView.java
index fe50d9a..9a25e9c 100644
--- a/quickstep/src/com/android/quickstep/views/RecentsView.java
+++ b/quickstep/src/com/android/quickstep/views/RecentsView.java
@@ -225,10 +225,6 @@
private static final String TAG = "RecentsView";
private static final boolean DEBUG = false;
- // TODO(b/184899234): We use this timeout to wait a fixed period after switching to the
- // screenshot when dismissing the current live task to ensure the app can try and get stopped.
- private static final int REMOVE_TASK_WAIT_FOR_APP_STOP_MS = 100;
-
public static final FloatProperty<RecentsView> CONTENT_ALPHA =
new FloatProperty<RecentsView>("contentAlpha") {
@Override
@@ -3893,14 +3889,13 @@
int[] taskIds = getTaskIdsForTaskViewId(dismissedTaskViewId);
int primaryTaskId = taskIds[0];
int secondaryTaskId = taskIds[1];
- UI_HELPER_EXECUTOR.getHandler().postDelayed(
+ UI_HELPER_EXECUTOR.getHandler().post(
() -> {
ActivityManagerWrapper.getInstance().removeTask(primaryTaskId);
if (secondaryTaskId != -1) {
ActivityManagerWrapper.getInstance().removeTask(secondaryTaskId);
}
- },
- REMOVE_TASK_WAIT_FOR_APP_STOP_MS);
+ });
}
protected void onDismissAnimationEnds() {
@@ -3923,9 +3918,8 @@
if (isSuccess) {
// Remove all the task views now
finishRecentsAnimation(true /* toRecents */, false /* shouldPip */, () -> {
- UI_HELPER_EXECUTOR.getHandler().postDelayed(
- ActivityManagerWrapper.getInstance()::removeAllRecentTasks,
- REMOVE_TASK_WAIT_FOR_APP_STOP_MS);
+ UI_HELPER_EXECUTOR.getHandler().post(
+ ActivityManagerWrapper.getInstance()::removeAllRecentTasks);
removeTasksViewsAndClearAllButton();
startHome();
});