Revert "Keep task hidden until task appeared"
This reverts commit cbd8793a441bc828ec22dda09145134da7105818.
Reason for revert: b/153273664
Bug: 153273664
Change-Id: Ic3a0778e896afd5e3fdbc234a5b3521257222dcd
diff --git a/services/core/java/com/android/server/wm/TaskOrganizerController.java b/services/core/java/com/android/server/wm/TaskOrganizerController.java
index 6e41aa5..b641e4c 100644
--- a/services/core/java/com/android/server/wm/TaskOrganizerController.java
+++ b/services/core/java/com/android/server/wm/TaskOrganizerController.java
@@ -25,7 +25,6 @@
import static com.android.server.wm.WindowOrganizerController.CONTROLLABLE_WINDOW_CONFIGS;
import android.annotation.Nullable;
-import android.app.ActivityManager;
import android.app.ActivityManager.RunningTaskInfo;
import android.app.WindowConfiguration;
import android.content.Intent;
@@ -39,7 +38,6 @@
import android.window.ITaskOrganizerController;
import android.window.WindowContainerToken;
-import com.android.internal.annotations.VisibleForTesting;
import com.android.internal.util.ArrayUtils;
import java.io.PrintWriter;
@@ -48,7 +46,6 @@
import java.util.LinkedList;
import java.util.List;
import java.util.WeakHashMap;
-import java.util.function.Consumer;
/**
* Stores the TaskOrganizers associated with a given windowing mode and
@@ -84,95 +81,17 @@
}
}
}
- }
-
- /**
- * A wrapper class around ITaskOrganizer to ensure that the calls are made in the right
- * lifecycle order since we may be updating the visibility of task surface controls in a pending
- * transaction before they are presented to the task org.
- */
- private class TaskOrganizerCallbacks {
- final WindowManagerService mService;
- final ITaskOrganizer mTaskOrganizer;
- final Consumer<Runnable> mDeferTaskOrgCallbacksConsumer;
-
- TaskOrganizerCallbacks(WindowManagerService wm, ITaskOrganizer taskOrg,
- Consumer<Runnable> deferTaskOrgCallbacksConsumer) {
- mService = wm;
- mDeferTaskOrgCallbacksConsumer = deferTaskOrgCallbacksConsumer;
- mTaskOrganizer = taskOrg;
- }
-
- IBinder getBinder() {
- return mTaskOrganizer.asBinder();
- }
-
- void onTaskAppeared(Task task) {
- final RunningTaskInfo taskInfo = task.getTaskInfo();
- mDeferTaskOrgCallbacksConsumer.accept(() -> {
- try {
- mTaskOrganizer.onTaskAppeared(taskInfo);
- } catch (RemoteException e) {
- Slog.e(TAG, "Exception sending onTaskAppeared callback", e);
- }
- });
- }
-
-
- void onTaskVanished(Task task) {
- final RunningTaskInfo taskInfo = task.getTaskInfo();
- mDeferTaskOrgCallbacksConsumer.accept(() -> {
- try {
- mTaskOrganizer.onTaskVanished(taskInfo);
- } catch (RemoteException e) {
- Slog.e(TAG, "Exception sending onTaskVanished callback", e);
- }
- });
- }
-
- void onTaskInfoChanged(Task task, ActivityManager.RunningTaskInfo taskInfo) {
- mDeferTaskOrgCallbacksConsumer.accept(() -> {
- if (!task.isOrganized()) {
- // This is safe to ignore if the task is no longer organized
- return;
- }
- try {
- mTaskOrganizer.onTaskInfoChanged(taskInfo);
- } catch (RemoteException e) {
- Slog.e(TAG, "Exception sending onTaskInfoChanged callback", e);
- }
- });
- }
-
- void onBackPressedOnTaskRoot(Task task) {
- mDeferTaskOrgCallbacksConsumer.accept(() -> {
- if (!task.isOrganized()) {
- // This is safe to ignore if the task is no longer organized
- return;
- }
- try {
- mTaskOrganizer.onBackPressedOnTaskRoot(task.getTaskInfo());
- } catch (Exception e) {
- Slog.e(TAG, "Exception sending onBackPressedOnTaskRoot callback", e);
- }
- });
- }
- }
+ };
private class TaskOrganizerState {
- private final TaskOrganizerCallbacks mOrganizer;
+ private final ITaskOrganizer mOrganizer;
private final DeathRecipient mDeathRecipient;
private final ArrayList<Task> mOrganizedTasks = new ArrayList<>();
private final int mUid;
private boolean mInterceptBackPressedOnTaskRoot;
TaskOrganizerState(ITaskOrganizer organizer, int uid) {
- final Consumer<Runnable> deferTaskOrgCallbacksConsumer =
- mDeferTaskOrgCallbacksConsumer != null
- ? mDeferTaskOrgCallbacksConsumer
- : mService.mWindowManager.mAnimator::addAfterPrepareSurfacesRunnable;
- mOrganizer = new TaskOrganizerCallbacks(mService.mWindowManager, organizer,
- deferTaskOrgCallbacksConsumer);
+ mOrganizer = organizer;
mDeathRecipient = new DeathRecipient(organizer);
try {
organizer.asBinder().linkToDeath(mDeathRecipient, 0);
@@ -188,18 +107,26 @@
void addTask(Task t) {
mOrganizedTasks.add(t);
- mOrganizer.onTaskAppeared(t);
+ try {
+ mOrganizer.onTaskAppeared(t.getTaskInfo());
+ } catch (Exception e) {
+ Slog.e(TAG, "Exception sending taskAppeared callback" + e);
+ }
}
void removeTask(Task t) {
+ try {
+ mOrganizer.onTaskVanished(t.getTaskInfo());
+ } catch (Exception e) {
+ Slog.e(TAG, "Exception sending taskVanished callback" + e);
+ }
mOrganizedTasks.remove(t);
- mOrganizer.onTaskVanished(t);
}
void dispose() {
releaseTasks();
for (int i = mTaskOrganizersForWindowingMode.size() - 1; i >= 0; --i) {
- mTaskOrganizersForWindowingMode.valueAt(i).remove(mOrganizer.getBinder());
+ mTaskOrganizersForWindowingMode.valueAt(i).remove(mOrganizer.asBinder());
}
}
@@ -212,7 +139,7 @@
}
void unlinkDeath() {
- mOrganizer.getBinder().unlinkToDeath(mDeathRecipient, 0);
+ mOrganizer.asBinder().unlinkToDeath(mDeathRecipient, 0);
}
}
@@ -222,10 +149,9 @@
private final WeakHashMap<Task, RunningTaskInfo> mLastSentTaskInfos = new WeakHashMap<>();
private final ArrayList<Task> mPendingTaskInfoChanges = new ArrayList<>();
- private final ActivityTaskManagerService mService;
+ final ActivityTaskManagerService mService;
- private RunningTaskInfo mTmpTaskInfo;
- private Consumer<Runnable> mDeferTaskOrgCallbacksConsumer;
+ RunningTaskInfo mTmpTaskInfo;
TaskOrganizerController(ActivityTaskManagerService atm) {
mService = atm;
@@ -237,15 +163,6 @@
}
/**
- * Specifies the consumer to run to defer the task org callbacks. Can be overridden while
- * testing to allow the callbacks to be sent synchronously.
- */
- @VisibleForTesting
- public void setDeferTaskOrgCallbacksConsumer(Consumer<Runnable> consumer) {
- mDeferTaskOrgCallbacksConsumer = consumer;
- }
-
- /**
* Register a TaskOrganizer to manage tasks as they enter the given windowing mode.
* If there was already a TaskOrganizer for this windowing mode it will be evicted
* but will continue to organize it's existing tasks.
@@ -336,7 +253,7 @@
if (state == null) {
return null;
}
- return state.mOrganizer.mTaskOrganizer;
+ return state.mOrganizer;
}
void onTaskAppeared(ITaskOrganizer organizer, Task task) {
@@ -441,10 +358,11 @@
// change.
mTmpTaskInfo = null;
- if (task.isOrganized()) {
- final TaskOrganizerState state = mTaskOrganizerStates.get(
- task.mTaskOrganizer.asBinder());
- state.mOrganizer.onTaskInfoChanged(task, newInfo);
+ if (task.mTaskOrganizer != null) {
+ try {
+ task.mTaskOrganizer.onTaskInfoChanged(newInfo);
+ } catch (RemoteException e) {
+ }
}
}
@@ -599,7 +517,11 @@
return false;
}
- state.mOrganizer.onBackPressedOnTaskRoot(task);
+ try {
+ state.mOrganizer.onBackPressedOnTaskRoot(task.getTaskInfo());
+ } catch (Exception e) {
+ Slog.e(TAG, "Exception sending interceptBackPressedOnTaskRoot callback" + e);
+ }
return true;
}
@@ -616,7 +538,7 @@
final TaskOrganizerState state = mTaskOrganizerStates.get(taskOrgs.get(j));
final ArrayList<Task> tasks = state.mOrganizedTasks;
pw.print(innerPrefix + " ");
- pw.println(state.mOrganizer.mTaskOrganizer + " uid=" + state.mUid + ":");
+ pw.println(state.mOrganizer + " uid=" + state.mUid + ":");
for (int k = 0; k < tasks.size(); k++) {
pw.println(innerPrefix + " " + tasks.get(k));
}