Commit 62a59806 authored Aug 21, 2025 by András Kurucz

[Flexiglass] Ignore lookahead pass for the notifications top callback

SingleShadeMeasurePolicy was reporting measurement results during both
the main and lookahead layout passes. This caused conflicting values for
the max scrim top calculation, which resulted in a visual flicker while
transitioning from Shade to Gone.

This CL makes sure that we're only using values from the main layout
pass, to provide a consistent value that reflects the current UI state.

Fixes: 437882570
Test: Slowly close SingleShade -> Observe the EmptyShadeView positioning
Flag: com.android.systemui.scene_container
Change-Id: Ic4ebb7147ea92670968ec28a86bda82049141a13

parent 3a575129

packages/SystemUI/compose/features/src/com/android/systemui/shade/ui/composable/SingleShadeMeasurePolicy.kt

+5 −1

Original line number	Diff line number	Diff line
		@@ -71,7 +71,11 @@ class SingleShadeMeasurePolicy(
		mediaAndQqsPlaceable = mediaAndQqsPlaceable,
		insetsTop = insetsTop,
		)
		// Don't send position updates during the lookahead pass, as it can report a value that is
		// not yet reflected in the UI.
		if (!isLookingAhead) {
		onNotificationsTopChanged(notificationsTop)
		}

		return layout(constraints.maxWidth, constraints.maxHeight) {
		shadeHeaderPlaceable?.placeRelative(x = insetsLeft, y = insetsTop)