Don't subscribe to inner observable if not active.

When subscribing to a `PropertySetterBindingInstance`, if the owner style is not active then there's no need to subscribe to the inner observable as this can cause resource lookups etc. Part of fixing #5027.
AvaloniaUI · Nov 17, 2020 · b3ed79a · b3ed79a
1 parent 5044684
commit b3ed79a
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/Avalonia.Styling/Styling/PropertySetterBindingInstance.cs b/src/Avalonia.Styling/Styling/PropertySetterBindingInstance.cs
@@ -92,6 +92,7 @@ public void Activate()
         {
             if (!_isActive)
             {
+                _innerSubscription ??= _binding.Observable.Subscribe(_inner);
                 _isActive = true;
                 PublishNext();
             }
@@ -102,6 +103,8 @@ public void Deactivate()
             if (_isActive)
             {
                 _isActive = false;
+                _innerSubscription?.Dispose();
+                _innerSubscription = null;
                 PublishNext();
             }
         }
@@ -148,7 +151,10 @@ void IObserver<BindingValue<T>>.OnNext(BindingValue<T> value)
 
         protected override void Subscribed()
         {
-            _innerSubscription = _binding.Observable.Subscribe(_inner);
+            if (_isActive)
+            {
+                _innerSubscription = _binding.Observable.Subscribe(_inner);
+            }
         }
 
         protected override void Unsubscribed()