Fix critical modal page leak on Android, and several other small memory leaks (#818)

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Author: Vetle444

CHANGELOG.md

@@ -1,3 +1,9 @@
+## [55.2.7]
+- [Android][Modal] Fixed memory leak when popping modal pages wrapped in NavigationPage, caused by StackNavigationManager not clearing references on disconnect (dotnet/maui#34456)
+- [Android][Button] Fixed memory leak caused by strong reference from platform MaterialButton to virtual Button
+- [Android][SearchBar] Fixed memory leak caused by bindings and event handlers not being cleaned up on disconnect
+- [iOS][SearchBar] Fixed incorrect event unsubscription (SearchButtonClicked was unsubscribed as CancelButtonClicked)
+
 ## [55.2.6]
 - [ContextMenu] Fixed that in some cases context menu never appears when tapping.
 

src/app/MemoryLeakTests/Tests/Components/ActivityIndicatorTest.cs

@@ -0,0 +1,14 @@
+using DIPS.Mobile.UI.Components.Loading;
+using ActivityIndicator = DIPS.Mobile.UI.Components.Loading.ActivityIndicator;
+
+namespace MemoryLeakTests.Tests;
+
+public class ActivityIndicatorTest : UITest
+{
+    public override void BeforeTest(ContentPage contentPage)
+    {
+        contentPage.Content = new ActivityIndicator { IsRunning = true };
+    }
+
+    public override string Name => "ActivityIndicator";
+}

src/app/MemoryLeakTests/Tests/Components/ButtonTest.cs

@@ -0,0 +1,13 @@
+using Button = DIPS.Mobile.UI.Components.Buttons.Button;
+
+namespace MemoryLeakTests.Tests;
+
+public class ButtonTest : UITest
+{
+    public override void BeforeTest(ContentPage contentPage)
+    {
+        contentPage.Content = new Button { Text = "Button", Command = new Command(() => { }) };
+    }
+
+    public override string Name => "Button";
+}

src/app/MemoryLeakTests/Tests/Components/ChipTest.cs

@@ -0,0 +1,13 @@
+using DIPS.Mobile.UI.Components.Chips;
+
+namespace MemoryLeakTests.Tests;
+
+public class ChipTest : UITest
+{
+    public override void BeforeTest(ContentPage contentPage)
+    {
+        contentPage.Content = new Chip { Title = "Chip", Command = new Command(() => { }) };
+    }
+
+    public override string Name => "Chip";
+}

src/app/MemoryLeakTests/Tests/Components/CollectionViewTest.cs

@@ -0,0 +1,21 @@
+using DIPS.Mobile.UI.Components.Labels;
+using CollectionView = DIPS.Mobile.UI.Components.Lists.CollectionView;
+using Label = DIPS.Mobile.UI.Components.Labels.Label;
+
+namespace MemoryLeakTests.Tests;
+
+public class CollectionViewTest : UITest
+{
+    public override void BeforeTest(ContentPage contentPage)
+    {
+        var collectionView = new CollectionView
+        {
+            ItemsSource = new List<string> { "Item 1", "Item 2", "Item 3" },
+            ItemTemplate = new DataTemplate(() => new Label { Text = "Item" })
+        };
+
+        contentPage.Content = collectionView;
+    }
+
+    public override string Name => "CollectionView";
+}

src/app/MemoryLeakTests/Tests/Components/EditorTest.cs

@@ -0,0 +1,20 @@
+using Editor = DIPS.Mobile.UI.Components.TextFields.Editor.Editor;
+
+namespace MemoryLeakTests.Tests;
+
+public class EditorTest : UITest
+{
+    public override void BeforeTest(ContentPage contentPage)
+    {
+        contentPage.Content = new VerticalStackLayout
+        {
+            Children =
+            {
+                new Editor { Placeholder = "Editor 1" },
+                new Editor { Placeholder = "Editor 2" }
+            }
+        };
+    }
+
+    public override string Name => "Editor";
+}

src/app/MemoryLeakTests/Tests/Components/EntryTest.cs

@@ -0,0 +1,20 @@
+using Entry = DIPS.Mobile.UI.Components.TextFields.Entry.Entry;
+
+namespace MemoryLeakTests.Tests;
+
+public class EntryTest : UITest
+{
+    public override void BeforeTest(ContentPage contentPage)
+    {
+        contentPage.Content = new VerticalStackLayout
+        {
+            Children =
+            {
+                new Entry { Placeholder = "Entry 1" },
+                new Entry { Placeholder = "Entry 2" }
+            }
+        };
+    }
+
+    public override string Name => "Entry";
+}

src/app/MemoryLeakTests/Tests/Components/ImageTest.cs

@@ -0,0 +1,22 @@
+using DIPS.Mobile.UI.Components.Images;
+using DIPS.Mobile.UI.Resources.Icons;
+using Image = DIPS.Mobile.UI.Components.Images.Image;
+
+namespace MemoryLeakTests.Tests;
+
+public class ImageTest : UITest
+{
+    public override void BeforeTest(ContentPage contentPage)
+    {
+        contentPage.Content = new VerticalStackLayout
+        {
+            Children =
+            {
+                new Image.Image { Source = Icons.GetIcon(IconName.check_line) },
+                new ImageButton { Source = Icons.GetIcon(IconName.close_line) }
+            }
+        };
+    }
+
+    public override string Name => "Image";
+}

src/app/MemoryLeakTests/Tests/Components/InputFieldTest.cs

@@ -0,0 +1,21 @@
+using DIPS.Mobile.UI.Components.TextFields.InputFields;
+using DIPS.Mobile.UI.Components.TextFields.InputFields.MultiLineInputField;
+
+namespace MemoryLeakTests.Tests;
+
+public class InputFieldTest : UITest
+{
+    public override void BeforeTest(ContentPage contentPage)
+    {
+        contentPage.Content = new VerticalStackLayout
+        {
+            Children =
+            {
+                new SingleLineInputField { HeaderText = "Single line" },
+                new MultiLineInputField { HeaderText = "Multi line" }
+            }
+        };
+    }
+
+    public override string Name => "InputFields";
+}

src/app/MemoryLeakTests/Tests/Components/LabelTest.cs

@@ -0,0 +1,14 @@
+using DIPS.Mobile.UI.Components.Labels;
+using Label = DIPS.Mobile.UI.Components.Labels.Label;
+
+namespace MemoryLeakTests.Tests;
+
+public class LabelTest : UITest
+{
+    public override void BeforeTest(ContentPage contentPage)
+    {
+        contentPage.Content = new Label { Text = "Label" };
+    }
+
+    public override string Name => "Label";
+}