From d4c9489b1d87c110eba474079d273abdbc2416b3 Mon Sep 17 00:00:00 2001 From: Caitlin Shkuratov Date: Thu, 29 Jun 2023 17:03:33 +0000 Subject: [Status Bar] Set status bar window insets to the status bar height. This is needed for the animation shown when you tap the ongoing call chip in order to launch the call app. If we do *not* set the insets, then the insets will change during the animation, which will result in the animation being cancelled and show jank when launching the app. Fixes: 283958440 Test: tap on ongoing call chip -> verify no jank during animation Test: status bar smoke test on phone (verify status bar in portrait and landscape, with and without ongoing call chip) Test: status bar smoke test on tablet (verify status bar in portrait and landscape, with and without ongoign call chip) Change-Id: Ic0f9d7d571d3872b96135080313439020a294d3d --- .../systemui/statusbar/window/StatusBarWindowController.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/window/StatusBarWindowController.java b/packages/SystemUI/src/com/android/systemui/statusbar/window/StatusBarWindowController.java index bcf3b0cbfc86..4a9921eb2d83 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/window/StatusBarWindowController.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/window/StatusBarWindowController.java @@ -240,8 +240,10 @@ public class StatusBarWindowController { Insets.of(0, safeTouchRegionHeight, 0, 0)); } lp.providedInsets = new InsetsFrameProvider[] { - new InsetsFrameProvider(mInsetsSourceOwner, 0, statusBars()), - new InsetsFrameProvider(mInsetsSourceOwner, 0, tappableElement()), + new InsetsFrameProvider(mInsetsSourceOwner, 0, statusBars()) + .setInsetsSize(Insets.of(0, height, 0, 0)), + new InsetsFrameProvider(mInsetsSourceOwner, 0, tappableElement()) + .setInsetsSize(Insets.of(0, height, 0, 0)), gestureInsetsProvider }; return lp; -- cgit v1.2.3-59-g8ed1b