Clean up

Author: maraf

src/Components/Endpoints/src/Assets/LinkPreload.cs

@@ -1,38 +1,35 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
-
-
-// Licensed to the .NET Foundation under one or more agreements.
-// The .NET Foundation licenses this file to you under the MIT license.
-
-
 using Microsoft.AspNetCore.Components.Endpoints;
 using Microsoft.AspNetCore.Components.Rendering;
 
 namespace Microsoft.AspNetCore.Components;
 
-public class LinkPreload : IComponent
+/// <summary>
+/// Represents link elements for preloading assets.
+/// </summary>
+public sealed class LinkPreload : IComponent
 {
     private RenderHandle renderHandle;
     private List<PreloadAsset>? assets;
 
     [Inject]
-    internal ResourcePreloadService Service { get; set; }
+    internal ResourcePreloadService? Service { get; set; }
 
-    public void Attach(RenderHandle renderHandle)
+    void IComponent.Attach(RenderHandle renderHandle)
     {
         this.renderHandle = renderHandle;
     }
 
-    public Task SetParametersAsync(ParameterView parameters)
+    Task IComponent.SetParametersAsync(ParameterView parameters)
     {
-        Service.SetPreloadHook(PreloadGroup);
+        Service?.SetPreloadingHandler(PreloadAssets);
         renderHandle.Render(RenderPreloadAssets);
         return Task.CompletedTask;
     }
 
-    private void PreloadGroup(List<PreloadAsset> assets)
+    private void PreloadAssets(List<PreloadAsset> assets)
     {
         if (this.assets != null)
         {

src/Components/Endpoints/src/Builder/ResourceCollectionConvention.cs

@@ -4,7 +4,6 @@
 using Microsoft.AspNetCore.Components;
 using Microsoft.AspNetCore.Components.Endpoints;
 using Microsoft.AspNetCore.Components.Web;
-using Microsoft.AspNetCore.Routing;
 
 namespace Microsoft.AspNetCore.Builder;
 

src/Components/Endpoints/src/Builder/ResourcePreloadCollection.cs

@@ -1,6 +1,8 @@
 // Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 
+using System.Diagnostics.CodeAnalysis;
+
 namespace Microsoft.AspNetCore.Components.Endpoints;
 
 internal class ResourcePreloadCollection
@@ -94,7 +96,7 @@ private static PreloadAsset CreateAsset(string url, IEnumerable<ResourceAssetPro
         return resourceAsset;
     }
 
-    public bool TryGetAssets(string group, out List<PreloadAsset> assets)
+    public bool TryGetAssets(string group, [MaybeNullWhen(false)] out List<PreloadAsset> assets)
         => _storage.TryGetValue(group, out assets);
 }
 

src/Components/Endpoints/src/Builder/ResourcePreloadService.cs

@@ -7,7 +7,7 @@ internal class ResourcePreloadService
 {
     private Action<List<PreloadAsset>>? handler;
 
-    public void SetPreloadHook(Action<List<PreloadAsset>> handler)
+    public void SetPreloadingHandler(Action<List<PreloadAsset>> handler)
         => this.handler = handler;
 
     public void Preload(List<PreloadAsset> assets)

src/Components/Endpoints/src/PublicAPI.Unshipped.txt

@@ -1,3 +1,5 @@
 #nullable enable
+Microsoft.AspNetCore.Components.LinkPreload
+Microsoft.AspNetCore.Components.LinkPreload.LinkPreload() -> void
 Microsoft.Extensions.DependencyInjection.RazorComponentsRazorComponentBuilderExtensions
 static Microsoft.Extensions.DependencyInjection.RazorComponentsRazorComponentBuilderExtensions.RegisterPersistentService<TPersistentService>(this Microsoft.Extensions.DependencyInjection.IRazorComponentsBuilder! builder, Microsoft.AspNetCore.Components.IComponentRenderMode! renderMode) -> Microsoft.Extensions.DependencyInjection.IRazorComponentsBuilder!

src/Components/Endpoints/src/Rendering/EndpointHtmlRenderer.Streaming.cs

@@ -12,7 +12,6 @@
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
 using Microsoft.Extensions.Options;
-using Microsoft.Extensions.Primitives;
 
 namespace Microsoft.AspNetCore.Components.Endpoints;
 

src/Components/Endpoints/src/Rendering/SSRRenderModeBoundary.cs

@@ -5,7 +5,6 @@
 using System.Diagnostics;
 using System.Diagnostics.CodeAnalysis;
 using System.Globalization;
-using System.Net.Http;
 using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Components.Rendering;
 using Microsoft.AspNetCore.Components.Web;
@@ -29,7 +28,7 @@ internal class SSRRenderModeBoundary : IComponent
     private RenderHandle _renderHandle;
     private IReadOnlyDictionary<string, object?>? _latestParameters;
     private ComponentMarkerKey? _markerKey;
-    private HttpContext _httpContext;
+    private readonly HttpContext _httpContext;
 
     public IComponentRenderMode RenderMode { get; }
 
@@ -109,10 +108,10 @@ public Task SetParametersAsync(ParameterView parameters)
 
         ValidateParameters(_latestParameters);
 
-        // Preload WebAssembly assets when using WebAssembly (not Auto) mode
         if (RenderMode is InteractiveWebAssemblyRenderMode)
         {
-            AppendWebAssemblyPreloadAssets();
+            // Preload WebAssembly assets when using WebAssembly (not Auto) mode
+            PreloadWebAssemblyAssets();
         }
 
         if (_prerender)
@@ -123,7 +122,7 @@ public Task SetParametersAsync(ParameterView parameters)
         return Task.CompletedTask;
     }
 
-    private void AppendWebAssemblyPreloadAssets()
+    private void PreloadWebAssemblyAssets()
     {
         var preloads = _httpContext.GetEndpoint()?.Metadata.GetMetadata<ResourcePreloadCollection>();
         if (preloads != null && preloads.TryGetAssets("webassembly", out var preloadAssets))