Browse Source

fix: Potential TransformGroup memory leak (#13529)

* fix: Potential TransformGroup memoey leak

* fix: Address review

* fix: Address review

* fix: review
pull/13677/head
workgroupengineering 2 years ago
committed by GitHub
parent
commit
d2aa8e6520
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 44
      src/Avalonia.Base/Media/TransformGroup.cs

44
src/Avalonia.Base/Media/TransformGroup.cs

@ -1,3 +1,4 @@
using System;
using Avalonia.Collections;
using Avalonia.Metadata;
@ -11,25 +12,17 @@ namespace Avalonia.Media
public static readonly StyledProperty<Transforms> ChildrenProperty =
AvaloniaProperty.Register<TransformGroup, Transforms>(nameof(Children));
[System.Diagnostics.CodeAnalysis.SuppressMessage("AvaloniaProperty", "AVP1012",
private IDisposable? _childrenNotificationSubscription;
private readonly EventHandler _childTransformChangedHandler;
[System.Diagnostics.CodeAnalysis.SuppressMessage("AvaloniaProperty", "AVP1012",
Justification = "Collection properties shouldn't be set with SetCurrentValue.")]
public TransformGroup()
{
_childTransformChangedHandler = (_, _) => RaiseChanged();
Children = new Transforms();
Children.ResetBehavior = ResetBehavior.Remove;
Children.CollectionChanged += delegate
{
Children.ForEachItem(
(tr) => tr.Changed += ChildTransform_Changed,
(tr) => tr.Changed -= ChildTransform_Changed,
() => { });
};
}
private void ChildTransform_Changed(object? sender, System.EventArgs e)
{
this.RaiseChanged();
}
/// <summary>
/// Gets or sets the children.
@ -61,6 +54,31 @@ namespace Avalonia.Media
return result;
}
}
protected override void OnPropertyChanged(AvaloniaPropertyChangedEventArgs change)
{
base.OnPropertyChanged(change);
if (change.Property == ChildrenProperty)
{
_childrenNotificationSubscription?.Dispose();
if (change.OldValue is Transforms oldTransforms)
{
foreach (var item in oldTransforms)
{
item.Changed -= _childTransformChangedHandler;
}
}
if (change.NewValue is Transforms newTransforms)
{
// Ensure reset behavior is Remove
newTransforms.ResetBehavior = ResetBehavior.Remove;
_childrenNotificationSubscription = newTransforms.ForEachItem(
(tr) => tr.Changed += _childTransformChangedHandler,
(tr) => tr.Changed -= _childTransformChangedHandler,
() => { });
}
}
}
}
public sealed class Transforms : AvaloniaList<Transform>

Loading…
Cancel
Save