From ec7708a4f63e917d19cb559d0361fb5df7bb882b Mon Sep 17 00:00:00 2001 From: Jorim Jaggi Date: Mon, 10 Apr 2017 18:47:36 -0700 Subject: [PATCH] Follow-up for I8cbf1bfbebc917b03f528f040bb019d193228bbb Actually update when force is true. Test: Check occluded Keyguard. Bug: 36407040 Change-Id: I8efdbca0bececf5075da71b2efe246fda37ef610 --- .../com/android/server/policy/PhoneWindowManager.java | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/services/core/java/com/android/server/policy/PhoneWindowManager.java b/services/core/java/com/android/server/policy/PhoneWindowManager.java index 95fb5af37b602..7505035f8068b 100644 --- a/services/core/java/com/android/server/policy/PhoneWindowManager.java +++ b/services/core/java/com/android/server/policy/PhoneWindowManager.java @@ -5475,9 +5475,10 @@ public class PhoneWindowManager implements WindowManagerPolicy { */ private boolean setKeyguardOccludedLw(boolean isOccluded, boolean force) { if (DEBUG_KEYGUARD) Slog.d(TAG, "setKeyguardOccluded occluded=" + isOccluded); - boolean wasOccluded = mKeyguardOccluded; - boolean showing = mKeyguardDelegate.isShowing(); - if (wasOccluded && !isOccluded && showing) { + final boolean wasOccluded = mKeyguardOccluded; + final boolean showing = mKeyguardDelegate.isShowing(); + final boolean changed = wasOccluded != isOccluded || force; + if (!isOccluded && changed && showing) { mKeyguardOccluded = false; mKeyguardDelegate.setOccluded(false, true /* animate */); if (mStatusBar != null) { @@ -5487,7 +5488,7 @@ public class PhoneWindowManager implements WindowManagerPolicy { } } return true; - } else if (!wasOccluded && isOccluded && showing) { + } else if (isOccluded && changed && showing) { mKeyguardOccluded = true; mKeyguardDelegate.setOccluded(true, false /* animate */); if (mStatusBar != null) { @@ -5495,7 +5496,7 @@ public class PhoneWindowManager implements WindowManagerPolicy { mStatusBar.getAttrs().flags &= ~FLAG_SHOW_WALLPAPER; } return true; - } else if (wasOccluded != isOccluded) { + } else if (changed) { mKeyguardOccluded = isOccluded; mKeyguardDelegate.setOccluded(isOccluded, false /* animate */); return false;