Fix RootNodeOwner.OwnerImpl.onRequestRelayout to only schedule layout, not measure

[m-sasha]

Currently, OwnerImpl.onRequestRelayout simply calls onRequestMeasure. This means that when a layout scope is invalidated, we will schedule a measuring too.

This is unnecessary, and in some cases (when the measure scope invalidates the layout scope) causes infinite re-layout.

Fixes JetBrains/compose-multiplatform#4760

Testing

• Added new unit tests
• Tested the user-provided code, as well as a reproducer:

fun main() = singleWindowApplication {
    val state = remember { mutableStateOf(1) }
    Layout(
        measurePolicy = { measurables, constraints ->
            val measurable = measurables.first()
            val placeable = measurable.measure(constraints)
            val prevValue = Snapshot.withoutReadObservation {
                state.value
            }
            state.value = prevValue+1
            layout(100, 100) {
                state.value  // Read the state value!
                placeable.placeRelative(0, 0)
            }
        },
        content = {
            Text("Hello")
        }
    )
}

This could be tested by QA

Release Notes

Fixes - Multiple Platforms

• Fix endless re-layout when layout is invalidated by measure, which includes measuring TextField(singleLine=true) with IntrinsicSize.

[m-sasha]

I've also renamed SnapshotInvalidationTracker.requestLayout to requestMeasureAndLayout (and the corresponding flag), because that's what it actually does.