Support NativeAOT in PropertySetter (#49897)

* Support NativeAOT in PropertySetter

MakeGenericType and MakeGeneridMethod aren't guaranteed to work on NativeAOT when using value types.

When dynamic code isn't supported, just use normal reflection to set the property instead of creating generic delegates.

* Add test to ensure PropertySetter works when IsDynamicCodeSupported == false.

Author: eerhardt

src/Components/Components/src/Reflection/PropertySetter.cs

@@ -3,6 +3,7 @@
 
 using System.Diagnostics.CodeAnalysis;
 using System.Reflection;
+using System.Runtime.CompilerServices;
 
 namespace Microsoft.AspNetCore.Components.Reflection;
 
@@ -26,14 +27,21 @@ public PropertySetter(Type targetType, PropertyInfo property)
                 "has no setter.");
         }
 
-        var setMethod = property.SetMethod;
+        if (RuntimeFeature.IsDynamicCodeSupported)
+        {
+            var setMethod = property.SetMethod;
 
-        var propertySetterAsAction =
-            setMethod.CreateDelegate(typeof(Action<,>).MakeGenericType(targetType, property.PropertyType));
-        var callPropertySetterClosedGenericMethod =
-            CallPropertySetterOpenGenericMethod.MakeGenericMethod(targetType, property.PropertyType);
-        _setterDelegate = (Action<object, object>)
-            callPropertySetterClosedGenericMethod.CreateDelegate(typeof(Action<object, object>), propertySetterAsAction);
+            var propertySetterAsAction =
+                setMethod.CreateDelegate(typeof(Action<,>).MakeGenericType(targetType, property.PropertyType));
+            var callPropertySetterClosedGenericMethod =
+                CallPropertySetterOpenGenericMethod.MakeGenericMethod(targetType, property.PropertyType);
+            _setterDelegate = (Action<object, object>)
+                callPropertySetterClosedGenericMethod.CreateDelegate(typeof(Action<object, object>), propertySetterAsAction);
+        }
+        else
+        {
+            _setterDelegate = property.SetValue;
+        }
     }
 
     public bool AcceptsDirectParameters { get; init; }

src/Components/Components/test/ParameterViewTest.Assignment.cs

@@ -3,6 +3,8 @@
 
 using System.Collections;
 using Microsoft.AspNetCore.Components.Rendering;
+using Microsoft.AspNetCore.Testing;
+using Microsoft.DotNet.RemoteExecutor;
 
 namespace Microsoft.AspNetCore.Components;
 
@@ -598,6 +600,32 @@ public void SupplyingNullWritesDefaultForType()
         Assert.Null(target.StringProp);
     }
 
+    [ConditionalFact]
+    [RemoteExecutionSupported]
+    public void WorksWhenDynamicCodeNotSupported()
+    {
+        var options = new RemoteInvokeOptions();
+        options.RuntimeConfigurationOptions.Add("System.Runtime.CompilerServices.RuntimeFeature.IsDynamicCodeSupported", "false");
+
+        using var remoteHandle = RemoteExecutor.Invoke(static () =>
+        {
+            // Arrange
+            var parameters = new ParameterViewBuilder
+            {
+                { nameof(HasInstanceProperties.IntProp), 5 },
+                { nameof(HasInstanceProperties.StringProp), "Hello" },
+            }.Build();
+            var target = new HasInstanceProperties();
+
+            // Act
+            parameters.SetParameterProperties(target);
+
+            // Assert
+            Assert.Equal(5, target.IntProp);
+            Assert.Equal("Hello", target.StringProp);
+        }, options);
+    }
+
     class HasInstanceProperties
     {
         [Parameter] public int IntProp { get; set; }