Review updates.

src/EFCore/Infrastructure/AccessorExtensions.cs

@@ -59,7 +59,7 @@ public static TService GetService<TService>(this IInfrastructure<IServiceProvide
     /// <returns>The requested service.</returns>
     [DebuggerStepThrough]
     public static object GetService(this IInfrastructure<IServiceProvider> accessor, Type serviceType)
-        => InfrastructureExtensions.GetService(serviceType, accessor);
+        => InfrastructureExtensions.GetService(accessor, serviceType);
 
     /// <summary>
     ///     <para>

src/EFCore/Infrastructure/Internal/InfrastructureExtensions.cs

@@ -21,15 +21,15 @@ public static class InfrastructureExtensions
     /// </summary>
     public static TService GetService<TService>(IInfrastructure<IServiceProvider> accessor)
         where TService : class
-        => (TService)GetService(typeof(TService), accessor);
+        => (TService)GetService(accessor, typeof(TService));
 
     /// <summary>
     ///     This is an internal API that supports the Entity Framework Core infrastructure and not subject to
     ///     the same compatibility standards as public APIs. It may be changed or removed without notice in
     ///     any release. You should only use it directly in your code with extreme caution and knowing that
     ///     doing so can result in application failures when updating to a new Entity Framework Core release.
     /// </summary>
-    public static object GetService(Type serviceType, IInfrastructure<IServiceProvider> accessor)
+    public static object GetService(IInfrastructure<IServiceProvider> accessor, Type serviceType)
     {
         var internalServiceProvider = accessor.Instance;
 

src/EFCore/Metadata/IReadOnlyNavigationBase.cs

@@ -52,7 +52,6 @@ bool LazyLoadingEnabled
         => (bool?)this[CoreAnnotationNames.LazyLoadingEnabled] ?? true;
 
     /// <inheritdoc />
-    // TODO: Remove when #3864 is implemented
     bool IReadOnlyPropertyBase.IsShadowProperty()
         => this.GetIdentifyingMemberInfo() == null;
 }