Remove additional obsolete APIs from Components.WebAssembly

Co-authored-by: javiercn <[email protected]>

Author: Copilot

src/Components/WebAssembly/JSInterop/src/InternalCalls.cs

@@ -1,22 +1,12 @@
 // 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;
-using System.Runtime.CompilerServices;
 using System.Runtime.InteropServices.JavaScript;
 
 namespace WebAssembly.JSInterop;
 
 internal static partial class InternalCalls
 {
-    // This method only exists for backwards compatibility and will be removed in the future.
-    // The exact namespace, type, and method name must match the corresponding entries
-    // in driver.c in the Mono distribution.
-    // See: https://github.com/mono/mono/blob/90574987940959fe386008a850982ea18236a533/sdks/wasm/src/driver.c#L318-L319
-    [MethodImpl(MethodImplOptions.InternalCall)]
-    [Obsolete]
-    public static extern TRes InvokeJS<T0, T1, T2, TRes>(out string exception, ref JSCallInfo callInfo, [AllowNull] T0 arg0, [AllowNull] T1 arg1, [AllowNull] T2 arg2);
-
     [JSImport("Blazor._internal.invokeJSJson", "blazor-internal")]
     public static partial string InvokeJSJson(
         string identifier,

src/Components/WebAssembly/WebAssembly.Authentication/src/PublicAPI.Unshipped.txt

@@ -1 +1,8 @@
 #nullable enable
+*REMOVED*Microsoft.AspNetCore.Components.WebAssembly.Authentication.AccessTokenResult.AccessTokenResult(Microsoft.AspNetCore.Components.WebAssembly.Authentication.AccessTokenResultStatus status, Microsoft.AspNetCore.Components.WebAssembly.Authentication.AccessToken! token, string! redirectUrl) -> void
+*REMOVED*Microsoft.AspNetCore.Components.WebAssembly.Authentication.AccessTokenResult.RedirectUrl.get -> string?
+*REMOVED*Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationService<TRemoteAuthenticationState, TAccount, TProviderOptions>.RemoteAuthenticationService(Microsoft.JSInterop.IJSRuntime! jsRuntime, Microsoft.Extensions.Options.IOptionsSnapshot<Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationOptions<TProviderOptions>!>! options, Microsoft.AspNetCore.Components.NavigationManager! navigation, Microsoft.AspNetCore.Components.WebAssembly.Authentication.AccountClaimsPrincipalFactory<TAccount!>! accountClaimsPrincipalFactory) -> void
+*REMOVED*Microsoft.AspNetCore.Components.WebAssembly.Authentication.SignOutSessionStateManager
+*REMOVED*Microsoft.AspNetCore.Components.WebAssembly.Authentication.SignOutSessionStateManager.SignOutSessionStateManager(Microsoft.JSInterop.IJSRuntime! jsRuntime) -> void
+*REMOVED*virtual Microsoft.AspNetCore.Components.WebAssembly.Authentication.SignOutSessionStateManager.SetSignOutState() -> System.Threading.Tasks.ValueTask
+*REMOVED*virtual Microsoft.AspNetCore.Components.WebAssembly.Authentication.SignOutSessionStateManager.ValidateSignOutState() -> System.Threading.Tasks.Task<bool>!

src/Components/WebAssembly/WebAssembly.Authentication/src/RemoteAuthenticatorViewCore.cs

@@ -106,10 +106,6 @@ public RemoteAuthenticationApplicationPathsOptions ApplicationPaths
 
     [Inject] internal AuthenticationStateProvider AuthenticationProvider { get; set; } = default!;
 
-#pragma warning disable CS0618 // Type or member is obsolete, we keep it for now for backwards compatibility
-    [Inject] internal SignOutSessionStateManager SignOutManager { get; set; } = default!;
-#pragma warning restore CS0618 // Type or member is obsolete, we keep it for now for backwards compatibility
-
     [Inject] internal ILogger<RemoteAuthenticatorViewCore<TAuthenticationState>> Logger { get; set; } = default!;
 
     /// <inheritdoc />
@@ -282,9 +278,7 @@ private async Task ProcessLogInCallback()
 
     private async Task ProcessLogOut(string returnUrl)
     {
-        if ((Navigation.HistoryEntryState != null && !ValidateSignOutRequestState()) ||
-            // For backcompat purposes, keep SignOutManager working, even though we now use the history.state for this.
-            (Navigation.HistoryEntryState == null && !await SignOutManager.ValidateSignOutState()))
+        if (Navigation.HistoryEntryState != null && !ValidateSignOutRequestState())
         {
             Log.LogoutOperationInitiatedExternally(Logger);
             Navigation.NavigateTo(ApplicationPaths.LogOutFailedPath, AuthenticationNavigationOptions with { HistoryEntryState = "The logout was not initiated from within the page." });

src/Components/WebAssembly/WebAssembly.Authentication/src/Services/AccessTokenNotAvailableException.cs

@@ -41,9 +41,7 @@ public void Redirect()
         }
         else
         {
-#pragma warning disable CS0618 // Type or member is obsolete
-            _navigation.NavigateTo(_tokenResult.RedirectUrl!);
-#pragma warning restore CS0618 // Type or member is obsolete
+            _navigation.NavigateTo(_tokenResult.InteractiveRequestUrl!);
         }
     }
 

src/Components/WebAssembly/WebAssembly.Authentication/src/Services/AccessTokenResult.cs

@@ -12,20 +12,6 @@ public class AccessTokenResult
 {
     private readonly AccessToken _token;
 
-    /// <summary>
-    /// Initializes a new instance of <see cref="AccessTokenResult"/>.
-    /// </summary>
-    /// <param name="status">The status of the result.</param>
-    /// <param name="token">The <see cref="AccessToken"/> in case it was successful.</param>
-    /// <param name="redirectUrl">The redirect uri to go to for provisioning the token.</param>
-    [Obsolete("Use the AccessTokenResult(AccessTokenResultStatus, AccessToken, string, InteractiveRequestOptions)")]
-    public AccessTokenResult(AccessTokenResultStatus status, AccessToken token, [StringSyntax(StringSyntaxAttribute.Uri)] string redirectUrl)
-    {
-        Status = status;
-        _token = token;
-        RedirectUrl = redirectUrl;
-    }
-
     /// <summary>
     /// Initializes a new instance of <see cref="AccessTokenResult"/>.
     /// </summary>
@@ -46,12 +32,6 @@ public AccessTokenResult(AccessTokenResultStatus status, AccessToken token, [Str
     /// </summary>
     public AccessTokenResultStatus Status { get; }
 
-    /// <summary>
-    /// Gets the URL to redirect to if <see cref="Status"/> is <see cref="AccessTokenResultStatus.RequiresRedirect"/>.
-    /// </summary>
-    [Obsolete("Use 'InteractiveRequestUrl' and 'InteractiveRequest' instead.")]
-    public string? RedirectUrl { get; }
-
     /// <summary>
     /// Gets the URL to call <see cref="NavigationManagerExtensions.NavigateToLogin(NavigationManager, string, InteractiveRequestOptions)"/> if <see cref="Status"/> is
     /// <see cref="AccessTokenResultStatus.RequiresRedirect"/>.

src/Components/WebAssembly/WebAssembly.Authentication/src/Services/RemoteAuthenticationService.cs

@@ -57,23 +57,6 @@ public class RemoteAuthenticationService<
     /// </summary>
     protected RemoteAuthenticationOptions<TProviderOptions> Options { get; }
 
-    /// <summary>
-    /// Initializes a new instance.
-    /// </summary>
-    /// <param name="jsRuntime">The <see cref="IJSRuntime"/> to use for performing JavaScript interop operations.</param>
-    /// <param name="options">The options to be passed down to the underlying JavaScript library handling the authentication operations.</param>
-    /// <param name="navigation">The <see cref="NavigationManager"/> used to generate URLs.</param>
-    /// <param name="accountClaimsPrincipalFactory">The <see cref="AccountClaimsPrincipalFactory{TAccount}"/> used to generate the <see cref="ClaimsPrincipal"/> for the user.</param>
-    [Obsolete("Use the constructor RemoteAuthenticationService(IJSRuntime,IOptionsSnapshot<RemoteAuthenticationOptions<TProviderOptions>>,NavigationManager,AccountClaimsPrincipalFactory<TAccount>,ILogger<RemoteAuthenticationService<TRemoteAuthenticationState, TAccount, TProviderOptions>>) instead.")]
-    public RemoteAuthenticationService(
-        IJSRuntime jsRuntime,
-        IOptionsSnapshot<RemoteAuthenticationOptions<TProviderOptions>> options,
-        NavigationManager navigation,
-        AccountClaimsPrincipalFactory<TAccount> accountClaimsPrincipalFactory)
-        : this(jsRuntime, options, navigation, accountClaimsPrincipalFactory, null)
-    {
-    }
-
     /// <summary>
     /// Initializes a new instance.
     /// </summary>

src/Components/WebAssembly/WebAssembly.Authentication/src/Services/SignOutSessionStateManager.cs

@@ -71,9 +71,6 @@ public static IRemoteAuthenticationBuilder<TRemoteAuthenticationState, TAccount>
 
         services.TryAddScoped<IRemoteAuthenticationPathsProvider, DefaultRemoteApplicationPathsProvider<TProviderOptions>>();
         services.TryAddScoped<IAccessTokenProviderAccessor, AccessTokenProviderAccessor>();
-#pragma warning disable CS0618 // Type or member is obsolete, we keep it for now for backwards compatibility
-        services.TryAddScoped<SignOutSessionStateManager>();
-#pragma warning restore CS0618 // Type or member is obsolete, we keep it for now for backwards compatibility
 
         services.TryAddScoped<AccountClaimsPrincipalFactory<TAccount>>();
 

src/Components/WebAssembly/WebAssembly.Authentication/src/WebAssemblyAuthenticationServiceCollectionExtensions.cs

@@ -130,17 +130,6 @@ private static async ValueTask ApplyPreviousDeltasAsync(HotReloadAgent agent)
     private static HotReloadAgent? GetAgent()
         => s_hotReloadAgent ?? (s_initialized ? throw new InvalidOperationException("Hot Reload agent not initialized") : null);
 
-    /// <summary>
-    /// For framework use only.
-    /// </summary>
-    [Obsolete("Use ApplyHotReloadDeltas instead")]
-    [JSInvokable(nameof(ApplyHotReloadDelta))]
-    public static void ApplyHotReloadDelta(string moduleIdString, byte[] metadataDelta, byte[] ilDelta, byte[] pdbBytes, int[]? updatedTypes)
-    {
-        GetAgent()?.ApplyDeltas(
-            [new UpdateDelta(Guid.Parse(moduleIdString, CultureInfo.InvariantCulture), metadataDelta, ilDelta, pdbBytes, updatedTypes ?? [])]);
-    }
-
     /// <summary>
     /// For framework use only.
     /// </summary>

src/Components/WebAssembly/WebAssembly/src/HotReload/WebAssemblyHotReload.cs

@@ -14,13 +14,6 @@ namespace Microsoft.AspNetCore.Components.WebAssembly.Infrastructure;
 [EditorBrowsable(EditorBrowsableState.Never)]
 public static class JSInteropMethods
 {
-    /// <summary>
-    /// For framework use only.
-    /// </summary>
-    [Obsolete("This API is for framework use only and is no longer used in the current version")]
-    public static void NotifyLocationChanged(string uri, bool isInterceptedLink)
-        => WebAssemblyNavigationManager.Instance.SetLocation(uri, null, isInterceptedLink);
-
     /// <summary>
     /// For framework use only.
     /// </summary>