Merge "Prevent NPE when dragging last bubble from expanded state" into rvc-dev am: d4952538cc (4234acc1) · Commits · e / os / android_frameworks_base

packages/SystemUI/src/com/android/systemui/bubbles/BubbleController.java

+11 −8

Original line number	Diff line number	Diff line
		@@ -714,6 +714,9 @@ public class BubbleController implements ConfigurationController.ConfigurationLi
		* the new params if the stack has been added.
		*/
		private void updateWmFlags() {
		if (mStackView == null) {
		return;
		}
		if (isStackExpanded() && !mImeVisible) {
		// If we're expanded, and the IME isn't visible, we want to be focusable. This ensures
		// that any taps within Bubbles (including on the ActivityView) results in Bubbles
		@@ -725,7 +728,7 @@ public class BubbleController implements ConfigurationController.ConfigurationLi
		mWmLayoutParams.flags \|= WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE;
		}

		if (mStackView != null && mAddedToWindowManager) {
		if (mAddedToWindowManager) {
		try {
		mWindowManager.updateViewLayout(mStackView, mWmLayoutParams);
		} catch (IllegalArgumentException e) {
		@@ -1245,24 +1248,23 @@ public class BubbleController implements ConfigurationController.ConfigurationLi
		}
		mDataRepository.removeBubbles(mCurrentUserId, bubblesToBeRemovedFromRepository);

		if (update.addedBubble != null) {
		if (update.addedBubble != null && mStackView != null) {
		mDataRepository.addBubble(mCurrentUserId, update.addedBubble);
		mStackView.addBubble(update.addedBubble);

		}

		if (update.updatedBubble != null) {
		if (update.updatedBubble != null && mStackView != null) {
		mStackView.updateBubble(update.updatedBubble);
		}

		// At this point, the correct bubbles are inflated in the stack.
		// Make sure the order in bubble data is reflected in bubble row.
		if (update.orderChanged) {
		if (update.orderChanged && mStackView != null) {
		mDataRepository.addBubbles(mCurrentUserId, update.bubbles);
		mStackView.updateBubbleOrder(update.bubbles);
		}

		if (update.selectionChanged) {
		if (update.selectionChanged && mStackView != null) {
		mStackView.setSelectedBubble(update.selectedBubble);
		if (update.selectedBubble != null && update.selectedBubble.getEntry() != null) {
		mNotificationGroupManager.updateSuppression(
		@@ -1272,8 +1274,10 @@ public class BubbleController implements ConfigurationController.ConfigurationLi

		// Expanding? Apply this last.
		if (update.expandedChanged && update.expanded) {
		if (mStackView != null) {
		mStackView.setExpanded(true);
		}
		}

		for (NotifCallback cb : mCallbacks) {
		cb.invalidateNotifications("BubbleData.Listener.applyUpdate");
		@@ -1378,7 +1382,6 @@ public class BubbleController implements ConfigurationController.ConfigurationLi
		}

		/**
		* Lets any listeners know if bubble state has changed.
		* Updates the visibility of the bubbles based on current state.
		* Does not un-bubble, just hides or un-hides.
		* Updates stack description for TalkBack focus.

packages/SystemUI/src/com/android/systemui/bubbles/BubbleData.java

+2 −1

Original line number	Diff line number	Diff line
		@@ -417,7 +417,8 @@ public class BubbleData {
		if (mBubbles.size() == 1) {
		// Going to become empty, handle specially.
		setExpandedInternal(false);
		setSelectedBubbleInternal(null);
		// Don't use setSelectedBubbleInternal because we don't want to trigger an applyUpdate
		mSelectedBubble = null;
		}
		if (indexToRemove < mBubbles.size() - 1) {
		// Removing anything but the last bubble means positions will change.

packages/SystemUI/src/com/android/systemui/bubbles/BubbleStackView.java

+3 −2

Original line number	Diff line number	Diff line
		@@ -93,7 +93,6 @@ import com.android.systemui.shared.system.QuickStepContract;
		import com.android.systemui.shared.system.SysUiStatsLog;
		import com.android.systemui.statusbar.notification.collection.NotificationEntry;
		import com.android.systemui.statusbar.phone.CollapsedStatusBarFragment;
		import com.android.systemui.statusbar.phone.NotificationShadeWindowController;
		import com.android.systemui.util.DismissCircleView;
		import com.android.systemui.util.FloatingContentCoordinator;
		import com.android.systemui.util.RelativeTouchListener;
		@@ -1515,7 +1514,9 @@ public class BubbleStackView extends FrameLayout
		// expanded view becomes visible on the screen. See b/126856255
		mExpandedViewContainer.setAlpha(0.0f);
		mSurfaceSynchronizer.syncSurfaceAndRun(() -> {
		if (previouslySelected != null) {
		previouslySelected.setContentVisibility(false);
		}
		updateExpandedBubble();
		requestUpdate();

packages/SystemUI/tests/src/com/android/systemui/bubbles/BubbleDataTest.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -537,7 +537,7 @@ public class BubbleDataTest extends SysuiTestCase {
		// Verify the selection was cleared.
		verifyUpdateReceived();
		assertThat(mBubbleData.isExpanded()).isFalse();
		assertSelectionCleared();
		assertThat(mBubbleData.getSelectedBubble()).isNull();
		}

		// EXPANDED / ADD / UPDATE