Merge "Remove spammy height update logs" into main (dc7443c0) · Commits · e / os / android_frameworks_base

packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackScrollLayout.java

+9 −3

Original line number	Diff line number	Diff line
		@@ -4698,9 +4698,8 @@ public class NotificationStackScrollLayout
		ExpandableView firstVisibleChild =
		firstSection == null ? null : firstSection.getFirstVisibleChild();
		if (row != null) {
		if (mLogger != null) {
		mLogger.childHeightUpdated(row, needsAnimation);
		}
		// TODO(b/424163539): child height updated logs are spammy, which hides other logs
		// logChildHeightUpdated(row, needsAnimation);
		if (row == firstVisibleChild
		\|\| row.getNotificationParent() == firstVisibleChild) {
		updateAlgorithmLayoutMinHeight();
		@@ -4714,6 +4713,13 @@ public class NotificationStackScrollLayout
		mAnimateStackYForContentHeightChange = previouslyNeededAnimation;
		}


		private void logChildHeightUpdated(ExpandableNotificationRow row, boolean needsAnimation) {
		if (mLogger != null) {
		mLogger.childHeightUpdated(row, needsAnimation);
		}
		}

		void onChildHeightReset(ExpandableView view) {
		updateAnimationState(view);
		updateChronometerForChild(view);

+2 −1

Original line number	Diff line number	Diff line
		@@ -138,7 +138,8 @@ constructor(

		/** An internal modification was made to notifications */
		fun notificationStackChanged() {
		logger?.notificationStackChanged()
		// TODO(b/424163539): child height updated logs are spammy, which hides other logs
		// logger?.notificationStackChanged()
		notificationStackChangedDebounced.value = notificationStackChangedDebounced.value + 1
		}