Merge "Fix possible NPE" into rvc-dev am: 1d63b6c88a

Change-Id: I71c90cec60ad152f13f0d67f51f7dde4b5a15178
This commit is contained in:
TreeHugger Robot
2020-04-01 03:07:27 +00:00
committed by Automerger Merge Worker
2 changed files with 8 additions and 3 deletions

View File

@@ -543,7 +543,8 @@ public class BubbleStackView extends FrameLayout {
.setStiffness(SpringForce.STIFFNESS_LOW)
.setDampingRatio(SpringForce.DAMPING_RATIO_LOW_BOUNCY));
mExpandedViewYAnim.addEndListener((anim, cancelled, value, velocity) -> {
if (mIsExpanded && mExpandedBubble != null) {
if (mIsExpanded && mExpandedBubble != null
&& mExpandedBubble.getExpandedView() != null) {
mExpandedBubble.getExpandedView().updateView();
}
});
@@ -562,7 +563,7 @@ public class BubbleStackView extends FrameLayout {
// Update the insets after we're done translating otherwise position
// calculation for them won't be correct.
() -> {
if (mExpandedBubble != null) {
if (mExpandedBubble != null && mExpandedBubble.getExpandedView() != null) {
mExpandedBubble.getExpandedView().updateInsets(insets);
}
});
@@ -577,7 +578,7 @@ public class BubbleStackView extends FrameLayout {
// Reposition & adjust the height for new orientation
if (mIsExpanded) {
mExpandedViewContainer.setTranslationY(getExpandedViewY());
if (mExpandedBubble != null) {
if (mExpandedBubble != null && mExpandedBubble.getExpandedView() != null) {
mExpandedBubble.getExpandedView().updateView();
}
}
@@ -1264,6 +1265,7 @@ public class BubbleStackView extends FrameLayout {
void showExpandedViewContents(int displayId) {
if (mExpandedBubble != null
&& mExpandedBubble.getExpandedView() != null
&& mExpandedBubble.getExpandedView().getVirtualDisplayId() == displayId) {
mExpandedBubble.setContentVisibility(true);
}

View File

@@ -356,6 +356,9 @@ public class PhysicsAnimationLayout extends FrameLayout {
/** Immediately re-orders the view to the given index. */
public void reorderView(View view, int index) {
if (view == null) {
return;
}
final int oldIndex = indexOfChild(view);
super.removeView(view);