Merge "Fix possible NPE" into sc-v2-dev

This commit is contained in:
Mady Mellor
2022-01-11 17:36:43 +00:00
committed by Android (Google) Code Review
2 changed files with 3 additions and 1 deletions

View File

@@ -231,8 +231,9 @@ public class BubbleFlyoutView extends FrameLayout {
* Fade animation for consecutive flyouts. * Fade animation for consecutive flyouts.
*/ */
void animateUpdate(Bubble.FlyoutMessage flyoutMessage, PointF stackPos, void animateUpdate(Bubble.FlyoutMessage flyoutMessage, PointF stackPos,
boolean hideDot, Runnable onHide) { boolean hideDot, float[] dotCenter, Runnable onHide) {
mOnHide = onHide; mOnHide = onHide;
mDotCenter = dotCenter;
final Runnable afterFadeOut = () -> { final Runnable afterFadeOut = () -> {
updateFlyoutMessage(flyoutMessage); updateFlyoutMessage(flyoutMessage);
// Wait for TextViews to layout with updated height. // Wait for TextViews to layout with updated height.

View File

@@ -2494,6 +2494,7 @@ public class BubbleStackView extends FrameLayout
if (mFlyout.getVisibility() == View.VISIBLE) { if (mFlyout.getVisibility() == View.VISIBLE) {
mFlyout.animateUpdate(bubble.getFlyoutMessage(), mFlyout.animateUpdate(bubble.getFlyoutMessage(),
mStackAnimationController.getStackPosition(), !bubble.showDot(), mStackAnimationController.getStackPosition(), !bubble.showDot(),
bubble.getIconView().getDotCenter(),
mAfterFlyoutHidden /* onHide */); mAfterFlyoutHidden /* onHide */);
} else { } else {
mFlyout.setVisibility(INVISIBLE); mFlyout.setVisibility(INVISIBLE);