Skip to content

Conversation

@maraf
Copy link
Member

@maraf maraf commented Nov 28, 2025

Reverts #61959

Related to #64450

@maraf maraf self-assigned this Nov 28, 2025
@maraf maraf requested a review from a team as a code owner November 28, 2025 13:46
Copilot AI review requested due to automatic review settings November 28, 2025 13:46
@maraf maraf added the area-blazor Includes: Blazor, Razor Components label Nov 28, 2025
Copilot finished reviewing on behalf of maraf November 28, 2025 13:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR reverts changes that introduced preloading capabilities in the Blazor WebAssembly benchmark application. The revert removes experimental features that used asset fingerprinting and preloading, returning the benchmark app to its simpler, standard configuration.

  • Removed preload and importmap elements from the HTML template
  • Reverted to standard blazor.webassembly.js script reference without fingerprint placeholders
  • Removed the OverrideHtmlAssetPlaceholders MSBuild property from the project file

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/Components/benchmarkapps/Wasm.Performance/TestApp/wwwroot/blazor-frame.html Removes preload link and importmap script tags; reverts to standard blazor.webassembly.js reference
src/Components/benchmarkapps/Wasm.Performance/TestApp/Wasm.Performance.TestApp.csproj Removes OverrideHtmlAssetPlaceholders property

@maraf maraf merged commit 4d19e42 into main Nov 28, 2025
37 checks passed
@maraf maraf deleted the revert-61959-BlazorBenchPreload branch November 28, 2025 16:07
@dotnet-policy-service dotnet-policy-service bot added this to the 11.0-preview1 milestone Nov 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-blazor Includes: Blazor, Razor Components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants