Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 035a06f4 authored by Louis Chang's avatar Louis Chang Committed by Automerger Merge Worker
Browse files

Merge "Fix black screen while animating a closing and changing TF" into udc-dev am: 3e120345 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/22223384



Change-Id: Id6ae11754466352407350e71d5b7b1891d0767ad
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents b64fca66 3e120345

libs/WindowManager/Jetpack/src/androidx/window/extensions/embedding/TaskFragmentAnimationRunner.java

+2 −1
Original line number Diff line number Diff line
@@ -214,7 +214,8 @@ class TaskFragmentAnimationRunner extends IRemoteAnimationRunner.Stub { 
                openingWholeScreenBounds.union(target.screenSpaceBounds);

            } else {

                closingTargets.add(target);

                closingWholeScreenBounds.union(target.screenSpaceBounds);

                // Union the start bounds since this may be the ClosingChanging animation.

                closingWholeScreenBounds.union(target.startBounds);

            }

        }

libs/WindowManager/Shell/src/com/android/wm/shell/activityembedding/ActivityEmbeddingAnimationRunner.java

+1 −1
Original line number Diff line number Diff line
@@ -224,7 +224,7 @@ class ActivityEmbeddingAnimationRunner { 
                openingWholeScreenBounds.union(change.getEndAbsBounds());

            } else {

                closingChanges.add(change);

                closingWholeScreenBounds.union(change.getEndAbsBounds());

                closingWholeScreenBounds.union(change.getStartAbsBounds());

            }

        }

services/core/java/com/android/server/wm/TaskFragment.java

+7 −2
Original line number Diff line number Diff line
@@ -2553,13 +2553,18 @@ class TaskFragment extends WindowContainer<WindowContainer> { 
        return task != null && !task.isDragResizing() && super.canStartChangeTransition();

    }



    /** Records the starting bounds of the closing organized TaskFragment. */

    void setClosingChangingStartBoundsIfNeeded() {

    /**

     * Returns {@code true} if the starting bounds of the closing organized TaskFragment is

     * recorded. Otherwise, return {@code false}.

     */

    boolean setClosingChangingStartBoundsIfNeeded() {

        if (isOrganizedTaskFragment() && mDisplayContent != null

                && mDisplayContent.mChangingContainers.remove(this)) {

            mDisplayContent.mClosingChangingContainers.put(

                    this, new Rect(mSurfaceFreezer.mFreezeBounds));

            return true;

        }

        return false;

    }



    @Override

services/core/java/com/android/server/wm/WindowContainer.java

+6 −2
Original line number Diff line number Diff line
@@ -1336,15 +1336,19 @@ class WindowContainer<E extends WindowContainer> extends ConfigurationContainer< 
        // If we are losing visibility, then a snapshot isn't necessary and we are no-longer

        // part of a change transition.

        if (!visible) {

            boolean skipUnfreeze = false;

            if (asTaskFragment() != null) {

                // If the organized TaskFragment is closing while resizing, we want to keep track of

                // its starting bounds to make sure the animation starts at the correct position.

                // This should be called before unfreeze() because we record the starting bounds

                // in SurfaceFreezer.

                asTaskFragment().setClosingChangingStartBoundsIfNeeded();

                skipUnfreeze = asTaskFragment().setClosingChangingStartBoundsIfNeeded();

            }



            if (!skipUnfreeze) {

                mSurfaceFreezer.unfreeze(getSyncTransaction());

            }

        }

        WindowContainer parent = getParent();

        if (parent != null) {

            parent.onChildVisibilityRequested(visible);