Feedback.

Author: ilonatommy

src/Components/Components/src/NavigationManager.cs

@@ -61,7 +61,6 @@ public event EventHandler<NotFoundEventArgs> OnNotFound
     // The URI. Always represented an absolute URI.
     private string? _uri;
     private bool _isInitialized;
-    private readonly NotFoundEventArgs _notFoundEventArgs = new();
 
     /// <summary>
     /// Gets or sets the current base URI. The <see cref="BaseUri" /> is always represented as an absolute URI in string form with trailing slash.
@@ -211,7 +210,7 @@ private void NotFoundCore()
         }
         else
         {
-            _notFound.Invoke(this, _notFoundEventArgs);
+            _notFound.Invoke(this, new NotFoundEventArgs());
         }
     }
 

src/Components/Components/src/Routing/NotFoundEventArgs.cs

@@ -9,7 +9,7 @@ namespace Microsoft.AspNetCore.Components.Routing;
 public sealed class NotFoundEventArgs : EventArgs
 {
     /// <summary>
-    /// Gets the path of NotFoundPage. If the path is set, it indicates that the router has handled the rendering of the NotFound contents.
+    /// Gets the path of NotFoundPage. If the path is set, it indicates that a subscriber has handled the rendering of the NotFound contents.
     /// </summary>
     public string? Path { get; set; }
 }

src/Components/test/testassets/Components.TestServer/RazorComponents/CustomRouter.cs

@@ -46,7 +46,7 @@ [Inject] private INavigationInterception NavigationInterception
 
     [Inject] private ILoggerFactory LoggerFactory
     {
-        get => throw new InvalidOperationException("This property should only be set via injection.");
+        get => throw new InvalidOperationException($"{nameof(LoggerFactory)} should only be set via injection.");
         set => _logger = value.CreateLogger<CustomRouter>();
     }