diff --git a/quickstep/src/com/android/quickstep/RecentsAnimationCallbacks.java b/quickstep/src/com/android/quickstep/RecentsAnimationCallbacks.java index da7a98f330..32e23893d3 100644 --- a/quickstep/src/com/android/quickstep/RecentsAnimationCallbacks.java +++ b/quickstep/src/com/android/quickstep/RecentsAnimationCallbacks.java @@ -128,15 +128,11 @@ public class RecentsAnimationCallbacks implements mController::finishAnimationToApp); } else { RemoteAnimationTarget[] nonAppTargets; - if (!TaskAnimationManager.ENABLE_SHELL_TRANSITIONS) { - nonAppTargets = mSystemUiProxy.onGoingToRecentsLegacy(appTargets); - } else { - final ArrayList apps = new ArrayList<>(); - final ArrayList nonApps = new ArrayList<>(); - classifyTargets(appTargets, apps, nonApps); - appTargets = apps.toArray(new RemoteAnimationTarget[apps.size()]); - nonAppTargets = nonApps.toArray(new RemoteAnimationTarget[nonApps.size()]); - } + final ArrayList apps = new ArrayList<>(); + final ArrayList nonApps = new ArrayList<>(); + classifyTargets(appTargets, apps, nonApps); + appTargets = apps.toArray(new RemoteAnimationTarget[apps.size()]); + nonAppTargets = nonApps.toArray(new RemoteAnimationTarget[nonApps.size()]); if (nonAppTargets == null) { nonAppTargets = new RemoteAnimationTarget[0]; } diff --git a/quickstep/src/com/android/quickstep/SystemUiProxy.java b/quickstep/src/com/android/quickstep/SystemUiProxy.java index 673de149b9..0c2f29b8db 100644 --- a/quickstep/src/com/android/quickstep/SystemUiProxy.java +++ b/quickstep/src/com/android/quickstep/SystemUiProxy.java @@ -1000,77 +1000,6 @@ public class SystemUiProxy implements ISystemUiProxy, NavHandle, SafeCloseable { } } - /** - * Start multiple tasks in split-screen simultaneously. - */ - public void startTasksWithLegacyTransition(int taskId1, Bundle options1, int taskId2, - Bundle options2, @StagePosition int splitPosition, - @PersistentSnapPosition int snapPosition, RemoteAnimationAdapter adapter, - InstanceId instanceId) { - if (mSystemUiProxy != null) { - try { - mSplitScreen.startTasksWithLegacyTransition(taskId1, options1, taskId2, options2, - splitPosition, snapPosition, adapter, instanceId); - } catch (RemoteException e) { - Log.w(TAG, splitFailureMessage( - "startTasksWithLegacyTransition", "RemoteException"), e); - } - } - } - - public void startIntentAndTaskWithLegacyTransition(PendingIntent pendingIntent, int userId1, - Bundle options1, int taskId, Bundle options2, @StagePosition int splitPosition, - @PersistentSnapPosition int snapPosition, RemoteAnimationAdapter adapter, - InstanceId instanceId) { - if (mSystemUiProxy != null) { - try { - mSplitScreen.startIntentAndTaskWithLegacyTransition(pendingIntent, userId1, - options1, taskId, options2, splitPosition, snapPosition, adapter, - instanceId); - } catch (RemoteException e) { - Log.w(TAG, splitFailureMessage( - "startIntentAndTaskWithLegacyTransition", "RemoteException"), e); - } - } - } - - public void startShortcutAndTaskWithLegacyTransition(ShortcutInfo shortcutInfo, Bundle options1, - int taskId, Bundle options2, @StagePosition int splitPosition, - @PersistentSnapPosition int snapPosition, RemoteAnimationAdapter adapter, - InstanceId instanceId) { - if (mSystemUiProxy != null) { - try { - mSplitScreen.startShortcutAndTaskWithLegacyTransition(shortcutInfo, options1, - taskId, options2, splitPosition, snapPosition, adapter, instanceId); - } catch (RemoteException e) { - Log.w(TAG, splitFailureMessage( - "startShortcutAndTaskWithLegacyTransition", "RemoteException"), e); - } - } - } - - /** - * Starts a pair of intents or shortcuts in split-screen using legacy transition. Passing a - * non-null shortcut info means to start the app as a shortcut. - */ - public void startIntentsWithLegacyTransition(PendingIntent pendingIntent1, int userId1, - @Nullable ShortcutInfo shortcutInfo1, @Nullable Bundle options1, - PendingIntent pendingIntent2, int userId2, @Nullable ShortcutInfo shortcutInfo2, - @Nullable Bundle options2, @StagePosition int sidePosition, - @PersistentSnapPosition int snapPosition, RemoteAnimationAdapter adapter, - InstanceId instanceId) { - if (mSystemUiProxy != null) { - try { - mSplitScreen.startIntentsWithLegacyTransition(pendingIntent1, userId1, - shortcutInfo1, options1, pendingIntent2, userId2, shortcutInfo2, options2, - sidePosition, snapPosition, adapter, instanceId); - } catch (RemoteException e) { - Log.w(TAG, splitFailureMessage( - "startIntentsWithLegacyTransition", "RemoteException"), e); - } - } - } - public void startShortcut(String packageName, String shortcutId, int position, Bundle options, UserHandle user, InstanceId instanceId) { if (mSplitScreen != null) { @@ -1105,36 +1034,6 @@ public class SystemUiProxy implements ISystemUiProxy, NavHandle, SafeCloseable { } } - /** - * Call this when going to recents so that shell can set-up and provide appropriate leashes - * for animation (eg. DividerBar). - * - * @return RemoteAnimationTargets of windows that need to animate but only exist in shell. - */ - @Nullable - public RemoteAnimationTarget[] onGoingToRecentsLegacy(RemoteAnimationTarget[] apps) { - if (!TaskAnimationManager.ENABLE_SHELL_TRANSITIONS && mSplitScreen != null) { - try { - return mSplitScreen.onGoingToRecentsLegacy(apps); - } catch (RemoteException e) { - Log.w(TAG, "Failed call onGoingToRecentsLegacy"); - } - } - return null; - } - - @Nullable - public RemoteAnimationTarget[] onStartingSplitLegacy(RemoteAnimationTarget[] apps) { - if (mSplitScreen != null) { - try { - return mSplitScreen.onStartingSplitLegacy(apps); - } catch (RemoteException e) { - Log.w(TAG, "Failed call onStartingSplitLegacy"); - } - } - return null; - } - // // One handed // diff --git a/quickstep/src/com/android/quickstep/TaskAnimationManager.java b/quickstep/src/com/android/quickstep/TaskAnimationManager.java index 723aa03b9c..f414399a0b 100644 --- a/quickstep/src/com/android/quickstep/TaskAnimationManager.java +++ b/quickstep/src/com/android/quickstep/TaskAnimationManager.java @@ -264,12 +264,7 @@ public class TaskAnimationManager implements RecentsAnimationCallbacks.RecentsAn } } - RemoteAnimationTarget[] nonAppTargets = ENABLE_SHELL_TRANSITIONS - ? null : getSystemUiProxy().onStartingSplitLegacy( - appearedTaskTargets); - if (nonAppTargets == null) { - nonAppTargets = new RemoteAnimationTarget[0]; - } + RemoteAnimationTarget[] nonAppTargets = new RemoteAnimationTarget[0]; if ((containerInterface.isInLiveTileMode() || mLastGestureState.getEndTarget() == RECENTS || isNonRecentsStartedTasksAppeared(appearedTaskTargets))