diff options
| author | 2024-09-05 17:51:10 +0200 | |
|---|---|---|
| committer | 2024-09-05 17:52:05 +0200 | |
| commit | 2944b92ea6e63aa46d5cdbe3a31364880cf927c3 (patch) | |
| tree | 2a7eb79752597be4f72d95e1726a21531607d246 | |
| parent | c83205653dd061ce7a4fb7c0cd7d9b51a7398459 (diff) | |
Add null check to EdgeBackGestureHandler#setBackAnimation
Bug: 364526759
Test: Manual, i.e. verify that back navigation works fine when BackAnimation is null
Flag: EXEMPT bugfix
Change-Id: I6ee307fd771ea17e6b1cd6ee2a129377754c5883
| -rw-r--r-- | packages/SystemUI/src/com/android/systemui/navigationbar/gestural/EdgeBackGestureHandler.java | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/navigationbar/gestural/EdgeBackGestureHandler.java b/packages/SystemUI/src/com/android/systemui/navigationbar/gestural/EdgeBackGestureHandler.java index 6bd880d56bbb..89d76f0038ee 100644 --- a/packages/SystemUI/src/com/android/systemui/navigationbar/gestural/EdgeBackGestureHandler.java +++ b/packages/SystemUI/src/com/android/systemui/navigationbar/gestural/EdgeBackGestureHandler.java @@ -29,6 +29,7 @@ import static com.android.systemui.shared.system.QuickStepContract.SYSUI_STATE_T import static java.util.stream.Collectors.joining; import android.annotation.NonNull; +import android.annotation.Nullable; import android.app.ActivityManager; import android.content.ComponentName; import android.content.Context; @@ -1331,21 +1332,20 @@ public class EdgeBackGestureHandler implements PluginListener<NavigationEdgeBack } } - public void setBackAnimation(BackAnimation backAnimation) { + public void setBackAnimation(@Nullable BackAnimation backAnimation) { mBackAnimation = backAnimation; - mBackAnimation.setPilferPointerCallback(() -> { - pilferPointers(); - }); - mBackAnimation.setTopUiRequestCallback( - (requestTopUi, tag) -> mUiThreadContext.getExecutor().execute(() -> - mNotificationShadeWindowController.setRequestTopUi(requestTopUi, tag))); - updateBackAnimationThresholds(); - if (mLightBarControllerProvider.get() != null) { - mBackAnimation.setStatusBarCustomizer((appearance) -> { - mUiThreadContext.getExecutor().execute(() -> - mLightBarControllerProvider.get() - .customizeStatusBarAppearance(appearance)); - }); + if (backAnimation != null) { + backAnimation.setPilferPointerCallback(this::pilferPointers); + backAnimation.setTopUiRequestCallback( + (requestTopUi, tag) -> mUiThreadContext.getExecutor().execute(() -> + mNotificationShadeWindowController.setRequestTopUi(requestTopUi, tag))); + updateBackAnimationThresholds(); + if (mLightBarControllerProvider.get() != null) { + mBackAnimation.setStatusBarCustomizer((appearance) -> + mUiThreadContext.getExecutor().execute(() -> + mLightBarControllerProvider.get() + .customizeStatusBarAppearance(appearance))); + } } } |