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

Nullcheck for StatusBarManagerInternal in InsetsPolicy am: 6426637598

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15747425

Change-Id: Ibe388af5ffb7650afc14e76efda6e3b5627a0f75
This commit is contained in:
shreerag@google.com
2021-09-03 08:53:47 +00:00
committed by Automerger Merge Worker
parent cec212c421 6426637598
commit 488f7d6a49
2 changed files with 27 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
services/core/java/com/android/server/wm/DisplayPolicy.java
View File

@@ -2273,7 +2273,10 @@ public class DisplayPolicy {
void notifyDisplayReady() { void notifyDisplayReady() {
mHandler.post(() -> { mHandler.post(() -> {
final int displayId = getDisplayId(); final int displayId = getDisplayId();
getStatusBarManagerInternal().onDisplayReady(displayId); StatusBarManagerInternal statusBar = getStatusBarManagerInternal();
if (statusBar != null) {
statusBar.onDisplayReady(displayId);
}
final WallpaperManagerInternal wpMgr = LocalServices final WallpaperManagerInternal wpMgr = LocalServices
.getService(WallpaperManagerInternal.class); .getService(WallpaperManagerInternal.class);
if (wpMgr != null) { if (wpMgr != null) {

32
services/core/java/com/android/server/wm/InsetsPolicy.java
View File

@@ -54,6 +54,7 @@ import android.view.WindowManager;
import com.android.internal.R; import com.android.internal.R;
import com.android.internal.annotations.VisibleForTesting; import com.android.internal.annotations.VisibleForTesting;
import com.android.server.DisplayThread; import com.android.server.DisplayThread;
import com.android.server.statusbar.StatusBarManagerInternal;
/** /**
* Policy that implements who gets control over the windows generating insets. * Policy that implements who gets control over the windows generating insets.
@@ -170,8 +171,12 @@ class InsetsPolicy {
changed = true; changed = true;
} }
if (changed) { if (changed) {
mPolicy.getStatusBarManagerInternal().showTransient(mDisplayContent.getDisplayId(), StatusBarManagerInternal statusBarManagerInternal =
mShowingTransientTypes.toArray()); mPolicy.getStatusBarManagerInternal();
if (statusBarManagerInternal != null) {
statusBarManagerInternal.showTransient(
mDisplayContent.getDisplayId(), mShowingTransientTypes.toArray());
}
updateBarControlTarget(mFocusedWin); updateBarControlTarget(mFocusedWin);
// The leashes can be created while updating bar control target. The surface transaction // The leashes can be created while updating bar control target. The surface transaction
@@ -306,9 +311,11 @@ class InsetsPolicy {
abortTypes.add(type); abortTypes.add(type);
} }
} }
if (abortTypes.size() > 0) { StatusBarManagerInternal statusBarManagerInternal =
mPolicy.getStatusBarManagerInternal().abortTransient(mDisplayContent.getDisplayId(), mPolicy.getStatusBarManagerInternal();
abortTypes.toArray()); if (abortTypes.size() > 0 && statusBarManagerInternal != null) {
statusBarManagerInternal.abortTransient(
mDisplayContent.getDisplayId(), abortTypes.toArray());
} }
} }
} }
@@ -318,8 +325,11 @@ class InsetsPolicy {
* updateBarControlTarget(mFocusedWin) after this invocation. * updateBarControlTarget(mFocusedWin) after this invocation.
*/ */
private void abortTransient() { private void abortTransient() {
mPolicy.getStatusBarManagerInternal().abortTransient(mDisplayContent.getDisplayId(), StatusBarManagerInternal statusBarManagerInternal = mPolicy.getStatusBarManagerInternal();
mShowingTransientTypes.toArray()); if (statusBarManagerInternal != null) {
statusBarManagerInternal.abortTransient(
mDisplayContent.getDisplayId(), mShowingTransientTypes.toArray());
}
mShowingTransientTypes.clear(); mShowingTransientTypes.clear();
} }
@@ -478,8 +488,12 @@ class InsetsPolicy {
final int state = visible ? WINDOW_STATE_SHOWING : WINDOW_STATE_HIDDEN; final int state = visible ? WINDOW_STATE_SHOWING : WINDOW_STATE_HIDDEN;
if (mState != state) { if (mState != state) {
mState = state; mState = state;
mPolicy.getStatusBarManagerInternal().setWindowState( StatusBarManagerInternal statusBarManagerInternal =
mDisplayContent.getDisplayId(), mId, state); mPolicy.getStatusBarManagerInternal();
if (statusBarManagerInternal != null) {
statusBarManagerInternal.setWindowState(
mDisplayContent.getDisplayId(), mId, state);
}
} }
} }
} }