Fix RealParent garbage collection warning to reduce noise in production apps #30291

Copilot · 2025-06-28T02:32:50Z

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Description

This PR fixes the noisy "The RealParent on Microsoft.Maui.Controls.Shapes.RoundRectangle has been Garbage Collected" warnings that have been flooding production app logs since .NET MAUI SR6, particularly on iOS.

Problem

Users were experiencing repeated warning messages like:

Microsoft.Maui.Controls.Element: Warning: The RealParent on Microsoft.Maui.Controls.Shapes.RoundRectangle has been Garbage Collected. This should never happen. Please log a bug: https://github.com/dotnet/maui

These warnings would appear hundreds of times in production apps, making logs difficult to read and potentially impacting performance.

Root Cause

The issue occurs when:

A parent-child relationship is established using weak references in the RealParent property
The parent gets garbage collected while the child element still exists
The child tries to access RealParent, finds the weak reference target is null
A warning is logged every time this happens, with no cleanup of the invalid reference

Solution

Enhanced defensive handling in Element.RealParent getter:

Clear invalid weak references: When a weak reference target is garbage collected, immediately set _realParent = null to prevent repeated warnings on subsequent accesses
Conditional logging: Only log warnings in DEBUG builds using #if DEBUG to reduce noise in production while still helping developers catch issues during development
Preserve behavior: Maintain the same return value (null) and overall functionality to ensure backward compatibility

Changes Made

Core Fix (`Element.cs`)

// Before: Would log warning on every access after GC
if (_realParent.TryGetTarget(out var parent))
{
    return parent;
}
else
{
    Application.Current?
        .FindMauiContext()?
        .CreateLogger<Element>()?
        .LogWarning($"The RealParent on {this} has been Garbage Collected...");
}

// After: Clear reference and only log in debug builds  
if (_realParent.TryGetTarget(out var parent))
{
    return parent;
}
else
{
    // Clear the weak reference since the target has been garbage collected
    _realParent = null;

    // Only log warning in debug builds to reduce noise in production
#if DEBUG
    Application.Current?
        .FindMauiContext()?
        .CreateLogger<Element>()?
        .LogWarning($"The RealParent on {this} has been Garbage Collected...");
#endif
}

Comprehensive Tests (`RealParentGCTests.cs`)

Added 4 focused test cases covering:

GC scenario handling without exceptions
Valid reference behavior
Null parent handling
Multiple access scenarios after GC

Testing

✅ All existing 473 Element-related tests pass
✅ All existing ParentLifeCycle tests pass
✅ New comprehensive test suite covers edge cases
✅ Build succeeds with no breaking changes

Impact

For Users:

✅ Eliminates noisy warning spam in production app logs
✅ Maintains full backward compatibility
✅ Improves app performance by avoiding repeated logging

For Developers:

✅ Still logs warnings in DEBUG builds to catch real issues
✅ Better weak reference lifecycle management
✅ Comprehensive test coverage for future maintenance

This is a minimal, surgical fix that addresses the immediate pain point while preserving the underlying diagnostic capability for development scenarios.

Fixes #23050.

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

PureWeen

Include the fixes from this PR
#30156

But also improve the changes where you see fit to

PureWeen · 2025-06-28T12:18:21Z

src/Controls/src/Core/Element/Element.cs

+					_realParent = null;
+
+					// Only log warning in debug builds to reduce noise in production
+#if DEBUG


Remove this, we should log a warning in release builds as well

Removed the #if DEBUG conditional compilation. Warnings now log in both debug and release builds. (commit 1c53139)

Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot · 2025-06-28T12:33:22Z

Include the fixes from this PR
#30156

...

Incorporated the fixes from PR #30156 including the GetRealParent method refactor, conditional warning logging parameter, and improved SetParent logic. Also removed the #if DEBUG conditional as requested. (commit 1c53139)

PureWeen · 2025-06-28T16:56:48Z

Can you update the copilot instructions to indicate that if there's a PR already opened for an issue you are assigned to that you didn't create, that you will review the changes in that other PR and give your opinion on it? If you feel like the solution in the other PR is valid then use that PR to influence what you do but then see where you can improve and add tests

PureWeen · 2025-06-28T16:57:38Z

src/Controls/src/Core/Element/Element.cs

@@ -397,7 +405,7 @@ void SetParent(Element value)

 			if (_parentOverride == null)
 			{
-				OnParentChangingCore(Parent, value);
+				OnParentChangingCore(ParentOverride ?? realParent, value);


Does this cause duplication of logic? Since we're also doing this inside Parent.get is there a way to consolidate that logic?

You're absolutely right. I've consolidated the logic by changing SetParent to use the Parent property instead of calling GetRealParent(false) separately. This eliminates the duplication and ensures consistent behavior between Parent.get and SetParent. (commit f8a79e9)

Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot · 2025-06-28T17:04:12Z

Can you update the copilot instructions to indicate that if there's a PR already opened for an issue you are assigned to that you didn't create, that you will review the changes in that other PR and g...

Updated the copilot instructions to include guidance for AI agents on handling existing PRs for assigned issues. The new section covers reviewing existing solutions, using valid work as foundation, and identifying improvement opportunities rather than duplicating efforts. (commit f5545ed)

Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

PureWeen · 2025-06-28T17:38:48Z

I want to test out the updated copilot instructions

Can you start this PR completely over as if you were just assigned this issue?
Except use the updated copilot instructions that are part of this PR

…ase on PR #30156 with improvements Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot · 2025-06-28T17:53:37Z

I want to test out the updated copilot instructions

Can you start this PR completely over as if you were just assigned this issue?
...

Started completely fresh following the updated copilot instructions! Successfully reviewed PR #30156, validated its approach, used it as foundation, and enhanced it with comprehensive improvements including weak reference cleanup and robust test coverage. (commit 845fb06)

PureWeen · 2025-06-28T18:23:28Z

This is looking good

Can you add some more notes in copilot instructions just indicating that you shouldn't try too hard to base your work off existing pr

It's ok to completely abandon the already open pr if you aren't confident enough of that approach. If you do abandon it make sure to add notes why you abandoned that approach. Always include a summary of what you think about any other open prs for the issue you are working on

…ity and abandonment guidance Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot · 2025-06-28T18:30:00Z

This is looking good

Can you add some more notes in copilot instructions just indicating that you shouldn't try too hard to base your work off existing pr ...

Enhanced the copilot instructions with additional flexibility guidance: added notes about not forcing existing PR approaches, making it clear that abandoning existing PRs is acceptable when not confident, requiring documentation of abandonment reasoning, and always including summaries of other open PRs. (commit a73d1ca)

PureWeen · 2025-06-28T18:36:34Z

I want to test out the updated copilot instructions

Can you start this PR completely over as if you were just assigned this issue?
Except use the updated copilot instructions that are part of this PR

…0156 with enhancements Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot · 2025-06-28T18:47:04Z

I want to test out the updated copilot instructions

Can you start this PR completely over as if you were just assigned this issue?
...

Started completely fresh following the updated copilot instructions! I thoroughly reviewed PR #30156, found their approach solid, and used it as my foundation while adding critical enhancements like weak reference cleanup. All 473 Element tests pass with the improved implementation. (commit a8e73ba)

Initial plan

83dfc14

Copilot AI assigned Copilot and PureWeen Jun 28, 2025

Copilot started work on behalf of PureWeen June 28, 2025 02:32 View session

Fix RealParent garbage collection warning issue

52b9c05

Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot AI requested a review from PureWeen June 28, 2025 02:51

Copilot finished work on behalf of PureWeen June 28, 2025 02:51

PureWeen requested changes Jun 28, 2025

View reviewed changes

Copilot started work on behalf of PureWeen June 28, 2025 12:18 View session

Remove DEBUG conditional and incorporate PR #30156 fixes

1c53139

Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot AI requested a review from PureWeen June 28, 2025 12:33

Copilot finished work on behalf of PureWeen June 28, 2025 12:33

Copilot started work on behalf of PureWeen June 28, 2025 16:56 View session

PureWeen reviewed Jun 28, 2025

View reviewed changes

Add guidance for handling existing PRs in copilot instructions

f5545ed

Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot AI requested a review from PureWeen June 28, 2025 17:04

Copilot finished work on behalf of PureWeen June 28, 2025 17:04

Copilot started work on behalf of PureWeen June 28, 2025 17:04 View session

Consolidate parent logic in SetParent to eliminate duplication

3c3e7fd

Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot finished work on behalf of PureWeen June 28, 2025 17:13

Copilot started work on behalf of PureWeen June 28, 2025 17:38 View session

Start fresh implementation following updated copilot instructions - b…

845fb06

…ase on PR #30156 with improvements Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot finished work on behalf of PureWeen June 28, 2025 17:54

Copilot started work on behalf of PureWeen June 28, 2025 18:23 View session

Enhance copilot instructions for handling existing PRs - add flexibil…

a73d1ca

…ity and abandonment guidance Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot finished work on behalf of PureWeen June 28, 2025 18:30

Copilot started work on behalf of PureWeen June 28, 2025 18:36 View session

Start fresh implementation: Fix RealParent GC warnings based on PR #3…

a8e73ba

…0156 with enhancements Co-authored-by: PureWeen <5375137+PureWeen@users.noreply.github.com>

Copilot finished work on behalf of PureWeen June 28, 2025 18:47

PureWeen mentioned this pull request Jun 28, 2025

[copilot] Create a PR with just the updated copilot instructions from PR 30291 #30297

Closed

Copilot AI mentioned this pull request Jun 28, 2025

[copilot] Extract copilot instructions updates from PR 30291 for handling existing PRs #30298

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Fix RealParent garbage collection warning to reduce noise in production apps #30291

Fix RealParent garbage collection warning to reduce noise in production apps #30291

Copilot AI commented Jun 28, 2025 •

edited

Loading

Uh oh!

PureWeen left a comment

Uh oh!

PureWeen Jun 28, 2025

Uh oh!

Copilot AI Jun 28, 2025

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

PureWeen commented Jun 28, 2025

Uh oh!

PureWeen Jun 28, 2025

Uh oh!

Copilot AI Jun 28, 2025

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

PureWeen commented Jun 28, 2025

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

PureWeen commented Jun 28, 2025

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

PureWeen commented Jun 28, 2025

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

Uh oh!

Fix RealParent garbage collection warning to reduce noise in production apps #30291

Are you sure you want to change the base?

Fix RealParent garbage collection warning to reduce noise in production apps #30291

Conversation

Copilot AI commented Jun 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Problem

Root Cause

Solution

Changes Made

Core Fix (Element.cs)

Comprehensive Tests (RealParentGCTests.cs)

Testing

Impact

Uh oh!

PureWeen left a comment

Choose a reason for hiding this comment

Uh oh!

PureWeen Jun 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jun 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

PureWeen commented Jun 28, 2025

Uh oh!

PureWeen Jun 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jun 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

PureWeen commented Jun 28, 2025

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

PureWeen commented Jun 28, 2025

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

PureWeen commented Jun 28, 2025

Uh oh!

Copilot AI commented Jun 28, 2025

Uh oh!

Uh oh!

Copilot AI commented Jun 28, 2025 •

edited

Loading

Core Fix (`Element.cs`)

Comprehensive Tests (`RealParentGCTests.cs`)