Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Hold onto occluded change until it's committed

Browse Source 
When we're delaying commit of setKeyguardOccluded due to a transition
taking over instead to synchronise occlude status more nicely, we hold
onto the final occlude status so we can be eventually consistent even
if the systemui restarts or gets something wrong.

The "notify" parameter in setKeyguardOccludedLw was risky because when
false it could lead to SystemUI not being informed of an update, but
the system server still considering its job done because the flag for
pending updates was cleared anyway.

This is a speculative fix to help the attached issue resolved itself
whenever it happens.

(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:52d8053880b1be999ebc75730333b2d9ea326fb4)

Test: atest android.server.wm.KeyguardTests
Bug: 269892931
Merged-In: I7d3ac020db34d8a15d18b25de07231bda83cbcf0
Change-Id: I7d3ac020db34d8a15d18b25de07231bda83cbcf0
This commit is contained in:
Robin Lee
2023-03-16 19:24:01 +01:00
parent 0af487566d
commit 8a8169dae5
1 changed files with 9 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
services/core/java/com/android/server/policy/PhoneWindowManager.java
View File

@@ -3341,8 +3341,7 @@ public class PhoneWindowManager implements WindowManagerPolicy {
mPendingKeyguardOccluded = occluded;
mKeyguardOccludedChanged = true;
} else {
setKeyguardOccludedLw(occluded, false /* force */,
false /* transitionStarted */);
setKeyguardOccludedLw(occluded, false /* transitionStarted */);
}
}
@@ -3351,8 +3350,7 @@ public class PhoneWindowManager implements WindowManagerPolicy {
if (mKeyguardOccludedChanged) {
if (DEBUG_KEYGUARD) Slog.d(TAG, "transition/occluded changed occluded="
+ mPendingKeyguardOccluded);
if (setKeyguardOccludedLw(mPendingKeyguardOccluded, false /* force */,
transitionStarted)) {
if (setKeyguardOccludedLw(mPendingKeyguardOccluded, transitionStarted)) {
return FINISH_LAYOUT_REDO_LAYOUT | FINISH_LAYOUT_REDO_WALLPAPER;
}
}
@@ -3601,22 +3599,15 @@ public class PhoneWindowManager implements WindowManagerPolicy {
}
/**
* Updates the occluded state of the Keyguard.
* Updates the occluded state of the Keyguard immediately via
* {@link com.android.internal.policy.IKeyguardService}.
*
* @param isOccluded Whether the Keyguard is occluded by another window.
* @param force notify the occluded status to KeyguardService and update flags even though
* occlude status doesn't change.
* @param transitionStarted {@code true} if keyguard (un)occluded transition started.
* @return Whether the flags have changed and we have to redo the layout.
*/
private boolean setKeyguardOccludedLw(boolean isOccluded, boolean force,
boolean transitionStarted) {
private boolean setKeyguardOccludedLw(boolean isOccluded, boolean transitionStarted) {
if (DEBUG_KEYGUARD) Slog.d(TAG, "setKeyguardOccluded occluded=" + isOccluded);
mKeyguardOccludedChanged = false;
if (isKeyguardOccluded() == isOccluded && !force) {
return false;
}
final boolean showing = mKeyguardDelegate.isShowing();
final boolean animate = showing && !isOccluded;
// When remote animation is enabled for keyguard (un)occlude transition, KeyguardService
@@ -3624,7 +3615,10 @@ public class PhoneWindowManager implements WindowManagerPolicy {
// to notify here.
final boolean notify = !WindowManagerService.sEnableRemoteKeyguardOccludeAnimation
|| !transitionStarted;
mKeyguardDelegate.setOccluded(isOccluded, animate, notify);
if (notify) {
mKeyguardOccludedChanged = false;
mKeyguardDelegate.setOccluded(isOccluded, animate, notify);
}
return showing;
}