Updating TaskbarStashController API

> Making the state animator nonNull to avoid null checks at caller
> Updating the methods names to explicitely indication animation
  being started or not

Bug: 265352919
Test: Verified on device
Change-Id: I1f5b47921f831ed0b202aced10825c8e1d39a4dc
This commit is contained in:
Sunny Goyal
2023-01-12 16:09:24 -08:00
parent bf69ef034a
commit a34521e434
3 changed files with 28 additions and 53 deletions

View File

@@ -73,12 +73,12 @@ public class FallbackTaskbarUIController extends TaskbarUIController {
boolean useStashedLauncherState = toState.hasOverviewActions();
boolean stashedLauncherState =
useStashedLauncherState && !FeatureFlags.ENABLE_TASKBAR_IN_OVERVIEW.get();
TaskbarStashController controller = mControllers.taskbarStashController;
TaskbarStashController stashController = mControllers.taskbarStashController;
// Set both FLAG_IN_STASHED_LAUNCHER_STATE and FLAG_IN_APP to ensure the state is respected.
// For all other states, just use the current stashed-in-app setting (e.g. if long clicked).
controller.updateStateForFlag(FLAG_IN_STASHED_LAUNCHER_STATE, stashedLauncherState);
controller.updateStateForFlag(FLAG_IN_APP, !useStashedLauncherState);
return controller.applyStateWithoutStart(duration);
stashController.updateStateForFlag(FLAG_IN_STASHED_LAUNCHER_STATE, stashedLauncherState);
stashController.updateStateForFlag(FLAG_IN_APP, !useStashedLauncherState);
return stashController.createApplyStateAnimator(duration);
}
private void animateToRecentsState(RecentsState toState) {