Merge "Fix NPE on TaskFragment" into udc-dev

This commit is contained in:
Massimo Carli
2023-06-05 11:16:16 +00:00
committed by Android (Google) Code Review
2 changed files with 1 additions and 3 deletions

View File

@@ -4281,6 +4281,7 @@ final class ActivityRecord extends WindowToken implements WindowManagerService.A
mTaskSupervisor.getActivityMetricsLogger().notifyActivityRemoved(this); mTaskSupervisor.getActivityMetricsLogger().notifyActivityRemoved(this);
mTaskSupervisor.mStoppingActivities.remove(this); mTaskSupervisor.mStoppingActivities.remove(this);
mLetterboxUiController.destroy();
waitingToShow = false; waitingToShow = false;
// Defer removal of this activity when either a child is animating, or app transition is on // Defer removal of this activity when either a child is animating, or app transition is on
@@ -4350,8 +4351,6 @@ final class ActivityRecord extends WindowToken implements WindowManagerService.A
mWmService.updateFocusedWindowLocked(UPDATE_FOCUS_NORMAL, true /*updateInputWindows*/); mWmService.updateFocusedWindowLocked(UPDATE_FOCUS_NORMAL, true /*updateInputWindows*/);
} }
mLetterboxUiController.destroy();
if (!delayed) { if (!delayed) {
updateReportedVisibilityLocked(); updateReportedVisibilityLocked();
} }

View File

@@ -406,7 +406,6 @@ public class SizeCompatTests extends WindowTestsBase {
clearInvocations(translucentActivity.mLetterboxUiController); clearInvocations(translucentActivity.mLetterboxUiController);
// We destroy the first opaque activity // We destroy the first opaque activity
mActivity.setState(DESTROYED, "testing");
mActivity.removeImmediately(); mActivity.removeImmediately();
// Check that updateInheritedLetterbox() is invoked again // Check that updateInheritedLetterbox() is invoked again