iOS PopModalAsync causes memory leaks

[era-maui]

Description

Whne you push a modal page that is wrapped in a NavigationPage
Navigation.PushModalAsync(new NavigationPage(new ModularPage())); on iOS we can see that the Page is never released from the memory. We are using Perfiew for debugging.

Here is a GCDump where we open and close a modal page multiple times
20240326_110245_0.zip

I am adding a sample which has both solutions outcommented. Seems counterintuitive that we need to call DisconnectHandler on existing pages and controls when popping a page from the Navigation stack.

NavigationSample.zip

Steps to Reproduce

1. Open sample app
2. Start profiling
3. Press the "Click me" button
4. Press "Close" button
5. Repeat steps 3 and 4
6. MAke a memory GCdump

Link to public reproduction project repository

No response

Version with bug

8.0.6 SR1

Is this a regression from previous behavior?

Yes, this used to work in Xamarin.Forms

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

No response

Did you find any workaround?

Seems that this can only be fixed by running disconnectHanler on each element on the Page including the page

page.Handler?.DisconnectHandler();

MyLayout.Handler?.DisconnectHandler();

Handler?.DisconnectHandler();

or creating a custom handler for the NavigationPage and running

ViewController.Dispose();
var viewController = ViewController as PageViewController;
viewController.CurrentView = null;

Relevant log output

No response

[AdamEssenmacher]

Possible duplicate or related:

#20094
#20119

[mattleibow]

/similarissues

[github-actions]

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Open similar issues:

• Page-level memory leak on modal navigation on iOS (#20094), similarity score: 0.84
• Page-level memory leak in NavigationPage on iOS (#20119), similarity score: 0.81

Closed similar issues:

• Page weak reference is still alive after pushing and popping pages modally or normally (#21143), similarity score: 0.78
• Memory Leak on Page Navigation while removing the page. (#21403), similarity score: 0.78
• Navigation.PopModalAsync not working on iOS (#3959), similarity score: 0.78

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

[PureWeen]

Can you test with the latest nightly build?
https://github.com/dotnet/maui/wiki/Nightly-Builds

[brentpbc]

I just tried with 8.0.60-ci.net8.24303.1 which seems to be the latest on https://dev.azure.com/xamarin/public/_artifacts/feed/maui-nightly

But creating a modal page with a NavigationPage still stops the page from being collected by the garbage collector on a real iOS device. A plain modal with no NavigationPage does get cleaned up after GC Collect.

GC.Collect();
GC.WaitForPendingFinalizers(); 

See attached test app. To test tap one of the page buttons, then navigate back and tap the Collect GC button. The isAlive should return to false and the text should change from red to black if it has been successfully cleaned up by the garbage collector. The destructors on the BasePage and BaseViewModel classes should also emit a Debug.Writeline() to the console when they are cleaned up.

MemoryTestApp.zip

[jonathanpeppers]

NavigationPage has circular references, which would be unrelated to modals:

• [ios/catalyst] fix leak in NavigationPage #22810

You can just do this to see it:

// The original NavigationPage & children leak
Application.Current.MainPage = new NavigationPage(new Page1());
Application.Current.MainPage = new Page2();