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

Commit 95c1e307 authored by Riddle Hsu's avatar Riddle Hsu Committed by Android (Google) Code Review
Browse files

Merge "Clean up launched flag removePrepareSurfaceInPlacement" into main

parents c9626f21 be9db2b3
Loading
Loading
Loading
Loading
+0 −11
Original line number Diff line number Diff line
@@ -52,17 +52,6 @@ flag {
    is_fixed_read_only: true
}

flag {
  name: "remove_prepare_surface_in_placement"
  namespace: "windowing_frontend"
  description: "Reduce unnecessary invocation to improve performance"
  bug: "330721336"
  is_fixed_read_only: true
  metadata {
    purpose: PURPOSE_BUGFIX
  }
}

flag {
  name: "close_to_square_config_includes_status_bar"
  namespace: "windowing_frontend"
+0 −3
Original line number Diff line number Diff line
@@ -5047,9 +5047,6 @@ class DisplayContent extends RootDisplayArea implements WindowManagerPolicy.Disp
        } finally {
            Trace.traceEnd(TRACE_TAG_WINDOW_MANAGER);
        }
        if (!com.android.window.flags.Flags.removePrepareSurfaceInPlacement()) {
            prepareSurfaces();
        }

        // This should be called after the insets have been dispatched to clients and we have
        // committed finish drawing windows.
+1 −2
Original line number Diff line number Diff line
@@ -4804,8 +4804,7 @@ class Task extends TaskFragment {
                    // task is still updated by core. Otherwise if the task is collected (e.g.
                    // rotation change) after leaving this scope, the visibility operation will be
                    // put in sync transaction, then it is not synced with reparent.
                    if (com.android.window.flags.Flags.removePrepareSurfaceInPlacement()
                            && lastParentBeforePip.mSyncState == SYNC_STATE_NONE) {
                    if (lastParentBeforePip.mSyncState == SYNC_STATE_NONE) {
                        lastParentBeforePip.prepareSurfaces();
                        // If the moveToFront is a part of finishing transition, then make sure
                        // the z-order of tasks are up-to-date.
+2 −6
Original line number Diff line number Diff line
@@ -4996,12 +4996,8 @@ class WindowState extends WindowContainer<WindowState> implements WindowManagerP
        anim.restrictDuration(MAX_ANIMATION_DURATION);
        anim.scaleCurrentDuration(mWmService.getWindowAnimationScaleLocked());
        final Point position = new Point();
        if (com.android.window.flags.Flags.removePrepareSurfaceInPlacement()) {
        transformFrameToSurfacePosition(mWindowFrames.mFrame.left, mWindowFrames.mFrame.top,
                position);
        } else {
            position.set(mSurfacePosition);
        }
        final AnimationAdapter adapter = new LocalAnimationAdapter(
                new WindowAnimationSpec(anim, position, false /* canSkipFirstFrame */,
                        0 /* windowCornerRadius */),