chore: prevent double Detach calls

Author: ramezgerges

src/Uno.UI/UI/Xaml/Controls/ContentPresenter/ContentPresenter.cs

@@ -925,7 +925,7 @@ internal override void LeaveImpl(LeaveParams @params)
 	{
 		base.LeaveImpl(@params);
 
-		if (IsNativeHost && _nativeElementAttached)
+		if (IsNativeHost)
 		{
 			DetachNativeElement(Content);
 		}

src/Uno.UI/UI/Xaml/Controls/ContentPresenter/ContentPresenter.skia.cs

@@ -90,12 +90,15 @@ partial void AttachNativeElement()
 	partial void DetachNativeElement(object content)
 	{
 #if DEBUG
-		global::System.Diagnostics.Debug.Assert(IsNativeHost && _nativeElementAttached);
-		_nativeElementAttached = false;
+		global::System.Diagnostics.Debug.Assert(IsNativeHost);
 #endif
 		_lastNativeArrangeArgs = null;
 		_nativeHosts.Remove(this);
-		_nativeElementHostingExtension.Value!.DetachNativeElement(content);
+		if (_nativeElementAttached)
+		{
+			_nativeElementAttached = false;
+			_nativeElementHostingExtension.Value!.DetachNativeElement(content);
+		}
 	}
 
 	private Size MeasureNativeElement(Size childMeasuredSize, Size availableSize)
@@ -153,6 +156,7 @@ internal static void OnNativeHostsRenderOrderChanged(List<Visual> nativeVisualsI
 				// We're detaching the native element as it's no longer in view, but conceptually, it's still in the tree, so IsNativeHost is still true
 				Debug.Assert(host.IsNativeHost);
 				host._nativeElementAttached = false;
+				host._lastNativeArrangeArgs = null;
 				host._nativeElementHostingExtension.Value!.DetachNativeElement(host.Content);
 			}
 			else if (host._nativeElementAttached)