Skip to content

[Blazor] Use preloading in benchmark app - #61959

Merged
maraf merged 8 commits into
dotnet:mainfrom
maraf:BlazorBenchPreload
Sep 26, 2025
Merged

maraf merged 8 commits into
dotnet:mainfrom
maraf:BlazorBenchPreload

Conversation

@maraf

@maraf maraf commented May 16, 2025

Copy link
Copy Markdown
Member

Apply preloading and fingerprinting placeholders to Wasm.Performance.TestApp

@maraf maraf self-assigned this May 16, 2025
@maraf
maraf requested a review from a team as a code owner May 16, 2025 08:09
@maraf maraf added the area-blazor Includes: Blazor, Razor Components label May 16, 2025
@maraf

maraf commented May 16, 2025

Copy link
Copy Markdown
Member Author

Failure is related

/home/vsts/work/1/s/.dotnet/sdk/10.0.100-preview.4.25216.37/Sdks/Microsoft.NET.Sdk.BlazorWebAssembly/targets/Microsoft.NET.Sdk.BlazorWebAssembly.6_0.targets(168,5): error : System.InvalidOperationException: No file exists for the asset at either location '/home/vsts/work/1/s/src/Components/benchmarkapps/Wasm.Performance/TestApp/bin/Release/net10.0/wwwroot/_framework/blazor.webassembly.js.cejseapcsj.map' or '/home/vsts/work/1/s/src/Components/benchmarkapps/Wasm.Performance/TestApp/bin/Release/net10.0/wwwroot/_framework/blazor.webassembly.js.cejseapcsj.map'. [/home/vsts/work/1/s/src/Components/benchmarkapps/Wasm.Performance/TestApp/Wasm.Performance.TestApp.csproj]
/home/vsts/work/1/s/.dotnet/sdk/10.0.100-preview.4.25216.37/Sdks/Microsoft.NET.Sdk.BlazorWebAssembly/targets/Microsoft.NET.Sdk.BlazorWebAssembly.6_0.targets(168,5): error :    at Microsoft.AspNetCore.StaticWebAssets.Tasks.StaticWebAsset.ResolveFile(String identity, String originalItemSpec) [/home/vsts/work/1/s/src/Components/benchmarkapps/Wasm.Performance/TestApp/Wasm.Performance.TestApp.csproj]
/home/vsts/work/1/s/.dotnet/sdk/10.0.100-preview.4.25216.37/Sdks/Microsoft.NET.Sdk.BlazorWebAssembly/targets/Microsoft.NET.Sdk.BlazorWebAssembly.6_0.targets(168,5): error :    at Microsoft.AspNetCore.StaticWebAssets.Tasks.DefineStaticWebAssets.Execute() [/home/vsts/work/1/s/src/Components/benchmarkapps/Wasm.Performance/TestApp/Wasm.Performance.TestApp.csproj]

@dotnet-policy-service dotnet-policy-service Bot added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label May 23, 2025
@maraf

maraf commented May 28, 2025

Copy link
Copy Markdown
Member Author

It needs SDK change from dotnet/sdk#49178

@maraf maraf removed the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Jun 23, 2025
@dotnet-policy-service dotnet-policy-service Bot added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Jun 30, 2025
@maraf
maraf marked this pull request as ready for review September 26, 2025 08:49
Copilot AI review requested due to automatic review settings September 26, 2025 08:49
@maraf
maraf merged commit 4753235 into dotnet:main Sep 26, 2025
30 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adds preloading and fingerprinting placeholders to the Blazor WebAssembly benchmark application to improve performance measurement capabilities.

  • Adds preloading support with WebAssembly preload link and import map script
  • Enables fingerprinting placeholders in the Blazor script reference
  • Configures the project to override HTML asset placeholders

Reviewed Changes

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

File Description
blazor-frame.html Adds preload link for WebAssembly, import map script, and fingerprinting placeholder in Blazor script src
Wasm.Performance.TestApp.csproj Enables HTML asset placeholder overriding via MSBuild property

@maraf
maraf deleted the BlazorBenchPreload branch September 26, 2025 08:49
@dotnet-policy-service dotnet-policy-service Bot added this to the 11.0-preview1 milestone Sep 26, 2025
maraf added a commit that referenced this pull request 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