Just trigger SizeChanged when Window size changes

[pictos]

Description of Change

Issues Fixed

Fixes #17991
Fixes #21306

[jsuarezruiz]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[jsuarezruiz]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[PureWeen]

I chatted with @mattleibow about this one a bit, and I think my suggestions are somewhat just digging us into a deeper hole here :-)

I think we've been trying to fix this issue the wrong way for some time now and need to revert this entire method back to what it originally looked like here

https://github.com/dotnet/maui/pull/4942/files#diff-07d5a2c0b5069d448623e88d890e41bc2c21e12f99bd01f1349903a332d4130cR202-R222

And then change this method

private protected override void UpdateHandlerValue(string property)
		{
			if (disableHandlerUpdate)
				return;

			base.UpdateHandlerValue(property);
		}

to

private protected override void UpdateHandlerValue(string property)
		{
			if (_batchFrameUpdate > 0)
				return;

			base.UpdateHandlerValue(property);
		}

[PureWeen]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[pictos]

Looks like the error are unrelated with my changes, from logs I could see something about Cake

[MartyIX]

I wonder if this PR fixes #21306 as well. My PR #21357 touches src/Controls/src/Core/Window/Window.cs as this PR.

I don't have my macOS machine with me now to test it but to verify if the bug is fixed or not, one can:

1. Run src/Controls/samples/Controls.Sample project
2. In the app, navigate to Core -> Multi-Window and click Set window size to 700x500 and
3. the application crashes.

[mattleibow]

@MartyIX it did not it seems. I added a comment to the issue with a tiny bit more detail, but yeah, we need to fix this sooner rather than later.

[MartyIX]

@MartyIX it did not it seems. I added a comment to the issue with a tiny bit more detail, but yeah, we need to fix this sooner rather than later.

Yes, I added some debug info in #21306 (comment). So I think it's clear what happens. Not so much clear how to fix it (at least to me).

[malsabi]

In SR6 .NET 8 CollectionView SizeChanged is not executing after this fix I guess. It was working fine on SR5. Is there a way to detect when the CollectionView size is changed ?

[mattleibow]

@malsabi could you file a regression issue? Then we can go this ASAP.

[malsabi]

@mattleibow Here you go #23029

[mattleibow]

Thanks, I will have a look.