Recreate TextLayout on measure to make sure constraint is updated (#1…

…4705)

* Recreate TextLayout on measure to make sure constraint is updated

* Add failing test

src/Avalonia.Controls/TextBlock.cs

@@ -677,7 +677,6 @@ protected override void OnMeasureInvalidated()
         {
             _textLayout?.Dispose();
             _textLayout = null;
-
             _textRuns = null;
 
             base.OnMeasureInvalidated();
@@ -690,6 +689,10 @@ protected override Size MeasureOverride(Size availableSize)
 
             _constraint = availableSize.Deflate(padding);
 
+            //Reset TextLayout otherwise constraint might be outdated.
+            _textLayout?.Dispose();
+            _textLayout = null;
+
             var inlines = Inlines;
 
             if (HasComplexContent)
@@ -819,15 +822,19 @@ private void OnInlinesChanged(InlineCollection? oldValue, InlineCollection? newV
             {
                 oldValue.LogicalChildren = null;
                 oldValue.InlineHost = null;
-                oldValue.Invalidated -= (s, e) => InvalidateMeasure();
+                oldValue.Invalidated -= Invalidated;
             }
 
             if (newValue is not null)
             {
                 newValue.LogicalChildren = LogicalChildren;
                 newValue.InlineHost = this;
-                newValue.Invalidated += (s, e) => InvalidateMeasure();
+                newValue.Invalidated += Invalidated;
             }
+
+            return;
+
+            void Invalidated(object? sender, EventArgs e) => InvalidateMeasure();
         }
 
         void IInlineHost.Invalidate()

tests/Avalonia.Controls.UnitTests/TextBlockTests.cs

@@ -30,6 +30,27 @@ public void Default_Text_Value_Should_Be_Null()
             Assert.Equal(null, textBlock.Text);
         }
 
+        [Fact]
+        public void Calling_Measure_Should_Update_Constraint_And_TextLayout()
+        {
+            using (UnitTestApplication.Start(TestServices.MockPlatformRenderInterface))
+            {
+                var textBlock = new TestTextBlock { Text = "Hello World" };
+
+                textBlock.Measure(new Size(100, 100));
+
+                var textLayout = textBlock.TextLayout;
+
+                Assert.Equal(new Size(100,100), textBlock.Constraint);
+
+                textBlock.Measure(new Size(50, 100));
+
+                Assert.Equal(new Size(50, 100), textBlock.Constraint);
+
+                Assert.NotEqual(textLayout, textBlock.TextLayout);
+            }
+        }
+
         [Fact]
         public void Changing_InlinesCollection_Should_Invalidate_Measure()
         {
@@ -328,5 +349,10 @@ public void TextBlock_TextLines_Should_Be_Empty()
                 Assert.NotEqual(count, count1);
             }
         }
+
+        private class TestTextBlock : TextBlock
+        {
+            public Size Constraint => _constraint;
+        }
     }
 }