commit 64fd8ed97fa6f737aa7a34e862ec330140187d01
author Winson Chung <winsonc@google.com> Thu Jan 05 23:27:52 2023 +0000
committer Winson Chung <winsonc@google.com> Thu Jan 05 23:29:47 2023 +0000
tree c284c544fe96638373661df3a53b6458b3037206
parent fa71876cc023e55bd939cf7f39a991abbb2eb4f9

Fix issue with sysui state flags not being committed

- These were not being explicitly committed which meant that they only
  properly set/unset when a subsequent sysui state change happened
  (which isn't always the case, leading to an mismatch in state)

Fixes: 263921242
Test: Open camera, show global actions dialog, go back and try to swipe
      home
Change-Id: I7734d8ab2f21c1a6a4b75ad39f0825c66872c0eb

packages/SystemUI/src/com/android/systemui/statusbar/phone/SystemUIDialog.java

diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/SystemUIDialog.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/SystemUIDialog.java
index 26bc3e3..608bfa6 100644
--- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/SystemUIDialog.java
+++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/SystemUIDialog.java
@@ -189,7 +189,8 @@
         // for foldables that often go from large <=> small screen when folding/unfolding.
         ViewRootImpl.addConfigCallback(this);
         mDialogManager.setShowing(this, true);
-        mSysUiState.setFlag(QuickStepContract.SYSUI_STATE_DIALOG_SHOWING, true);
+        mSysUiState.setFlag(QuickStepContract.SYSUI_STATE_DIALOG_SHOWING, true)
+                .commitUpdate(mContext.getDisplayId());
     }
 
     @Override
@@ -202,7 +203,8 @@
 
         ViewRootImpl.removeConfigCallback(this);
         mDialogManager.setShowing(this, false);
-        mSysUiState.setFlag(QuickStepContract.SYSUI_STATE_DIALOG_SHOWING, false);
+        mSysUiState.setFlag(QuickStepContract.SYSUI_STATE_DIALOG_SHOWING, false)
+                .commitUpdate(mContext.getDisplayId());
     }
 
     public void setShowForAllUsers(boolean show) {