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

Commit 21749319 authored by Marzia Favaro's avatar Marzia Favaro
Browse files

Ignore exclusion region changes for app handle 

These are not needed and are causing jank

Bug: 383672263
Test: Reduced app missed frames http://shortn/_phPzGecJ02
Flag: com.android.window.flags.app_handle_no_relayout_on_exclusion_change
Change-Id: I5e55fe4abd6f95bf421d660db37eafa4353108da
parent 0026e374

core/java/android/window/flags/windowing_frontend.aconfig

+10 −0
Original line number Diff line number Diff line
@@ -179,6 +179,16 @@ flag { 
  bug: "377689543"

}



flag {

  name: "app_handle_no_relayout_on_exclusion_change"

  namespace: "windowing_frontend"

  description: "Remove unnecessary relayouts for app handle when exclusion regions change"

  bug: "383672263"

  metadata {

    purpose: PURPOSE_BUGFIX

  }

}



flag {

  name: "respect_non_top_visible_fixed_orientation"

  namespace: "windowing_frontend"

libs/WindowManager/Shell/src/com/android/wm/shell/windowdecor/DesktopModeWindowDecoration.java

+9 −0
Original line number Diff line number Diff line
@@ -422,6 +422,15 @@ public class DesktopModeWindowDecoration extends WindowDecoration<WindowDecorLin 
        mOpenInBrowserClickListener = listener;

    }



    @Override

    void onExclusionRegionChanged(@NonNull Region exclusionRegion) {

        if (Flags.appHandleNoRelayoutOnExclusionChange() && isAppHandle(mWindowDecorViewHolder)) {

            // Avoid unnecessary relayouts for app handle. See b/383672263

            return;

        }

        relayout(mTaskInfo, mHasGlobalFocus, exclusionRegion);

    }



    @Override

    void relayout(ActivityManager.RunningTaskInfo taskInfo, boolean hasGlobalFocus,

            @NonNull Region displayExclusionRegion) {