[windows] fix memory leak when CollectionView.ItemsSource changes

src/Controls/src/Core/Handlers/Items/ItemsViewHandler.Windows.cs

@@ -123,6 +123,7 @@ protected virtual void CleanUpCollectionViewSource()
 				CollectionViewSource.Source = null;
 				CollectionViewSource = null;
 			}
+			VirtualView?.ClearLogicalChildren();
 
 			if (VirtualView?.ItemsSource == null)
 			{

src/Controls/src/Core/Items/ItemsView.cs

@@ -142,6 +142,15 @@ public void RemoveLogicalChild(Element element)
 			VisualDiagnostics.OnChildRemoved(this, element, oldLogicalIndex);
 		}
 
+		internal void ClearLogicalChildren()
+		{
+			// Reverse for-loop, so children can be removed while iterating
+			for (int i = _logicalChildren.Count - 1; i >= 0; i--)
+			{
+				RemoveLogicalChild(_logicalChildren[i]);
+			}
+		}
+
 		internal override IReadOnlyList<Element> LogicalChildrenInternal => _logicalChildren.AsReadOnly();
 
 		internal static readonly BindableProperty InternalItemsLayoutProperty =

src/Controls/tests/DeviceTests/Elements/CollectionView/CollectionViewTests.cs

@@ -1,4 +1,9 @@
-using Microsoft.Maui.Controls;
+using System;
+using System.Collections;
+using System.Collections.ObjectModel;
+using System.Reflection;
+using System.Threading.Tasks;
+using Microsoft.Maui.Controls;
 using Microsoft.Maui.Controls.Handlers.Items;
 using Microsoft.Maui.DeviceTests.Stubs;
 using Microsoft.Maui.Handlers;
@@ -28,5 +33,49 @@ void SetupBuilder()
 				});
 			});
 		}
+
+		[Fact]
+		public async Task ItemsSourceDoesNotLeak()
+		{
+			SetupBuilder();
+
+			IList logicalChildren = null;
+			WeakReference weakReference = null;
+			var collectionView = new CollectionView
+			{
+				ItemTemplate = new DataTemplate(() => new Label())
+			};
+
+			await CreateHandlerAndAddToWindow<CollectionViewHandler>(collectionView, async handler =>
+			{
+				var data = new ObservableCollection<string>()
+				{
+					"Item 1",
+					"Item 2",
+					"Item 3"
+				};
+				weakReference = new WeakReference(data);
+				collectionView.ItemsSource = data;
+				await Task.Delay(100);
+
+				// Get ItemsView._logicalChildren
+				var flags = BindingFlags.NonPublic | BindingFlags.Instance;
+				logicalChildren = typeof(ItemsView).GetField("_logicalChildren", flags).GetValue(collectionView) as IList;
+				Assert.NotNull(logicalChildren);
+
+				// Replace with cloned collection
+				collectionView.ItemsSource = new ObservableCollection<string>(data);
+				await Task.Delay(100);
+			});
+
+			await Task.Yield();
+			GC.Collect();
+			GC.WaitForPendingFinalizers();
+
+			Assert.NotNull(weakReference);
+			Assert.False(weakReference.IsAlive, "ObservableCollection should not be alive!");
+			Assert.NotNull(logicalChildren);
+			Assert.True(logicalChildren.Count <= 3, "_logicalChildren should not grow in size!");
+		}
 	}
 }