Merge "Improving comment inside expandWithQs for special split shade case" into tm-dev am: 7144969d6b

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

Change-Id: Icfb736cb8201976ad0b99aa7424067ef8d52c446
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Michał Brzeziński
2022-05-11 20:26:20 +00:00
committed by Automerger Merge Worker

View File

@@ -1693,9 +1693,11 @@ public class NotificationPanelViewController extends PanelViewController {
if (mShouldUseSplitNotificationShade && isOnKeyguard()) { if (mShouldUseSplitNotificationShade && isOnKeyguard()) {
// It's a special case as this method is likely to not be initiated by finger movement // It's a special case as this method is likely to not be initiated by finger movement
// but rather called from adb shell or accessibility service. // but rather called from adb shell or accessibility service.
// In the future method below could be used for non-split shade as well but currently // We're using LockscreenShadeTransitionController because on lockscreen that's the
// motion in that case looks worse than using flingSettings. // source of truth for all shade motion. Not using it would make part of state to be
// TODO: make below function transitioning smoothly also in portrait with empty target // outdated and will cause bugs. Ideally we'd use this controller also for non-split
// case but currently motion in portrait looks worse than when using flingSettings.
// TODO: make below function transitioning smoothly also in portrait with null target
mLockscreenShadeTransitionController.goToLockedShade( mLockscreenShadeTransitionController.goToLockedShade(
/* expandedView= */null, /* needsQSAnimation= */false); /* expandedView= */null, /* needsQSAnimation= */false);
} else if (isFullyCollapsed()) { } else if (isFullyCollapsed()) {