Address PR feedback

Author: halter73

src/Http/Http.Abstractions/src/Metadata/IResult.cs renamed to src/Http/Http.Abstractions/src/IResult.cs

@@ -3,7 +3,7 @@
 
 using System.Threading.Tasks;
 
-namespace Microsoft.AspNetCore.Http.Metadata
+namespace Microsoft.AspNetCore.Http
 {
     /// <summary>
     /// Defines a contract that represents the result of an HTTP endpoint.

src/Http/Http.Abstractions/src/PublicAPI.Unshipped.txt

@@ -4,6 +4,8 @@
 *REMOVED*Microsoft.AspNetCore.Routing.RouteValueDictionary.TryAdd(string! key, object! value) -> bool
 *REMOVED*static Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.UseMiddleware(this Microsoft.AspNetCore.Builder.IApplicationBuilder! app, System.Type! middleware, params object![]! args) -> Microsoft.AspNetCore.Builder.IApplicationBuilder!
 *REMOVED*static Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.UseMiddleware<TMiddleware>(this Microsoft.AspNetCore.Builder.IApplicationBuilder! app, params object![]! args) -> Microsoft.AspNetCore.Builder.IApplicationBuilder!
+Microsoft.AspNetCore.Http.IResult
+Microsoft.AspNetCore.Http.IResult.ExecuteAsync(Microsoft.AspNetCore.Http.HttpContext! httpContext) -> System.Threading.Tasks.Task!
 Microsoft.AspNetCore.Http.Metadata.IFromBodyMetadata
 Microsoft.AspNetCore.Http.Metadata.IFromBodyMetadata.AllowEmpty.get -> bool
 Microsoft.AspNetCore.Http.Metadata.IFromFormMetadata
@@ -15,8 +17,6 @@ Microsoft.AspNetCore.Http.Metadata.IFromQueryMetadata.Name.get -> string?
 Microsoft.AspNetCore.Http.Metadata.IFromRouteMetadata
 Microsoft.AspNetCore.Http.Metadata.IFromRouteMetadata.Name.get -> string?
 Microsoft.AspNetCore.Http.Metadata.IFromServiceMetadata
-Microsoft.AspNetCore.Http.Metadata.IResult
-Microsoft.AspNetCore.Http.Metadata.IResult.ExecuteAsync(Microsoft.AspNetCore.Http.HttpContext! httpContext) -> System.Threading.Tasks.Task!
 Microsoft.AspNetCore.Http.Endpoint.Endpoint(Microsoft.AspNetCore.Http.RequestDelegate? requestDelegate, Microsoft.AspNetCore.Http.EndpointMetadataCollection? metadata, string? displayName) -> void
 Microsoft.AspNetCore.Http.Endpoint.RequestDelegate.get -> Microsoft.AspNetCore.Http.RequestDelegate?
 Microsoft.AspNetCore.Routing.RouteValueDictionary.TryAdd(string! key, object? value) -> bool

src/Http/Routing/src/Internal/MapActionExpressionTreeBuilder.cs

@@ -11,7 +11,6 @@
 using System.Reflection;
 using System.Threading;
 using System.Threading.Tasks;
-using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Http.Metadata;
 using Microsoft.Extensions.DependencyInjection;
@@ -320,7 +319,7 @@ public static RequestDelegate BuildRequestDelegate(Delegate action)
         private static void LogRequestBodyIOException(HttpContext httpContext, IOException exception)
         {
             var loggerFactory = httpContext.RequestServices.GetRequiredService<ILoggerFactory>();
-            var logger = loggerFactory.CreateLogger(typeof(MapActionEndpointRouteBuilderExtensions).FullName!);
+            var logger = loggerFactory.CreateLogger("Microsoft.AspNetCore.Routing.MapAction");
             Log.RequestBodyIOException(logger, exception);
         }
 

src/Http/Routing/test/UnitTests/Internal/MapActionExpressionTreeBuilderTest.cs

@@ -11,7 +11,6 @@
 using System.Text.Json;
 using System.Threading;
 using System.Threading.Tasks;
-using Microsoft.AspNetCore.Builder;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Http.Features;
 using Microsoft.AspNetCore.Http.Metadata;
@@ -257,7 +256,7 @@ public async Task RequestDelegateLogsFromBodyIOExceptionsAsDebug()
         {
             var invoked = false;
 
-            var sink = new TestSink(context => context.LoggerName == typeof(MapActionEndpointRouteBuilderExtensions).FullName);
+            var sink = new TestSink(context => context.LoggerName == "Microsoft.AspNetCore.Routing.MapAction");
             var testLoggerFactory = new TestLoggerFactory(sink, enabled: true);
 
             void TestAction([FromBody] Todo todo)
@@ -321,7 +320,7 @@ public async Task RequestDelegateLogsFromFormIOExceptionsAsDebug()
         {
             var invoked = false;
 
-            var sink = new TestSink(context => context.LoggerName == typeof(MapActionEndpointRouteBuilderExtensions).FullName);
+            var sink = new TestSink(context => context.LoggerName == "Microsoft.AspNetCore.Routing.MapAction");
             var testLoggerFactory = new TestLoggerFactory(sink, enabled: true);
 
             void TestAction([FromForm] int value)

src/Mvc/Mvc.Core/src/JsonResult.cs

@@ -5,7 +5,6 @@
 using System.Text.Json;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Http;
-using Microsoft.AspNetCore.Http.Metadata;
 using Microsoft.AspNetCore.Mvc.Infrastructure;
 using Microsoft.Extensions.DependencyInjection;
 

src/Mvc/Mvc.Core/src/StatusCodeResult.cs

@@ -4,7 +4,6 @@
 using System;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Http;
-using Microsoft.AspNetCore.Http.Metadata;
 using Microsoft.AspNetCore.Mvc.Infrastructure;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Logging;