[browser] Fix debugger support

eng/testing/scenarios/BuildWasmAppsJobsList.txt

@@ -44,3 +44,4 @@ Wasm.Build.Tests.WasmTemplateTests
 Wasm.Build.Tests.WorkloadTests
 Wasm.Build.Tests.TestAppScenarios.DownloadResourceProgressTests
 Wasm.Build.Tests.MT.Blazor.SimpleMultiThreadedTests
+Wasm.Build.Tests.TestAppScenarios.DebugLevelTests

src/mono/browser/runtime/cwraps.ts

@@ -166,7 +166,7 @@ export interface t_Cwraps {
  mono_wasm_load_icu_data(offset: VoidPtr): number;
  mono_wasm_add_assembly(name: string, data: VoidPtr, size: number): number;
  mono_wasm_add_satellite_assembly(name: string, culture: string, data: VoidPtr, size: number): void;
- mono_wasm_load_runtime(unused: string, debugLevel: number): void;
+ mono_wasm_load_runtime(debugLevel: number): void;
  mono_wasm_change_debugger_log_level(value: number): void;
 
  mono_wasm_assembly_load(name: string): MonoAssembly;

src/mono/browser/runtime/driver.c

@@ -181,7 +181,7 @@ cleanup_runtime_config (MonovmRuntimeConfigArguments *args, void *user_data)
 }
 
 EMSCRIPTEN_KEEPALIVE void
-mono_wasm_load_runtime (const char *unused, int debug_level)
+mono_wasm_load_runtime (int debug_level)
 {
 	const char *interp_opts = "";
 

src/mono/browser/runtime/lazyLoading.ts

@@ -26,7 +26,7 @@ export async function loadLazyAssembly(assemblyNameToLoad: string): Promise<bool
  }
 
  const pdbNameToLoad = changeExtension(dllAsset.name, ".pdb");
- const shouldLoadPdb = loaderHelpers.hasDebuggingEnabled(loaderHelpers.config) && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
+ const shouldLoadPdb = loaderHelpers.config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
 
  const dllBytesPromise = loaderHelpers.retrieve_asset_download(dllAsset);
 

src/mono/browser/runtime/loader/assets.ts

@@ -266,7 +266,7 @@ export function prepareAssets() {
  }
  }
 
- if (config.debugLevel != 0 && resources.pdb) {
+ if (config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && resources.pdb) {
  for (const name in resources.pdb) {
  assetsToLoad.push({
  name,

src/mono/browser/runtime/loader/config.ts

@@ -197,13 +197,6 @@ export function normalizeConfig() {
  config.environmentVariables["MONO_SLEEP_ABORT_LIMIT"] = "5000";
  }
 
- // Default values (when WasmDebugLevel is not set)
- // - Build (debug) => debugBuild=true & debugLevel=-1 => -1
- // - Build (release) => debugBuild=true & debugLevel=0 => 0
- // - Publish (debug) => debugBuild=false & debugLevel=-1 => 0
- // - Publish (release) => debugBuild=false & debugLevel=0 => 0
- config.debugLevel = hasDebuggingEnabled(config) ? config.debugLevel : 0;
-
  if (BuildConfiguration === "Debug" && config.diagnosticTracing === undefined) {
  config.diagnosticTracing = true;
  }
@@ -264,14 +257,13 @@ export async function mono_wasm_load_config(module: DotnetModuleInternal): Promi
  }
 }
 
-export function hasDebuggingEnabled(config: MonoConfigInternal): boolean {
+export function isDebuggingSupported(): boolean {
  // Copied from blazor MonoDebugger.ts/attachDebuggerHotkey
  if (!globalThis.navigator) {
  return false;
  }
 
- const hasReferencedPdbs = !!config.resources!.pdb;
- return (hasReferencedPdbs || config.debugLevel != 0) && (loaderHelpers.isChromium || loaderHelpers.isFirefox);
+ return loaderHelpers.isChromium || loaderHelpers.isFirefox;
 }
 
 async function loadBootConfig(module: DotnetModuleInternal): Promise<void> {

src/mono/browser/runtime/loader/globals.ts

@@ -15,7 +15,7 @@ import { assertIsControllablePromise, createPromiseController, getPromiseControl
 import { mono_download_assets, resolve_single_asset_path, retrieve_asset_download } from "./assets";
 import { mono_log_error, set_thread_prefix, setup_proxy_console } from "./logging";
 import { invokeLibraryInitializers } from "./libraryInitializers";
-import { deep_merge_config, hasDebuggingEnabled } from "./config";
+import { deep_merge_config, isDebuggingSupported } from "./config";
 import { logDownloadStatsToConsole, purgeUnusedCacheEntriesAsync } from "./assetsCache";
 
 // if we are the first script loaded in the web worker, we are expected to become the sidecar
@@ -118,9 +118,9 @@ export function setLoaderGlobals(
  purgeUnusedCacheEntriesAsync,
  installUnhandledErrorHandler,
 
- hasDebuggingEnabled,
  retrieve_asset_download,
  invokeLibraryInitializers,
+ isDebuggingSupported,
 
  // from wasm-feature-detect npm package
  exceptions,

src/mono/browser/runtime/startup.ts

@@ -515,7 +515,7 @@ async function start_runtime() {
  if (runtimeHelpers.config.browserProfilerOptions)
  mono_wasm_init_browser_profiler(runtimeHelpers.config.browserProfilerOptions);
 
- mono_wasm_load_runtime("unused", runtimeHelpers.config.debugLevel);
+ mono_wasm_load_runtime();
 
  if (runtimeHelpers.config.interpreterPgo)
  setTimeout(maybeSaveInterpPgoTable, (runtimeHelpers.config.interpreterPgoSaveDelay || 15) * 1000);
@@ -534,17 +534,21 @@ async function maybeSaveInterpPgoTable() {
  await interp_pgo_save_data();
 }
 
-export function mono_wasm_load_runtime(unused?: string, debugLevel?: number): void {
+export function mono_wasm_load_runtime(): void {
  mono_log_debug("mono_wasm_load_runtime");
  try {
  const mark = startMeasure();
+ let debugLevel = runtimeHelpers.config.debugLevel;
  if (debugLevel == undefined) {
  debugLevel = 0;
  if (runtimeHelpers.config.debugLevel) {
  debugLevel = 0 + debugLevel;
  }
  }
- cwraps.mono_wasm_load_runtime(unused || "unused", debugLevel);
+ if (!loaderHelpers.isDebuggingSupported() || !runtimeHelpers.config.resources!.pdb) {
+ debugLevel = 0;
+ }
+ cwraps.mono_wasm_load_runtime(debugLevel);
  endMeasure(mark, MeasuredBlock.loadRuntime);
 
  } catch (err: any) {

src/mono/browser/runtime/types/internal.ts

@@ -155,7 +155,6 @@ export type LoaderHelpers = {
  out(message: string): void;
  err(message: string): void;
 
- hasDebuggingEnabled(config: MonoConfig): boolean,
  retrieve_asset_download(asset: AssetEntry): Promise<ArrayBuffer>;
  onDownloadResourceProgress?: (resourcesLoaded: number, totalResources: number) => void;
  logDownloadStatsToConsole: () => void;
@@ -166,6 +165,7 @@ export type LoaderHelpers = {
  invokeLibraryInitializers: (functionName: string, args: any[]) => Promise<void>,
  libraryInitializers?: { scriptName: string, exports: any }[];
 
+ isDebuggingSupported(): boolean,
  isChromium: boolean,
  isFirefox: boolean
 

src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

@@ -203,9 +203,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  <_BlazorWebAssemblyStartupMemoryCache>$(BlazorWebAssemblyStartupMemoryCache)</_BlazorWebAssemblyStartupMemoryCache>
  <_BlazorWebAssemblyJiterpreter>$(BlazorWebAssemblyJiterpreter)</_BlazorWebAssemblyJiterpreter>
  <_BlazorWebAssemblyRuntimeOptions>$(BlazorWebAssemblyRuntimeOptions)</_BlazorWebAssemblyRuntimeOptions>
- <_WasmDebugLevel>$(WasmDebugLevel)</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == ''">0</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == '0' and ('$(DebuggerSupport)' == 'true' or '$(Configuration)' == 'Debug')">-1</_WasmDebugLevel>
 
  <!-- Workaround for https://github.com/dotnet/sdk/issues/12114-->
  <PublishDir Condition="'$(AppendRuntimeIdentifierToOutputPath)' != 'true' AND '$(PublishDir)' == '$(OutputPath)$(RuntimeIdentifier)\$(PublishDirName)\'">$(OutputPath)$(PublishDirName)\</PublishDir>
@@ -364,7 +361,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmOutputWithHash)"
  DebugBuild="true"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="false"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(_WasmBuildBootJsonPath)"
@@ -380,7 +377,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="false" />
 
  <ItemGroup>
  <FileWrites Include="$(_WasmBuildBootJsonPath)" />
@@ -533,7 +531,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  </Target>
 
  <Target Name="GeneratePublishWasmBootJson" DependsOnTargets="$(GeneratePublishWasmBootJsonDependsOn)">
-
  <ItemGroup>
  <_WasmPublishAsset
  Include="@(StaticWebAsset)"
@@ -549,7 +546,6 @@ Copyright (c) .NET Foundation. All rights reserved.
 
  <!-- We remove the extensions since they are added to the list of static web assets but we need to compute the target path for them -->
  <_WasmPublishAsset Remove="@(_BlazorExtensionsCandidatesForPublish)" />
-
  </ItemGroup>
 
  <ComputeStaticWebAssetsTargetPaths
@@ -568,7 +564,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmPublishBootResourceWithHash)"
  DebugBuild="false"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="$(PublishTrimmed)"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(IntermediateOutputPath)blazor.publish.boot.json"
@@ -584,7 +580,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="true" />
 
  <ItemGroup>
  <FileWrites Include="$(IntermediateOutputPath)blazor.publish.boot.json" />

src/mono/wasi/build/WasiApp.targets

@@ -66,6 +66,11 @@
  Outputs="$(WasmAppDir)\.stamp"
  Condition="'$(WasmGenerateAppBundle)' == 'true'">
 
+ <PropertyGroup>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(CopyOutputSymbolsToPublishDirectory)' == 'true' and '$(_IsPublishing)' == 'true'">true</_WasmOutputSymbolsToAppBundle>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(_WasmOutputSymbolsToAppBundle)' == ''">false</_WasmOutputSymbolsToAppBundle>
+ </PropertyGroup>
+
  <WasiAppBuilder
  AppDir="$(WasmAppDir)"
  Assemblies="@(_WasmAssembliesInternal)"
@@ -80,7 +85,7 @@
  FilesToIncludeInFileSystem="@(WasmFilesToIncludeInFileSystem)"
  ExtraFilesToDeploy="@(WasmExtraFilesToDeploy)"
  NativeAssets="@(WasmNativeAsset)"
-DebugLevel="$(WasmDebugLevel)"
+OutputSymbolsToAppBundle="$(_WasmOutputSymbolsToAppBundle)"
  RuntimeConfigJsonPath="$(_WasmRuntimeConfigFilePath)"
  />
  </Target>

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorWasmTestBase.cs

@@ -63,7 +63,9 @@ public string CreateBlazorWasmTemplateProject(string id)
  (CommandResult res, string logPath) = BlazorBuildInternal(options.Id, options.Config, publish: false, setWasmDevel: false, expectSuccess: options.ExpectSuccess, extraArgs);
 
  if (options.ExpectSuccess && options.AssertAppBundle)
+ {
  AssertBundle(res.Output, options with { IsPublish = false });
+ }
 
  return (res, logPath);
  }
@@ -77,6 +79,10 @@ public string CreateBlazorWasmTemplateProject(string id)
 
  if (options.ExpectSuccess && options.AssertAppBundle)
  {
+ // Because we do relink in Release publish by default
+ if (options.Config == "Release")
+ options = options with { ExpectedFileType = NativeFilesType.Relinked };
+
  AssertBundle(res.Output, options with { IsPublish = true });
  }
 

src/mono/wasm/Wasm.Build.Tests/Templates/InterpPgoTests.cs

@@ -34,7 +34,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
 
  string id = $"browser_{config}_{GetRandomId()}";
  _testOutput.WriteLine("/// Creating project");
- string projectFile = CreateWasmTemplateProject(id, "wasmbrowser");
+ string projectFile = CreateWasmTemplateProject(id, "wasmbrowser", extraProperties: "<WasmDebugLevel>0</WasmDebugLevel>");
 
  _testOutput.WriteLine("/// Updating JS");
  UpdateBrowserMainJs((js) => {
@@ -53,6 +53,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
  // then call INTERNAL.interp_pgo_save_data() to save the interp PGO table
  js = js.Replace(
  "const text = exports.MyClass.Greeting();",
+ "console.log(`WASM debug level ${getConfig().debugLevel}`);\n" + 
  "let text = '';\n" +
  $"for (let i = 0; i < {iterationCount}; i++) {{ text = exports.MyClass.Greeting(); }};\n" +
  "await INTERNAL.interp_pgo_save_data();"

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppSettingsTests.cs

@@ -28,7 +28,7 @@ public async Task LoadAppSettingsBasedOnApplicationEnvironment(string applicatio
  CopyTestAsset("WasmBasicTestApp", "AppSettingsTests");
  PublishProject("Debug");
 
- var result = await RunSdkStyleApp(new(
+ var result = await RunSdkStyleAppForPublish(new(
  Configuration: "Debug",
  TestScenario: "AppSettingsTest",
  BrowserQueryString: new Dictionary<string, string> { ["applicationEnvironment"] = applicationEnvironment }

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppTestBase.cs

@@ -36,23 +36,29 @@ protected void CopyTestAsset(string assetName, string generatedProjectNamePrefix
  _projectDir = Path.Combine(_projectDir!, "App");
  }
 
- protected void BuildProject(string configuration)
+ protected void BuildProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
- protected void PublishProject(string configuration)
+ protected void PublishProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
  protected ToolCommand CreateDotNetCommand() => new DotNetCommand(s_buildEnv, _testOutput)
  .WithWorkingDirectory(_projectDir!)
  .WithEnvironmentVariable("NUGET_PACKAGES", _nugetPackagesDir);
 
- protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
+ protected Task<RunResult> RunSdkStyleAppForBuild(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.DotnetRun);
+
+ protected Task<RunResult> RunSdkStyleAppForPublish(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.WebServer);
+
+ private async Task<RunResult> RunSdkStyleApp(RunOptions options, BlazorRunHost host = BlazorRunHost.DotnetRun)
  {
  string queryString = "?test=" + options.TestScenario;
  if (options.BrowserQueryString != null)
@@ -67,9 +73,10 @@ protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
  CheckCounter: false,
  Config: options.Configuration,
  OnConsoleMessage: OnConsoleMessage,
- QueryString: queryString);
+ QueryString: queryString,
+ Host: host);
 
- await BlazorRunForBuildWithDotnetRun(blazorRunOptions);
+ await BlazorRunTest(blazorRunOptions);
 
  void OnConsoleMessage(IConsoleMessage msg)
  {

eng/testing/scenarios/BuildWasmAppsJobsList.txt

@@ -44,3 +44,4 @@ Wasm.Build.Tests.WasmTemplateTests
 Wasm.Build.Tests.WorkloadTests
 Wasm.Build.Tests.TestAppScenarios.DownloadResourceProgressTests
 Wasm.Build.Tests.MT.Blazor.SimpleMultiThreadedTests
+Wasm.Build.Tests.TestAppScenarios.DebugLevelTests

src/mono/browser/runtime/cwraps.ts

@@ -166,7 +166,7 @@ export interface t_Cwraps {
  mono_wasm_load_icu_data(offset: VoidPtr): number;
  mono_wasm_add_assembly(name: string, data: VoidPtr, size: number): number;
  mono_wasm_add_satellite_assembly(name: string, culture: string, data: VoidPtr, size: number): void;
- mono_wasm_load_runtime(unused: string, debugLevel: number): void;
+ mono_wasm_load_runtime(debugLevel: number): void;
  mono_wasm_change_debugger_log_level(value: number): void;
 
  mono_wasm_assembly_load(name: string): MonoAssembly;

src/mono/browser/runtime/driver.c

@@ -181,7 +181,7 @@ cleanup_runtime_config (MonovmRuntimeConfigArguments *args, void *user_data)
 }
 
 EMSCRIPTEN_KEEPALIVE void
-mono_wasm_load_runtime (const char *unused, int debug_level)
+mono_wasm_load_runtime (int debug_level)
 {
 	const char *interp_opts = "";
 

src/mono/browser/runtime/lazyLoading.ts

@@ -26,7 +26,7 @@ export async function loadLazyAssembly(assemblyNameToLoad: string): Promise<bool
  }
 
  const pdbNameToLoad = changeExtension(dllAsset.name, ".pdb");
- const shouldLoadPdb = loaderHelpers.hasDebuggingEnabled(loaderHelpers.config) && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
+ const shouldLoadPdb = loaderHelpers.config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
 
  const dllBytesPromise = loaderHelpers.retrieve_asset_download(dllAsset);
 

src/mono/browser/runtime/loader/assets.ts

@@ -266,7 +266,7 @@ export function prepareAssets() {
  }
  }
 
- if (config.debugLevel != 0 && resources.pdb) {
+ if (config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && resources.pdb) {
  for (const name in resources.pdb) {
  assetsToLoad.push({
  name,

src/mono/browser/runtime/loader/config.ts

@@ -197,13 +197,6 @@ export function normalizeConfig() {
  config.environmentVariables["MONO_SLEEP_ABORT_LIMIT"] = "5000";
  }
 
- // Default values (when WasmDebugLevel is not set)
- // - Build (debug) => debugBuild=true & debugLevel=-1 => -1
- // - Build (release) => debugBuild=true & debugLevel=0 => 0
- // - Publish (debug) => debugBuild=false & debugLevel=-1 => 0
- // - Publish (release) => debugBuild=false & debugLevel=0 => 0
- config.debugLevel = hasDebuggingEnabled(config) ? config.debugLevel : 0;
-
  if (BuildConfiguration === "Debug" && config.diagnosticTracing === undefined) {
  config.diagnosticTracing = true;
  }
@@ -264,14 +257,13 @@ export async function mono_wasm_load_config(module: DotnetModuleInternal): Promi
  }
 }
 
-export function hasDebuggingEnabled(config: MonoConfigInternal): boolean {
+export function isDebuggingSupported(): boolean {
  // Copied from blazor MonoDebugger.ts/attachDebuggerHotkey
  if (!globalThis.navigator) {
  return false;
  }
 
- const hasReferencedPdbs = !!config.resources!.pdb;
- return (hasReferencedPdbs || config.debugLevel != 0) && (loaderHelpers.isChromium || loaderHelpers.isFirefox);
+ return loaderHelpers.isChromium || loaderHelpers.isFirefox;
 }
 
 async function loadBootConfig(module: DotnetModuleInternal): Promise<void> {

src/mono/browser/runtime/loader/globals.ts

@@ -15,7 +15,7 @@ import { assertIsControllablePromise, createPromiseController, getPromiseControl
 import { mono_download_assets, resolve_single_asset_path, retrieve_asset_download } from "./assets";
 import { mono_log_error, set_thread_prefix, setup_proxy_console } from "./logging";
 import { invokeLibraryInitializers } from "./libraryInitializers";
-import { deep_merge_config, hasDebuggingEnabled } from "./config";
+import { deep_merge_config, isDebuggingSupported } from "./config";
 import { logDownloadStatsToConsole, purgeUnusedCacheEntriesAsync } from "./assetsCache";
 
 // if we are the first script loaded in the web worker, we are expected to become the sidecar
@@ -118,9 +118,9 @@ export function setLoaderGlobals(
  purgeUnusedCacheEntriesAsync,
  installUnhandledErrorHandler,
 
- hasDebuggingEnabled,
  retrieve_asset_download,
  invokeLibraryInitializers,
+ isDebuggingSupported,
 
  // from wasm-feature-detect npm package
  exceptions,

src/mono/browser/runtime/startup.ts

@@ -515,7 +515,7 @@ async function start_runtime() {
  if (runtimeHelpers.config.browserProfilerOptions)
  mono_wasm_init_browser_profiler(runtimeHelpers.config.browserProfilerOptions);
 
- mono_wasm_load_runtime("unused", runtimeHelpers.config.debugLevel);
+ mono_wasm_load_runtime();
 
  if (runtimeHelpers.config.interpreterPgo)
  setTimeout(maybeSaveInterpPgoTable, (runtimeHelpers.config.interpreterPgoSaveDelay || 15) * 1000);
@@ -534,17 +534,21 @@ async function maybeSaveInterpPgoTable() {
  await interp_pgo_save_data();
 }
 
-export function mono_wasm_load_runtime(unused?: string, debugLevel?: number): void {
+export function mono_wasm_load_runtime(): void {
  mono_log_debug("mono_wasm_load_runtime");
  try {
  const mark = startMeasure();
+ let debugLevel = runtimeHelpers.config.debugLevel;
  if (debugLevel == undefined) {
  debugLevel = 0;
  if (runtimeHelpers.config.debugLevel) {
  debugLevel = 0 + debugLevel;
  }
  }
- cwraps.mono_wasm_load_runtime(unused || "unused", debugLevel);
+ if (!loaderHelpers.isDebuggingSupported() || !runtimeHelpers.config.resources!.pdb) {
+ debugLevel = 0;
+ }
+ cwraps.mono_wasm_load_runtime(debugLevel);
  endMeasure(mark, MeasuredBlock.loadRuntime);
 
  } catch (err: any) {

src/mono/browser/runtime/types/internal.ts

@@ -155,7 +155,6 @@ export type LoaderHelpers = {
  out(message: string): void;
  err(message: string): void;
 
- hasDebuggingEnabled(config: MonoConfig): boolean,
  retrieve_asset_download(asset: AssetEntry): Promise<ArrayBuffer>;
  onDownloadResourceProgress?: (resourcesLoaded: number, totalResources: number) => void;
  logDownloadStatsToConsole: () => void;
@@ -166,6 +165,7 @@ export type LoaderHelpers = {
  invokeLibraryInitializers: (functionName: string, args: any[]) => Promise<void>,
  libraryInitializers?: { scriptName: string, exports: any }[];
 
+ isDebuggingSupported(): boolean,
  isChromium: boolean,
  isFirefox: boolean
 

src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

@@ -203,9 +203,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  <_BlazorWebAssemblyStartupMemoryCache>$(BlazorWebAssemblyStartupMemoryCache)</_BlazorWebAssemblyStartupMemoryCache>
  <_BlazorWebAssemblyJiterpreter>$(BlazorWebAssemblyJiterpreter)</_BlazorWebAssemblyJiterpreter>
  <_BlazorWebAssemblyRuntimeOptions>$(BlazorWebAssemblyRuntimeOptions)</_BlazorWebAssemblyRuntimeOptions>
- <_WasmDebugLevel>$(WasmDebugLevel)</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == ''">0</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == '0' and ('$(DebuggerSupport)' == 'true' or '$(Configuration)' == 'Debug')">-1</_WasmDebugLevel>
 
  <!-- Workaround for https://github.com/dotnet/sdk/issues/12114-->
  <PublishDir Condition="'$(AppendRuntimeIdentifierToOutputPath)' != 'true' AND '$(PublishDir)' == '$(OutputPath)$(RuntimeIdentifier)\$(PublishDirName)\'">$(OutputPath)$(PublishDirName)\</PublishDir>
@@ -364,7 +361,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmOutputWithHash)"
  DebugBuild="true"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="false"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(_WasmBuildBootJsonPath)"
@@ -380,7 +377,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="false" />
 
  <ItemGroup>
  <FileWrites Include="$(_WasmBuildBootJsonPath)" />
@@ -533,7 +531,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  </Target>
 
  <Target Name="GeneratePublishWasmBootJson" DependsOnTargets="$(GeneratePublishWasmBootJsonDependsOn)">
-
  <ItemGroup>
  <_WasmPublishAsset
  Include="@(StaticWebAsset)"
@@ -549,7 +546,6 @@ Copyright (c) .NET Foundation. All rights reserved.
 
  <!-- We remove the extensions since they are added to the list of static web assets but we need to compute the target path for them -->
  <_WasmPublishAsset Remove="@(_BlazorExtensionsCandidatesForPublish)" />
-
  </ItemGroup>
 
  <ComputeStaticWebAssetsTargetPaths
@@ -568,7 +564,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmPublishBootResourceWithHash)"
  DebugBuild="false"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="$(PublishTrimmed)"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(IntermediateOutputPath)blazor.publish.boot.json"
@@ -584,7 +580,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="true" />
 
  <ItemGroup>
  <FileWrites Include="$(IntermediateOutputPath)blazor.publish.boot.json" />

src/mono/wasi/build/WasiApp.targets

@@ -66,6 +66,11 @@
  Outputs="$(WasmAppDir)\.stamp"
  Condition="'$(WasmGenerateAppBundle)' == 'true'">
 
+ <PropertyGroup>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(CopyOutputSymbolsToPublishDirectory)' == 'true' and '$(_IsPublishing)' == 'true'">true</_WasmOutputSymbolsToAppBundle>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(_WasmOutputSymbolsToAppBundle)' == ''">false</_WasmOutputSymbolsToAppBundle>
+ </PropertyGroup>
+
  <WasiAppBuilder
  AppDir="$(WasmAppDir)"
  Assemblies="@(_WasmAssembliesInternal)"
@@ -80,7 +85,7 @@
  FilesToIncludeInFileSystem="@(WasmFilesToIncludeInFileSystem)"
  ExtraFilesToDeploy="@(WasmExtraFilesToDeploy)"
  NativeAssets="@(WasmNativeAsset)"
-DebugLevel="$(WasmDebugLevel)"
+OutputSymbolsToAppBundle="$(_WasmOutputSymbolsToAppBundle)"
  RuntimeConfigJsonPath="$(_WasmRuntimeConfigFilePath)"
  />
  </Target>

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorWasmTestBase.cs

@@ -63,7 +63,9 @@ public string CreateBlazorWasmTemplateProject(string id)
  (CommandResult res, string logPath) = BlazorBuildInternal(options.Id, options.Config, publish: false, setWasmDevel: false, expectSuccess: options.ExpectSuccess, extraArgs);
 
  if (options.ExpectSuccess && options.AssertAppBundle)
+ {
  AssertBundle(res.Output, options with { IsPublish = false });
+ }
 
  return (res, logPath);
  }
@@ -77,6 +79,10 @@ public string CreateBlazorWasmTemplateProject(string id)
 
  if (options.ExpectSuccess && options.AssertAppBundle)
  {
+ // Because we do relink in Release publish by default
+ if (options.Config == "Release")
+ options = options with { ExpectedFileType = NativeFilesType.Relinked };
+
  AssertBundle(res.Output, options with { IsPublish = true });
  }
 

src/mono/wasm/Wasm.Build.Tests/Templates/InterpPgoTests.cs

@@ -34,7 +34,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
 
  string id = $"browser_{config}_{GetRandomId()}";
  _testOutput.WriteLine("/// Creating project");
- string projectFile = CreateWasmTemplateProject(id, "wasmbrowser");
+ string projectFile = CreateWasmTemplateProject(id, "wasmbrowser", extraProperties: "<WasmDebugLevel>0</WasmDebugLevel>");
 
  _testOutput.WriteLine("/// Updating JS");
  UpdateBrowserMainJs((js) => {
@@ -53,6 +53,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
  // then call INTERNAL.interp_pgo_save_data() to save the interp PGO table
  js = js.Replace(
  "const text = exports.MyClass.Greeting();",
+ "console.log(`WASM debug level ${getConfig().debugLevel}`);\n" + 
  "let text = '';\n" +
  $"for (let i = 0; i < {iterationCount}; i++) {{ text = exports.MyClass.Greeting(); }};\n" +
  "await INTERNAL.interp_pgo_save_data();"

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppSettingsTests.cs

@@ -28,7 +28,7 @@ public async Task LoadAppSettingsBasedOnApplicationEnvironment(string applicatio
  CopyTestAsset("WasmBasicTestApp", "AppSettingsTests");
  PublishProject("Debug");
 
- var result = await RunSdkStyleApp(new(
+ var result = await RunSdkStyleAppForPublish(new(
  Configuration: "Debug",
  TestScenario: "AppSettingsTest",
  BrowserQueryString: new Dictionary<string, string> { ["applicationEnvironment"] = applicationEnvironment }

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppTestBase.cs

@@ -36,23 +36,29 @@ protected void CopyTestAsset(string assetName, string generatedProjectNamePrefix
  _projectDir = Path.Combine(_projectDir!, "App");
  }
 
- protected void BuildProject(string configuration)
+ protected void BuildProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
- protected void PublishProject(string configuration)
+ protected void PublishProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
  protected ToolCommand CreateDotNetCommand() => new DotNetCommand(s_buildEnv, _testOutput)
  .WithWorkingDirectory(_projectDir!)
  .WithEnvironmentVariable("NUGET_PACKAGES", _nugetPackagesDir);
 
- protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
+ protected Task<RunResult> RunSdkStyleAppForBuild(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.DotnetRun);
+
+ protected Task<RunResult> RunSdkStyleAppForPublish(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.WebServer);
+
+ private async Task<RunResult> RunSdkStyleApp(RunOptions options, BlazorRunHost host = BlazorRunHost.DotnetRun)
  {
  string queryString = "?test=" + options.TestScenario;
  if (options.BrowserQueryString != null)
@@ -67,9 +73,10 @@ protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
  CheckCounter: false,
  Config: options.Configuration,
  OnConsoleMessage: OnConsoleMessage,
- QueryString: queryString);
+ QueryString: queryString,
+ Host: host);
 
- await BlazorRunForBuildWithDotnetRun(blazorRunOptions);
+ await BlazorRunTest(blazorRunOptions);
 
  void OnConsoleMessage(IConsoleMessage msg)
  {

eng/testing/scenarios/BuildWasmAppsJobsList.txt

@@ -44,3 +44,4 @@ Wasm.Build.Tests.WasmTemplateTests
 Wasm.Build.Tests.WorkloadTests
 Wasm.Build.Tests.TestAppScenarios.DownloadResourceProgressTests
 Wasm.Build.Tests.MT.Blazor.SimpleMultiThreadedTests
+Wasm.Build.Tests.TestAppScenarios.DebugLevelTests

src/mono/browser/runtime/cwraps.ts

@@ -166,7 +166,7 @@ export interface t_Cwraps {
  mono_wasm_load_icu_data(offset: VoidPtr): number;
  mono_wasm_add_assembly(name: string, data: VoidPtr, size: number): number;
  mono_wasm_add_satellite_assembly(name: string, culture: string, data: VoidPtr, size: number): void;
- mono_wasm_load_runtime(unused: string, debugLevel: number): void;
+ mono_wasm_load_runtime(debugLevel: number): void;
  mono_wasm_change_debugger_log_level(value: number): void;
 
  mono_wasm_assembly_load(name: string): MonoAssembly;

src/mono/browser/runtime/driver.c

@@ -181,7 +181,7 @@ cleanup_runtime_config (MonovmRuntimeConfigArguments *args, void *user_data)
 }
 
 EMSCRIPTEN_KEEPALIVE void
-mono_wasm_load_runtime (const char *unused, int debug_level)
+mono_wasm_load_runtime (int debug_level)
 {
 	const char *interp_opts = "";
 

src/mono/browser/runtime/lazyLoading.ts

@@ -26,7 +26,7 @@ export async function loadLazyAssembly(assemblyNameToLoad: string): Promise<bool
  }
 
  const pdbNameToLoad = changeExtension(dllAsset.name, ".pdb");
- const shouldLoadPdb = loaderHelpers.hasDebuggingEnabled(loaderHelpers.config) && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
+ const shouldLoadPdb = loaderHelpers.config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
 
  const dllBytesPromise = loaderHelpers.retrieve_asset_download(dllAsset);
 

src/mono/browser/runtime/loader/assets.ts

@@ -266,7 +266,7 @@ export function prepareAssets() {
  }
  }
 
- if (config.debugLevel != 0 && resources.pdb) {
+ if (config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && resources.pdb) {
  for (const name in resources.pdb) {
  assetsToLoad.push({
  name,

src/mono/browser/runtime/loader/config.ts

@@ -197,13 +197,6 @@ export function normalizeConfig() {
  config.environmentVariables["MONO_SLEEP_ABORT_LIMIT"] = "5000";
  }
 
- // Default values (when WasmDebugLevel is not set)
- // - Build (debug) => debugBuild=true & debugLevel=-1 => -1
- // - Build (release) => debugBuild=true & debugLevel=0 => 0
- // - Publish (debug) => debugBuild=false & debugLevel=-1 => 0
- // - Publish (release) => debugBuild=false & debugLevel=0 => 0
- config.debugLevel = hasDebuggingEnabled(config) ? config.debugLevel : 0;
-
  if (BuildConfiguration === "Debug" && config.diagnosticTracing === undefined) {
  config.diagnosticTracing = true;
  }
@@ -264,14 +257,13 @@ export async function mono_wasm_load_config(module: DotnetModuleInternal): Promi
  }
 }
 
-export function hasDebuggingEnabled(config: MonoConfigInternal): boolean {
+export function isDebuggingSupported(): boolean {
  // Copied from blazor MonoDebugger.ts/attachDebuggerHotkey
  if (!globalThis.navigator) {
  return false;
  }
 
- const hasReferencedPdbs = !!config.resources!.pdb;
- return (hasReferencedPdbs || config.debugLevel != 0) && (loaderHelpers.isChromium || loaderHelpers.isFirefox);
+ return loaderHelpers.isChromium || loaderHelpers.isFirefox;
 }
 
 async function loadBootConfig(module: DotnetModuleInternal): Promise<void> {

src/mono/browser/runtime/loader/globals.ts

@@ -15,7 +15,7 @@ import { assertIsControllablePromise, createPromiseController, getPromiseControl
 import { mono_download_assets, resolve_single_asset_path, retrieve_asset_download } from "./assets";
 import { mono_log_error, set_thread_prefix, setup_proxy_console } from "./logging";
 import { invokeLibraryInitializers } from "./libraryInitializers";
-import { deep_merge_config, hasDebuggingEnabled } from "./config";
+import { deep_merge_config, isDebuggingSupported } from "./config";
 import { logDownloadStatsToConsole, purgeUnusedCacheEntriesAsync } from "./assetsCache";
 
 // if we are the first script loaded in the web worker, we are expected to become the sidecar
@@ -118,9 +118,9 @@ export function setLoaderGlobals(
  purgeUnusedCacheEntriesAsync,
  installUnhandledErrorHandler,
 
- hasDebuggingEnabled,
  retrieve_asset_download,
  invokeLibraryInitializers,
+ isDebuggingSupported,
 
  // from wasm-feature-detect npm package
  exceptions,

src/mono/browser/runtime/startup.ts

@@ -515,7 +515,7 @@ async function start_runtime() {
  if (runtimeHelpers.config.browserProfilerOptions)
  mono_wasm_init_browser_profiler(runtimeHelpers.config.browserProfilerOptions);
 
- mono_wasm_load_runtime("unused", runtimeHelpers.config.debugLevel);
+ mono_wasm_load_runtime();
 
  if (runtimeHelpers.config.interpreterPgo)
  setTimeout(maybeSaveInterpPgoTable, (runtimeHelpers.config.interpreterPgoSaveDelay || 15) * 1000);
@@ -534,17 +534,21 @@ async function maybeSaveInterpPgoTable() {
  await interp_pgo_save_data();
 }
 
-export function mono_wasm_load_runtime(unused?: string, debugLevel?: number): void {
+export function mono_wasm_load_runtime(): void {
  mono_log_debug("mono_wasm_load_runtime");
  try {
  const mark = startMeasure();
+ let debugLevel = runtimeHelpers.config.debugLevel;
  if (debugLevel == undefined) {
  debugLevel = 0;
  if (runtimeHelpers.config.debugLevel) {
  debugLevel = 0 + debugLevel;
  }
  }
- cwraps.mono_wasm_load_runtime(unused || "unused", debugLevel);
+ if (!loaderHelpers.isDebuggingSupported() || !runtimeHelpers.config.resources!.pdb) {
+ debugLevel = 0;
+ }
+ cwraps.mono_wasm_load_runtime(debugLevel);
  endMeasure(mark, MeasuredBlock.loadRuntime);
 
  } catch (err: any) {

src/mono/browser/runtime/types/internal.ts

@@ -155,7 +155,6 @@ export type LoaderHelpers = {
  out(message: string): void;
  err(message: string): void;
 
- hasDebuggingEnabled(config: MonoConfig): boolean,
  retrieve_asset_download(asset: AssetEntry): Promise<ArrayBuffer>;
  onDownloadResourceProgress?: (resourcesLoaded: number, totalResources: number) => void;
  logDownloadStatsToConsole: () => void;
@@ -166,6 +165,7 @@ export type LoaderHelpers = {
  invokeLibraryInitializers: (functionName: string, args: any[]) => Promise<void>,
  libraryInitializers?: { scriptName: string, exports: any }[];
 
+ isDebuggingSupported(): boolean,
  isChromium: boolean,
  isFirefox: boolean
 

src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

@@ -203,9 +203,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  <_BlazorWebAssemblyStartupMemoryCache>$(BlazorWebAssemblyStartupMemoryCache)</_BlazorWebAssemblyStartupMemoryCache>
  <_BlazorWebAssemblyJiterpreter>$(BlazorWebAssemblyJiterpreter)</_BlazorWebAssemblyJiterpreter>
  <_BlazorWebAssemblyRuntimeOptions>$(BlazorWebAssemblyRuntimeOptions)</_BlazorWebAssemblyRuntimeOptions>
- <_WasmDebugLevel>$(WasmDebugLevel)</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == ''">0</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == '0' and ('$(DebuggerSupport)' == 'true' or '$(Configuration)' == 'Debug')">-1</_WasmDebugLevel>
 
  <!-- Workaround for https://github.com/dotnet/sdk/issues/12114-->
  <PublishDir Condition="'$(AppendRuntimeIdentifierToOutputPath)' != 'true' AND '$(PublishDir)' == '$(OutputPath)$(RuntimeIdentifier)\$(PublishDirName)\'">$(OutputPath)$(PublishDirName)\</PublishDir>
@@ -364,7 +361,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmOutputWithHash)"
  DebugBuild="true"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="false"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(_WasmBuildBootJsonPath)"
@@ -380,7 +377,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="false" />
 
  <ItemGroup>
  <FileWrites Include="$(_WasmBuildBootJsonPath)" />
@@ -533,7 +531,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  </Target>
 
  <Target Name="GeneratePublishWasmBootJson" DependsOnTargets="$(GeneratePublishWasmBootJsonDependsOn)">
-
  <ItemGroup>
  <_WasmPublishAsset
  Include="@(StaticWebAsset)"
@@ -549,7 +546,6 @@ Copyright (c) .NET Foundation. All rights reserved.
 
  <!-- We remove the extensions since they are added to the list of static web assets but we need to compute the target path for them -->
  <_WasmPublishAsset Remove="@(_BlazorExtensionsCandidatesForPublish)" />
-
  </ItemGroup>
 
  <ComputeStaticWebAssetsTargetPaths
@@ -568,7 +564,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmPublishBootResourceWithHash)"
  DebugBuild="false"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="$(PublishTrimmed)"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(IntermediateOutputPath)blazor.publish.boot.json"
@@ -584,7 +580,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="true" />
 
  <ItemGroup>
  <FileWrites Include="$(IntermediateOutputPath)blazor.publish.boot.json" />

src/mono/wasi/build/WasiApp.targets

@@ -66,6 +66,11 @@
  Outputs="$(WasmAppDir)\.stamp"
  Condition="'$(WasmGenerateAppBundle)' == 'true'">
 
+ <PropertyGroup>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(CopyOutputSymbolsToPublishDirectory)' == 'true' and '$(_IsPublishing)' == 'true'">true</_WasmOutputSymbolsToAppBundle>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(_WasmOutputSymbolsToAppBundle)' == ''">false</_WasmOutputSymbolsToAppBundle>
+ </PropertyGroup>
+
  <WasiAppBuilder
  AppDir="$(WasmAppDir)"
  Assemblies="@(_WasmAssembliesInternal)"
@@ -80,7 +85,7 @@
  FilesToIncludeInFileSystem="@(WasmFilesToIncludeInFileSystem)"
  ExtraFilesToDeploy="@(WasmExtraFilesToDeploy)"
  NativeAssets="@(WasmNativeAsset)"
-DebugLevel="$(WasmDebugLevel)"
+OutputSymbolsToAppBundle="$(_WasmOutputSymbolsToAppBundle)"
  RuntimeConfigJsonPath="$(_WasmRuntimeConfigFilePath)"
  />
  </Target>

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorWasmTestBase.cs

@@ -63,7 +63,9 @@ public string CreateBlazorWasmTemplateProject(string id)
  (CommandResult res, string logPath) = BlazorBuildInternal(options.Id, options.Config, publish: false, setWasmDevel: false, expectSuccess: options.ExpectSuccess, extraArgs);
 
  if (options.ExpectSuccess && options.AssertAppBundle)
+ {
  AssertBundle(res.Output, options with { IsPublish = false });
+ }
 
  return (res, logPath);
  }
@@ -77,6 +79,10 @@ public string CreateBlazorWasmTemplateProject(string id)
 
  if (options.ExpectSuccess && options.AssertAppBundle)
  {
+ // Because we do relink in Release publish by default
+ if (options.Config == "Release")
+ options = options with { ExpectedFileType = NativeFilesType.Relinked };
+
  AssertBundle(res.Output, options with { IsPublish = true });
  }
 

src/mono/wasm/Wasm.Build.Tests/Templates/InterpPgoTests.cs

@@ -34,7 +34,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
 
  string id = $"browser_{config}_{GetRandomId()}";
  _testOutput.WriteLine("/// Creating project");
- string projectFile = CreateWasmTemplateProject(id, "wasmbrowser");
+ string projectFile = CreateWasmTemplateProject(id, "wasmbrowser", extraProperties: "<WasmDebugLevel>0</WasmDebugLevel>");
 
  _testOutput.WriteLine("/// Updating JS");
  UpdateBrowserMainJs((js) => {
@@ -53,6 +53,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
  // then call INTERNAL.interp_pgo_save_data() to save the interp PGO table
  js = js.Replace(
  "const text = exports.MyClass.Greeting();",
+ "console.log(`WASM debug level ${getConfig().debugLevel}`);\n" + 
  "let text = '';\n" +
  $"for (let i = 0; i < {iterationCount}; i++) {{ text = exports.MyClass.Greeting(); }};\n" +
  "await INTERNAL.interp_pgo_save_data();"

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppSettingsTests.cs

@@ -28,7 +28,7 @@ public async Task LoadAppSettingsBasedOnApplicationEnvironment(string applicatio
  CopyTestAsset("WasmBasicTestApp", "AppSettingsTests");
  PublishProject("Debug");
 
- var result = await RunSdkStyleApp(new(
+ var result = await RunSdkStyleAppForPublish(new(
  Configuration: "Debug",
  TestScenario: "AppSettingsTest",
  BrowserQueryString: new Dictionary<string, string> { ["applicationEnvironment"] = applicationEnvironment }

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppTestBase.cs

@@ -36,23 +36,29 @@ protected void CopyTestAsset(string assetName, string generatedProjectNamePrefix
  _projectDir = Path.Combine(_projectDir!, "App");
  }
 
- protected void BuildProject(string configuration)
+ protected void BuildProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
- protected void PublishProject(string configuration)
+ protected void PublishProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
  protected ToolCommand CreateDotNetCommand() => new DotNetCommand(s_buildEnv, _testOutput)
  .WithWorkingDirectory(_projectDir!)
  .WithEnvironmentVariable("NUGET_PACKAGES", _nugetPackagesDir);
 
- protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
+ protected Task<RunResult> RunSdkStyleAppForBuild(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.DotnetRun);
+
+ protected Task<RunResult> RunSdkStyleAppForPublish(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.WebServer);
+
+ private async Task<RunResult> RunSdkStyleApp(RunOptions options, BlazorRunHost host = BlazorRunHost.DotnetRun)
  {
  string queryString = "?test=" + options.TestScenario;
  if (options.BrowserQueryString != null)
@@ -67,9 +73,10 @@ protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
  CheckCounter: false,
  Config: options.Configuration,
  OnConsoleMessage: OnConsoleMessage,
- QueryString: queryString);
+ QueryString: queryString,
+ Host: host);
 
- await BlazorRunForBuildWithDotnetRun(blazorRunOptions);
+ await BlazorRunTest(blazorRunOptions);
 
  void OnConsoleMessage(IConsoleMessage msg)
  {

eng/testing/scenarios/BuildWasmAppsJobsList.txt

@@ -44,3 +44,4 @@ Wasm.Build.Tests.WasmTemplateTests
 Wasm.Build.Tests.WorkloadTests
 Wasm.Build.Tests.TestAppScenarios.DownloadResourceProgressTests
 Wasm.Build.Tests.MT.Blazor.SimpleMultiThreadedTests
+Wasm.Build.Tests.TestAppScenarios.DebugLevelTests

src/mono/browser/runtime/cwraps.ts

@@ -166,7 +166,7 @@ export interface t_Cwraps {
  mono_wasm_load_icu_data(offset: VoidPtr): number;
  mono_wasm_add_assembly(name: string, data: VoidPtr, size: number): number;
  mono_wasm_add_satellite_assembly(name: string, culture: string, data: VoidPtr, size: number): void;
- mono_wasm_load_runtime(unused: string, debugLevel: number): void;
+ mono_wasm_load_runtime(debugLevel: number): void;
  mono_wasm_change_debugger_log_level(value: number): void;
 
  mono_wasm_assembly_load(name: string): MonoAssembly;

src/mono/browser/runtime/driver.c

@@ -181,7 +181,7 @@ cleanup_runtime_config (MonovmRuntimeConfigArguments *args, void *user_data)
 }
 
 EMSCRIPTEN_KEEPALIVE void
-mono_wasm_load_runtime (const char *unused, int debug_level)
+mono_wasm_load_runtime (int debug_level)
 {
 	const char *interp_opts = "";
 

src/mono/browser/runtime/lazyLoading.ts

@@ -26,7 +26,7 @@ export async function loadLazyAssembly(assemblyNameToLoad: string): Promise<bool
  }
 
  const pdbNameToLoad = changeExtension(dllAsset.name, ".pdb");
- const shouldLoadPdb = loaderHelpers.hasDebuggingEnabled(loaderHelpers.config) && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
+ const shouldLoadPdb = loaderHelpers.config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
 
  const dllBytesPromise = loaderHelpers.retrieve_asset_download(dllAsset);
 

src/mono/browser/runtime/loader/assets.ts

@@ -266,7 +266,7 @@ export function prepareAssets() {
  }
  }
 
- if (config.debugLevel != 0 && resources.pdb) {
+ if (config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && resources.pdb) {
  for (const name in resources.pdb) {
  assetsToLoad.push({
  name,

src/mono/browser/runtime/loader/config.ts

@@ -197,13 +197,6 @@ export function normalizeConfig() {
  config.environmentVariables["MONO_SLEEP_ABORT_LIMIT"] = "5000";
  }
 
- // Default values (when WasmDebugLevel is not set)
- // - Build (debug) => debugBuild=true & debugLevel=-1 => -1
- // - Build (release) => debugBuild=true & debugLevel=0 => 0
- // - Publish (debug) => debugBuild=false & debugLevel=-1 => 0
- // - Publish (release) => debugBuild=false & debugLevel=0 => 0
- config.debugLevel = hasDebuggingEnabled(config) ? config.debugLevel : 0;
-
  if (BuildConfiguration === "Debug" && config.diagnosticTracing === undefined) {
  config.diagnosticTracing = true;
  }
@@ -264,14 +257,13 @@ export async function mono_wasm_load_config(module: DotnetModuleInternal): Promi
  }
 }
 
-export function hasDebuggingEnabled(config: MonoConfigInternal): boolean {
+export function isDebuggingSupported(): boolean {
  // Copied from blazor MonoDebugger.ts/attachDebuggerHotkey
  if (!globalThis.navigator) {
  return false;
  }
 
- const hasReferencedPdbs = !!config.resources!.pdb;
- return (hasReferencedPdbs || config.debugLevel != 0) && (loaderHelpers.isChromium || loaderHelpers.isFirefox);
+ return loaderHelpers.isChromium || loaderHelpers.isFirefox;
 }
 
 async function loadBootConfig(module: DotnetModuleInternal): Promise<void> {

src/mono/browser/runtime/loader/globals.ts

@@ -15,7 +15,7 @@ import { assertIsControllablePromise, createPromiseController, getPromiseControl
 import { mono_download_assets, resolve_single_asset_path, retrieve_asset_download } from "./assets";
 import { mono_log_error, set_thread_prefix, setup_proxy_console } from "./logging";
 import { invokeLibraryInitializers } from "./libraryInitializers";
-import { deep_merge_config, hasDebuggingEnabled } from "./config";
+import { deep_merge_config, isDebuggingSupported } from "./config";
 import { logDownloadStatsToConsole, purgeUnusedCacheEntriesAsync } from "./assetsCache";
 
 // if we are the first script loaded in the web worker, we are expected to become the sidecar
@@ -118,9 +118,9 @@ export function setLoaderGlobals(
  purgeUnusedCacheEntriesAsync,
  installUnhandledErrorHandler,
 
- hasDebuggingEnabled,
  retrieve_asset_download,
  invokeLibraryInitializers,
+ isDebuggingSupported,
 
  // from wasm-feature-detect npm package
  exceptions,

src/mono/browser/runtime/startup.ts

@@ -515,7 +515,7 @@ async function start_runtime() {
  if (runtimeHelpers.config.browserProfilerOptions)
  mono_wasm_init_browser_profiler(runtimeHelpers.config.browserProfilerOptions);
 
- mono_wasm_load_runtime("unused", runtimeHelpers.config.debugLevel);
+ mono_wasm_load_runtime();
 
  if (runtimeHelpers.config.interpreterPgo)
  setTimeout(maybeSaveInterpPgoTable, (runtimeHelpers.config.interpreterPgoSaveDelay || 15) * 1000);
@@ -534,17 +534,21 @@ async function maybeSaveInterpPgoTable() {
  await interp_pgo_save_data();
 }
 
-export function mono_wasm_load_runtime(unused?: string, debugLevel?: number): void {
+export function mono_wasm_load_runtime(): void {
  mono_log_debug("mono_wasm_load_runtime");
  try {
  const mark = startMeasure();
+ let debugLevel = runtimeHelpers.config.debugLevel;
  if (debugLevel == undefined) {
  debugLevel = 0;
  if (runtimeHelpers.config.debugLevel) {
  debugLevel = 0 + debugLevel;
  }
  }
- cwraps.mono_wasm_load_runtime(unused || "unused", debugLevel);
+ if (!loaderHelpers.isDebuggingSupported() || !runtimeHelpers.config.resources!.pdb) {
+ debugLevel = 0;
+ }
+ cwraps.mono_wasm_load_runtime(debugLevel);
  endMeasure(mark, MeasuredBlock.loadRuntime);
 
  } catch (err: any) {

src/mono/browser/runtime/types/internal.ts

@@ -155,7 +155,6 @@ export type LoaderHelpers = {
  out(message: string): void;
  err(message: string): void;
 
- hasDebuggingEnabled(config: MonoConfig): boolean,
  retrieve_asset_download(asset: AssetEntry): Promise<ArrayBuffer>;
  onDownloadResourceProgress?: (resourcesLoaded: number, totalResources: number) => void;
  logDownloadStatsToConsole: () => void;
@@ -166,6 +165,7 @@ export type LoaderHelpers = {
  invokeLibraryInitializers: (functionName: string, args: any[]) => Promise<void>,
  libraryInitializers?: { scriptName: string, exports: any }[];
 
+ isDebuggingSupported(): boolean,
  isChromium: boolean,
  isFirefox: boolean
 

src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

@@ -203,9 +203,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  <_BlazorWebAssemblyStartupMemoryCache>$(BlazorWebAssemblyStartupMemoryCache)</_BlazorWebAssemblyStartupMemoryCache>
  <_BlazorWebAssemblyJiterpreter>$(BlazorWebAssemblyJiterpreter)</_BlazorWebAssemblyJiterpreter>
  <_BlazorWebAssemblyRuntimeOptions>$(BlazorWebAssemblyRuntimeOptions)</_BlazorWebAssemblyRuntimeOptions>
- <_WasmDebugLevel>$(WasmDebugLevel)</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == ''">0</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == '0' and ('$(DebuggerSupport)' == 'true' or '$(Configuration)' == 'Debug')">-1</_WasmDebugLevel>
 
  <!-- Workaround for https://github.com/dotnet/sdk/issues/12114-->
  <PublishDir Condition="'$(AppendRuntimeIdentifierToOutputPath)' != 'true' AND '$(PublishDir)' == '$(OutputPath)$(RuntimeIdentifier)\$(PublishDirName)\'">$(OutputPath)$(PublishDirName)\</PublishDir>
@@ -364,7 +361,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmOutputWithHash)"
  DebugBuild="true"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="false"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(_WasmBuildBootJsonPath)"
@@ -380,7 +377,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="false" />
 
  <ItemGroup>
  <FileWrites Include="$(_WasmBuildBootJsonPath)" />
@@ -533,7 +531,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  </Target>
 
  <Target Name="GeneratePublishWasmBootJson" DependsOnTargets="$(GeneratePublishWasmBootJsonDependsOn)">
-
  <ItemGroup>
  <_WasmPublishAsset
  Include="@(StaticWebAsset)"
@@ -549,7 +546,6 @@ Copyright (c) .NET Foundation. All rights reserved.
 
  <!-- We remove the extensions since they are added to the list of static web assets but we need to compute the target path for them -->
  <_WasmPublishAsset Remove="@(_BlazorExtensionsCandidatesForPublish)" />
-
  </ItemGroup>
 
  <ComputeStaticWebAssetsTargetPaths
@@ -568,7 +564,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmPublishBootResourceWithHash)"
  DebugBuild="false"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="$(PublishTrimmed)"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(IntermediateOutputPath)blazor.publish.boot.json"
@@ -584,7 +580,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="true" />
 
  <ItemGroup>
  <FileWrites Include="$(IntermediateOutputPath)blazor.publish.boot.json" />

src/mono/wasi/build/WasiApp.targets

@@ -66,6 +66,11 @@
  Outputs="$(WasmAppDir)\.stamp"
  Condition="'$(WasmGenerateAppBundle)' == 'true'">
 
+ <PropertyGroup>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(CopyOutputSymbolsToPublishDirectory)' == 'true' and '$(_IsPublishing)' == 'true'">true</_WasmOutputSymbolsToAppBundle>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(_WasmOutputSymbolsToAppBundle)' == ''">false</_WasmOutputSymbolsToAppBundle>
+ </PropertyGroup>
+
  <WasiAppBuilder
  AppDir="$(WasmAppDir)"
  Assemblies="@(_WasmAssembliesInternal)"
@@ -80,7 +85,7 @@
  FilesToIncludeInFileSystem="@(WasmFilesToIncludeInFileSystem)"
  ExtraFilesToDeploy="@(WasmExtraFilesToDeploy)"
  NativeAssets="@(WasmNativeAsset)"
-DebugLevel="$(WasmDebugLevel)"
+OutputSymbolsToAppBundle="$(_WasmOutputSymbolsToAppBundle)"
  RuntimeConfigJsonPath="$(_WasmRuntimeConfigFilePath)"
  />
  </Target>

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorWasmTestBase.cs

@@ -63,7 +63,9 @@ public string CreateBlazorWasmTemplateProject(string id)
  (CommandResult res, string logPath) = BlazorBuildInternal(options.Id, options.Config, publish: false, setWasmDevel: false, expectSuccess: options.ExpectSuccess, extraArgs);
 
  if (options.ExpectSuccess && options.AssertAppBundle)
+ {
  AssertBundle(res.Output, options with { IsPublish = false });
+ }
 
  return (res, logPath);
  }
@@ -77,6 +79,10 @@ public string CreateBlazorWasmTemplateProject(string id)
 
  if (options.ExpectSuccess && options.AssertAppBundle)
  {
+ // Because we do relink in Release publish by default
+ if (options.Config == "Release")
+ options = options with { ExpectedFileType = NativeFilesType.Relinked };
+
  AssertBundle(res.Output, options with { IsPublish = true });
  }
 

src/mono/wasm/Wasm.Build.Tests/Templates/InterpPgoTests.cs

@@ -34,7 +34,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
 
  string id = $"browser_{config}_{GetRandomId()}";
  _testOutput.WriteLine("/// Creating project");
- string projectFile = CreateWasmTemplateProject(id, "wasmbrowser");
+ string projectFile = CreateWasmTemplateProject(id, "wasmbrowser", extraProperties: "<WasmDebugLevel>0</WasmDebugLevel>");
 
  _testOutput.WriteLine("/// Updating JS");
  UpdateBrowserMainJs((js) => {
@@ -53,6 +53,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
  // then call INTERNAL.interp_pgo_save_data() to save the interp PGO table
  js = js.Replace(
  "const text = exports.MyClass.Greeting();",
+ "console.log(`WASM debug level ${getConfig().debugLevel}`);\n" + 
  "let text = '';\n" +
  $"for (let i = 0; i < {iterationCount}; i++) {{ text = exports.MyClass.Greeting(); }};\n" +
  "await INTERNAL.interp_pgo_save_data();"

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppSettingsTests.cs

@@ -28,7 +28,7 @@ public async Task LoadAppSettingsBasedOnApplicationEnvironment(string applicatio
  CopyTestAsset("WasmBasicTestApp", "AppSettingsTests");
  PublishProject("Debug");
 
- var result = await RunSdkStyleApp(new(
+ var result = await RunSdkStyleAppForPublish(new(
  Configuration: "Debug",
  TestScenario: "AppSettingsTest",
  BrowserQueryString: new Dictionary<string, string> { ["applicationEnvironment"] = applicationEnvironment }

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppTestBase.cs

@@ -36,23 +36,29 @@ protected void CopyTestAsset(string assetName, string generatedProjectNamePrefix
  _projectDir = Path.Combine(_projectDir!, "App");
  }
 
- protected void BuildProject(string configuration)
+ protected void BuildProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
- protected void PublishProject(string configuration)
+ protected void PublishProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
  protected ToolCommand CreateDotNetCommand() => new DotNetCommand(s_buildEnv, _testOutput)
  .WithWorkingDirectory(_projectDir!)
  .WithEnvironmentVariable("NUGET_PACKAGES", _nugetPackagesDir);
 
- protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
+ protected Task<RunResult> RunSdkStyleAppForBuild(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.DotnetRun);
+
+ protected Task<RunResult> RunSdkStyleAppForPublish(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.WebServer);
+
+ private async Task<RunResult> RunSdkStyleApp(RunOptions options, BlazorRunHost host = BlazorRunHost.DotnetRun)
  {
  string queryString = "?test=" + options.TestScenario;
  if (options.BrowserQueryString != null)
@@ -67,9 +73,10 @@ protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
  CheckCounter: false,
  Config: options.Configuration,
  OnConsoleMessage: OnConsoleMessage,
- QueryString: queryString);
+ QueryString: queryString,
+ Host: host);
 
- await BlazorRunForBuildWithDotnetRun(blazorRunOptions);
+ await BlazorRunTest(blazorRunOptions);
 
  void OnConsoleMessage(IConsoleMessage msg)
  {

eng/testing/scenarios/BuildWasmAppsJobsList.txt

@@ -44,3 +44,4 @@ Wasm.Build.Tests.WasmTemplateTests
 Wasm.Build.Tests.WorkloadTests
 Wasm.Build.Tests.TestAppScenarios.DownloadResourceProgressTests
 Wasm.Build.Tests.MT.Blazor.SimpleMultiThreadedTests
+Wasm.Build.Tests.TestAppScenarios.DebugLevelTests

src/mono/browser/runtime/cwraps.ts

@@ -166,7 +166,7 @@ export interface t_Cwraps {
  mono_wasm_load_icu_data(offset: VoidPtr): number;
  mono_wasm_add_assembly(name: string, data: VoidPtr, size: number): number;
  mono_wasm_add_satellite_assembly(name: string, culture: string, data: VoidPtr, size: number): void;
- mono_wasm_load_runtime(unused: string, debugLevel: number): void;
+ mono_wasm_load_runtime(debugLevel: number): void;
  mono_wasm_change_debugger_log_level(value: number): void;
 
  mono_wasm_assembly_load(name: string): MonoAssembly;

src/mono/browser/runtime/driver.c

@@ -181,7 +181,7 @@ cleanup_runtime_config (MonovmRuntimeConfigArguments *args, void *user_data)
 }
 
 EMSCRIPTEN_KEEPALIVE void
-mono_wasm_load_runtime (const char *unused, int debug_level)
+mono_wasm_load_runtime (int debug_level)
 {
 	const char *interp_opts = "";
 

src/mono/browser/runtime/lazyLoading.ts

@@ -26,7 +26,7 @@ export async function loadLazyAssembly(assemblyNameToLoad: string): Promise<bool
  }
 
  const pdbNameToLoad = changeExtension(dllAsset.name, ".pdb");
- const shouldLoadPdb = loaderHelpers.hasDebuggingEnabled(loaderHelpers.config) && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
+ const shouldLoadPdb = loaderHelpers.config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
 
  const dllBytesPromise = loaderHelpers.retrieve_asset_download(dllAsset);
 

src/mono/browser/runtime/loader/assets.ts

@@ -266,7 +266,7 @@ export function prepareAssets() {
  }
  }
 
- if (config.debugLevel != 0 && resources.pdb) {
+ if (config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && resources.pdb) {
  for (const name in resources.pdb) {
  assetsToLoad.push({
  name,

src/mono/browser/runtime/loader/config.ts

@@ -197,13 +197,6 @@ export function normalizeConfig() {
  config.environmentVariables["MONO_SLEEP_ABORT_LIMIT"] = "5000";
  }
 
- // Default values (when WasmDebugLevel is not set)
- // - Build (debug) => debugBuild=true & debugLevel=-1 => -1
- // - Build (release) => debugBuild=true & debugLevel=0 => 0
- // - Publish (debug) => debugBuild=false & debugLevel=-1 => 0
- // - Publish (release) => debugBuild=false & debugLevel=0 => 0
- config.debugLevel = hasDebuggingEnabled(config) ? config.debugLevel : 0;
-
  if (BuildConfiguration === "Debug" && config.diagnosticTracing === undefined) {
  config.diagnosticTracing = true;
  }
@@ -264,14 +257,13 @@ export async function mono_wasm_load_config(module: DotnetModuleInternal): Promi
  }
 }
 
-export function hasDebuggingEnabled(config: MonoConfigInternal): boolean {
+export function isDebuggingSupported(): boolean {
  // Copied from blazor MonoDebugger.ts/attachDebuggerHotkey
  if (!globalThis.navigator) {
  return false;
  }
 
- const hasReferencedPdbs = !!config.resources!.pdb;
- return (hasReferencedPdbs || config.debugLevel != 0) && (loaderHelpers.isChromium || loaderHelpers.isFirefox);
+ return loaderHelpers.isChromium || loaderHelpers.isFirefox;
 }
 
 async function loadBootConfig(module: DotnetModuleInternal): Promise<void> {

src/mono/browser/runtime/loader/globals.ts

@@ -15,7 +15,7 @@ import { assertIsControllablePromise, createPromiseController, getPromiseControl
 import { mono_download_assets, resolve_single_asset_path, retrieve_asset_download } from "./assets";
 import { mono_log_error, set_thread_prefix, setup_proxy_console } from "./logging";
 import { invokeLibraryInitializers } from "./libraryInitializers";
-import { deep_merge_config, hasDebuggingEnabled } from "./config";
+import { deep_merge_config, isDebuggingSupported } from "./config";
 import { logDownloadStatsToConsole, purgeUnusedCacheEntriesAsync } from "./assetsCache";
 
 // if we are the first script loaded in the web worker, we are expected to become the sidecar
@@ -118,9 +118,9 @@ export function setLoaderGlobals(
  purgeUnusedCacheEntriesAsync,
  installUnhandledErrorHandler,
 
- hasDebuggingEnabled,
  retrieve_asset_download,
  invokeLibraryInitializers,
+ isDebuggingSupported,
 
  // from wasm-feature-detect npm package
  exceptions,

src/mono/browser/runtime/startup.ts

@@ -515,7 +515,7 @@ async function start_runtime() {
  if (runtimeHelpers.config.browserProfilerOptions)
  mono_wasm_init_browser_profiler(runtimeHelpers.config.browserProfilerOptions);
 
- mono_wasm_load_runtime("unused", runtimeHelpers.config.debugLevel);
+ mono_wasm_load_runtime();
 
  if (runtimeHelpers.config.interpreterPgo)
  setTimeout(maybeSaveInterpPgoTable, (runtimeHelpers.config.interpreterPgoSaveDelay || 15) * 1000);
@@ -534,17 +534,21 @@ async function maybeSaveInterpPgoTable() {
  await interp_pgo_save_data();
 }
 
-export function mono_wasm_load_runtime(unused?: string, debugLevel?: number): void {
+export function mono_wasm_load_runtime(): void {
  mono_log_debug("mono_wasm_load_runtime");
  try {
  const mark = startMeasure();
+ let debugLevel = runtimeHelpers.config.debugLevel;
  if (debugLevel == undefined) {
  debugLevel = 0;
  if (runtimeHelpers.config.debugLevel) {
  debugLevel = 0 + debugLevel;
  }
  }
- cwraps.mono_wasm_load_runtime(unused || "unused", debugLevel);
+ if (!loaderHelpers.isDebuggingSupported() || !runtimeHelpers.config.resources!.pdb) {
+ debugLevel = 0;
+ }
+ cwraps.mono_wasm_load_runtime(debugLevel);
  endMeasure(mark, MeasuredBlock.loadRuntime);
 
  } catch (err: any) {

src/mono/browser/runtime/types/internal.ts

@@ -155,7 +155,6 @@ export type LoaderHelpers = {
  out(message: string): void;
  err(message: string): void;
 
- hasDebuggingEnabled(config: MonoConfig): boolean,
  retrieve_asset_download(asset: AssetEntry): Promise<ArrayBuffer>;
  onDownloadResourceProgress?: (resourcesLoaded: number, totalResources: number) => void;
  logDownloadStatsToConsole: () => void;
@@ -166,6 +165,7 @@ export type LoaderHelpers = {
  invokeLibraryInitializers: (functionName: string, args: any[]) => Promise<void>,
  libraryInitializers?: { scriptName: string, exports: any }[];
 
+ isDebuggingSupported(): boolean,
  isChromium: boolean,
  isFirefox: boolean
 

src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

@@ -203,9 +203,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  <_BlazorWebAssemblyStartupMemoryCache>$(BlazorWebAssemblyStartupMemoryCache)</_BlazorWebAssemblyStartupMemoryCache>
  <_BlazorWebAssemblyJiterpreter>$(BlazorWebAssemblyJiterpreter)</_BlazorWebAssemblyJiterpreter>
  <_BlazorWebAssemblyRuntimeOptions>$(BlazorWebAssemblyRuntimeOptions)</_BlazorWebAssemblyRuntimeOptions>
- <_WasmDebugLevel>$(WasmDebugLevel)</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == ''">0</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == '0' and ('$(DebuggerSupport)' == 'true' or '$(Configuration)' == 'Debug')">-1</_WasmDebugLevel>
 
  <!-- Workaround for https://github.com/dotnet/sdk/issues/12114-->
  <PublishDir Condition="'$(AppendRuntimeIdentifierToOutputPath)' != 'true' AND '$(PublishDir)' == '$(OutputPath)$(RuntimeIdentifier)\$(PublishDirName)\'">$(OutputPath)$(PublishDirName)\</PublishDir>
@@ -364,7 +361,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmOutputWithHash)"
  DebugBuild="true"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="false"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(_WasmBuildBootJsonPath)"
@@ -380,7 +377,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="false" />
 
  <ItemGroup>
  <FileWrites Include="$(_WasmBuildBootJsonPath)" />
@@ -533,7 +531,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  </Target>
 
  <Target Name="GeneratePublishWasmBootJson" DependsOnTargets="$(GeneratePublishWasmBootJsonDependsOn)">
-
  <ItemGroup>
  <_WasmPublishAsset
  Include="@(StaticWebAsset)"
@@ -549,7 +546,6 @@ Copyright (c) .NET Foundation. All rights reserved.
 
  <!-- We remove the extensions since they are added to the list of static web assets but we need to compute the target path for them -->
  <_WasmPublishAsset Remove="@(_BlazorExtensionsCandidatesForPublish)" />
-
  </ItemGroup>
 
  <ComputeStaticWebAssetsTargetPaths
@@ -568,7 +564,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmPublishBootResourceWithHash)"
  DebugBuild="false"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="$(PublishTrimmed)"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(IntermediateOutputPath)blazor.publish.boot.json"
@@ -584,7 +580,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="true" />
 
  <ItemGroup>
  <FileWrites Include="$(IntermediateOutputPath)blazor.publish.boot.json" />

src/mono/wasi/build/WasiApp.targets

@@ -66,6 +66,11 @@
  Outputs="$(WasmAppDir)\.stamp"
  Condition="'$(WasmGenerateAppBundle)' == 'true'">
 
+ <PropertyGroup>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(CopyOutputSymbolsToPublishDirectory)' == 'true' and '$(_IsPublishing)' == 'true'">true</_WasmOutputSymbolsToAppBundle>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(_WasmOutputSymbolsToAppBundle)' == ''">false</_WasmOutputSymbolsToAppBundle>
+ </PropertyGroup>
+
  <WasiAppBuilder
  AppDir="$(WasmAppDir)"
  Assemblies="@(_WasmAssembliesInternal)"
@@ -80,7 +85,7 @@
  FilesToIncludeInFileSystem="@(WasmFilesToIncludeInFileSystem)"
  ExtraFilesToDeploy="@(WasmExtraFilesToDeploy)"
  NativeAssets="@(WasmNativeAsset)"
-DebugLevel="$(WasmDebugLevel)"
+OutputSymbolsToAppBundle="$(_WasmOutputSymbolsToAppBundle)"
  RuntimeConfigJsonPath="$(_WasmRuntimeConfigFilePath)"
  />
  </Target>

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorWasmTestBase.cs

@@ -63,7 +63,9 @@ public string CreateBlazorWasmTemplateProject(string id)
  (CommandResult res, string logPath) = BlazorBuildInternal(options.Id, options.Config, publish: false, setWasmDevel: false, expectSuccess: options.ExpectSuccess, extraArgs);
 
  if (options.ExpectSuccess && options.AssertAppBundle)
+ {
  AssertBundle(res.Output, options with { IsPublish = false });
+ }
 
  return (res, logPath);
  }
@@ -77,6 +79,10 @@ public string CreateBlazorWasmTemplateProject(string id)
 
  if (options.ExpectSuccess && options.AssertAppBundle)
  {
+ // Because we do relink in Release publish by default
+ if (options.Config == "Release")
+ options = options with { ExpectedFileType = NativeFilesType.Relinked };
+
  AssertBundle(res.Output, options with { IsPublish = true });
  }
 

src/mono/wasm/Wasm.Build.Tests/Templates/InterpPgoTests.cs

@@ -34,7 +34,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
 
  string id = $"browser_{config}_{GetRandomId()}";
  _testOutput.WriteLine("/// Creating project");
- string projectFile = CreateWasmTemplateProject(id, "wasmbrowser");
+ string projectFile = CreateWasmTemplateProject(id, "wasmbrowser", extraProperties: "<WasmDebugLevel>0</WasmDebugLevel>");
 
  _testOutput.WriteLine("/// Updating JS");
  UpdateBrowserMainJs((js) => {
@@ -53,6 +53,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
  // then call INTERNAL.interp_pgo_save_data() to save the interp PGO table
  js = js.Replace(
  "const text = exports.MyClass.Greeting();",
+ "console.log(`WASM debug level ${getConfig().debugLevel}`);\n" + 
  "let text = '';\n" +
  $"for (let i = 0; i < {iterationCount}; i++) {{ text = exports.MyClass.Greeting(); }};\n" +
  "await INTERNAL.interp_pgo_save_data();"

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppSettingsTests.cs

@@ -28,7 +28,7 @@ public async Task LoadAppSettingsBasedOnApplicationEnvironment(string applicatio
  CopyTestAsset("WasmBasicTestApp", "AppSettingsTests");
  PublishProject("Debug");
 
- var result = await RunSdkStyleApp(new(
+ var result = await RunSdkStyleAppForPublish(new(
  Configuration: "Debug",
  TestScenario: "AppSettingsTest",
  BrowserQueryString: new Dictionary<string, string> { ["applicationEnvironment"] = applicationEnvironment }

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppTestBase.cs

@@ -36,23 +36,29 @@ protected void CopyTestAsset(string assetName, string generatedProjectNamePrefix
  _projectDir = Path.Combine(_projectDir!, "App");
  }
 
- protected void BuildProject(string configuration)
+ protected void BuildProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
- protected void PublishProject(string configuration)
+ protected void PublishProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
  protected ToolCommand CreateDotNetCommand() => new DotNetCommand(s_buildEnv, _testOutput)
  .WithWorkingDirectory(_projectDir!)
  .WithEnvironmentVariable("NUGET_PACKAGES", _nugetPackagesDir);
 
- protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
+ protected Task<RunResult> RunSdkStyleAppForBuild(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.DotnetRun);
+
+ protected Task<RunResult> RunSdkStyleAppForPublish(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.WebServer);
+
+ private async Task<RunResult> RunSdkStyleApp(RunOptions options, BlazorRunHost host = BlazorRunHost.DotnetRun)
  {
  string queryString = "?test=" + options.TestScenario;
  if (options.BrowserQueryString != null)
@@ -67,9 +73,10 @@ protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
  CheckCounter: false,
  Config: options.Configuration,
  OnConsoleMessage: OnConsoleMessage,
- QueryString: queryString);
+ QueryString: queryString,
+ Host: host);
 
- await BlazorRunForBuildWithDotnetRun(blazorRunOptions);
+ await BlazorRunTest(blazorRunOptions);
 
  void OnConsoleMessage(IConsoleMessage msg)
  {

eng/testing/scenarios/BuildWasmAppsJobsList.txt

@@ -44,3 +44,4 @@ Wasm.Build.Tests.WasmTemplateTests
 Wasm.Build.Tests.WorkloadTests
 Wasm.Build.Tests.TestAppScenarios.DownloadResourceProgressTests
 Wasm.Build.Tests.MT.Blazor.SimpleMultiThreadedTests
+Wasm.Build.Tests.TestAppScenarios.DebugLevelTests

src/mono/browser/runtime/cwraps.ts

@@ -166,7 +166,7 @@ export interface t_Cwraps {
  mono_wasm_load_icu_data(offset: VoidPtr): number;
  mono_wasm_add_assembly(name: string, data: VoidPtr, size: number): number;
  mono_wasm_add_satellite_assembly(name: string, culture: string, data: VoidPtr, size: number): void;
- mono_wasm_load_runtime(unused: string, debugLevel: number): void;
+ mono_wasm_load_runtime(debugLevel: number): void;
  mono_wasm_change_debugger_log_level(value: number): void;
 
  mono_wasm_assembly_load(name: string): MonoAssembly;

src/mono/browser/runtime/driver.c

@@ -181,7 +181,7 @@ cleanup_runtime_config (MonovmRuntimeConfigArguments *args, void *user_data)
 }
 
 EMSCRIPTEN_KEEPALIVE void
-mono_wasm_load_runtime (const char *unused, int debug_level)
+mono_wasm_load_runtime (int debug_level)
 {
 	const char *interp_opts = "";
 

src/mono/browser/runtime/lazyLoading.ts

@@ -26,7 +26,7 @@ export async function loadLazyAssembly(assemblyNameToLoad: string): Promise<bool
  }
 
  const pdbNameToLoad = changeExtension(dllAsset.name, ".pdb");
- const shouldLoadPdb = loaderHelpers.hasDebuggingEnabled(loaderHelpers.config) && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
+ const shouldLoadPdb = loaderHelpers.config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
 
  const dllBytesPromise = loaderHelpers.retrieve_asset_download(dllAsset);
 

src/mono/browser/runtime/loader/assets.ts

@@ -266,7 +266,7 @@ export function prepareAssets() {
  }
  }
 
- if (config.debugLevel != 0 && resources.pdb) {
+ if (config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && resources.pdb) {
  for (const name in resources.pdb) {
  assetsToLoad.push({
  name,

src/mono/browser/runtime/loader/config.ts

@@ -197,13 +197,6 @@ export function normalizeConfig() {
  config.environmentVariables["MONO_SLEEP_ABORT_LIMIT"] = "5000";
  }
 
- // Default values (when WasmDebugLevel is not set)
- // - Build (debug) => debugBuild=true & debugLevel=-1 => -1
- // - Build (release) => debugBuild=true & debugLevel=0 => 0
- // - Publish (debug) => debugBuild=false & debugLevel=-1 => 0
- // - Publish (release) => debugBuild=false & debugLevel=0 => 0
- config.debugLevel = hasDebuggingEnabled(config) ? config.debugLevel : 0;
-
  if (BuildConfiguration === "Debug" && config.diagnosticTracing === undefined) {
  config.diagnosticTracing = true;
  }
@@ -264,14 +257,13 @@ export async function mono_wasm_load_config(module: DotnetModuleInternal): Promi
  }
 }
 
-export function hasDebuggingEnabled(config: MonoConfigInternal): boolean {
+export function isDebuggingSupported(): boolean {
  // Copied from blazor MonoDebugger.ts/attachDebuggerHotkey
  if (!globalThis.navigator) {
  return false;
  }
 
- const hasReferencedPdbs = !!config.resources!.pdb;
- return (hasReferencedPdbs || config.debugLevel != 0) && (loaderHelpers.isChromium || loaderHelpers.isFirefox);
+ return loaderHelpers.isChromium || loaderHelpers.isFirefox;
 }
 
 async function loadBootConfig(module: DotnetModuleInternal): Promise<void> {

src/mono/browser/runtime/loader/globals.ts

@@ -15,7 +15,7 @@ import { assertIsControllablePromise, createPromiseController, getPromiseControl
 import { mono_download_assets, resolve_single_asset_path, retrieve_asset_download } from "./assets";
 import { mono_log_error, set_thread_prefix, setup_proxy_console } from "./logging";
 import { invokeLibraryInitializers } from "./libraryInitializers";
-import { deep_merge_config, hasDebuggingEnabled } from "./config";
+import { deep_merge_config, isDebuggingSupported } from "./config";
 import { logDownloadStatsToConsole, purgeUnusedCacheEntriesAsync } from "./assetsCache";
 
 // if we are the first script loaded in the web worker, we are expected to become the sidecar
@@ -118,9 +118,9 @@ export function setLoaderGlobals(
  purgeUnusedCacheEntriesAsync,
  installUnhandledErrorHandler,
 
- hasDebuggingEnabled,
  retrieve_asset_download,
  invokeLibraryInitializers,
+ isDebuggingSupported,
 
  // from wasm-feature-detect npm package
  exceptions,

src/mono/browser/runtime/startup.ts

@@ -515,7 +515,7 @@ async function start_runtime() {
  if (runtimeHelpers.config.browserProfilerOptions)
  mono_wasm_init_browser_profiler(runtimeHelpers.config.browserProfilerOptions);
 
- mono_wasm_load_runtime("unused", runtimeHelpers.config.debugLevel);
+ mono_wasm_load_runtime();
 
  if (runtimeHelpers.config.interpreterPgo)
  setTimeout(maybeSaveInterpPgoTable, (runtimeHelpers.config.interpreterPgoSaveDelay || 15) * 1000);
@@ -534,17 +534,21 @@ async function maybeSaveInterpPgoTable() {
  await interp_pgo_save_data();
 }
 
-export function mono_wasm_load_runtime(unused?: string, debugLevel?: number): void {
+export function mono_wasm_load_runtime(): void {
  mono_log_debug("mono_wasm_load_runtime");
  try {
  const mark = startMeasure();
+ let debugLevel = runtimeHelpers.config.debugLevel;
  if (debugLevel == undefined) {
  debugLevel = 0;
  if (runtimeHelpers.config.debugLevel) {
  debugLevel = 0 + debugLevel;
  }
  }
- cwraps.mono_wasm_load_runtime(unused || "unused", debugLevel);
+ if (!loaderHelpers.isDebuggingSupported() || !runtimeHelpers.config.resources!.pdb) {
+ debugLevel = 0;
+ }
+ cwraps.mono_wasm_load_runtime(debugLevel);
  endMeasure(mark, MeasuredBlock.loadRuntime);
 
  } catch (err: any) {

src/mono/browser/runtime/types/internal.ts

@@ -155,7 +155,6 @@ export type LoaderHelpers = {
  out(message: string): void;
  err(message: string): void;
 
- hasDebuggingEnabled(config: MonoConfig): boolean,
  retrieve_asset_download(asset: AssetEntry): Promise<ArrayBuffer>;
  onDownloadResourceProgress?: (resourcesLoaded: number, totalResources: number) => void;
  logDownloadStatsToConsole: () => void;
@@ -166,6 +165,7 @@ export type LoaderHelpers = {
  invokeLibraryInitializers: (functionName: string, args: any[]) => Promise<void>,
  libraryInitializers?: { scriptName: string, exports: any }[];
 
+ isDebuggingSupported(): boolean,
  isChromium: boolean,
  isFirefox: boolean
 

src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

@@ -203,9 +203,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  <_BlazorWebAssemblyStartupMemoryCache>$(BlazorWebAssemblyStartupMemoryCache)</_BlazorWebAssemblyStartupMemoryCache>
  <_BlazorWebAssemblyJiterpreter>$(BlazorWebAssemblyJiterpreter)</_BlazorWebAssemblyJiterpreter>
  <_BlazorWebAssemblyRuntimeOptions>$(BlazorWebAssemblyRuntimeOptions)</_BlazorWebAssemblyRuntimeOptions>
- <_WasmDebugLevel>$(WasmDebugLevel)</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == ''">0</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == '0' and ('$(DebuggerSupport)' == 'true' or '$(Configuration)' == 'Debug')">-1</_WasmDebugLevel>
 
  <!-- Workaround for https://github.com/dotnet/sdk/issues/12114-->
  <PublishDir Condition="'$(AppendRuntimeIdentifierToOutputPath)' != 'true' AND '$(PublishDir)' == '$(OutputPath)$(RuntimeIdentifier)\$(PublishDirName)\'">$(OutputPath)$(PublishDirName)\</PublishDir>
@@ -364,7 +361,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmOutputWithHash)"
  DebugBuild="true"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="false"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(_WasmBuildBootJsonPath)"
@@ -380,7 +377,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="false" />
 
  <ItemGroup>
  <FileWrites Include="$(_WasmBuildBootJsonPath)" />
@@ -533,7 +531,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  </Target>
 
  <Target Name="GeneratePublishWasmBootJson" DependsOnTargets="$(GeneratePublishWasmBootJsonDependsOn)">
-
  <ItemGroup>
  <_WasmPublishAsset
  Include="@(StaticWebAsset)"
@@ -549,7 +546,6 @@ Copyright (c) .NET Foundation. All rights reserved.
 
  <!-- We remove the extensions since they are added to the list of static web assets but we need to compute the target path for them -->
  <_WasmPublishAsset Remove="@(_BlazorExtensionsCandidatesForPublish)" />
-
  </ItemGroup>
 
  <ComputeStaticWebAssetsTargetPaths
@@ -568,7 +564,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmPublishBootResourceWithHash)"
  DebugBuild="false"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="$(PublishTrimmed)"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(IntermediateOutputPath)blazor.publish.boot.json"
@@ -584,7 +580,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="true" />
 
  <ItemGroup>
  <FileWrites Include="$(IntermediateOutputPath)blazor.publish.boot.json" />

src/mono/wasi/build/WasiApp.targets

@@ -66,6 +66,11 @@
  Outputs="$(WasmAppDir)\.stamp"
  Condition="'$(WasmGenerateAppBundle)' == 'true'">
 
+ <PropertyGroup>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(CopyOutputSymbolsToPublishDirectory)' == 'true' and '$(_IsPublishing)' == 'true'">true</_WasmOutputSymbolsToAppBundle>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(_WasmOutputSymbolsToAppBundle)' == ''">false</_WasmOutputSymbolsToAppBundle>
+ </PropertyGroup>
+
  <WasiAppBuilder
  AppDir="$(WasmAppDir)"
  Assemblies="@(_WasmAssembliesInternal)"
@@ -80,7 +85,7 @@
  FilesToIncludeInFileSystem="@(WasmFilesToIncludeInFileSystem)"
  ExtraFilesToDeploy="@(WasmExtraFilesToDeploy)"
  NativeAssets="@(WasmNativeAsset)"
-DebugLevel="$(WasmDebugLevel)"
+OutputSymbolsToAppBundle="$(_WasmOutputSymbolsToAppBundle)"
  RuntimeConfigJsonPath="$(_WasmRuntimeConfigFilePath)"
  />
  </Target>

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorWasmTestBase.cs

@@ -63,7 +63,9 @@ public string CreateBlazorWasmTemplateProject(string id)
  (CommandResult res, string logPath) = BlazorBuildInternal(options.Id, options.Config, publish: false, setWasmDevel: false, expectSuccess: options.ExpectSuccess, extraArgs);
 
  if (options.ExpectSuccess && options.AssertAppBundle)
+ {
  AssertBundle(res.Output, options with { IsPublish = false });
+ }
 
  return (res, logPath);
  }
@@ -77,6 +79,10 @@ public string CreateBlazorWasmTemplateProject(string id)
 
  if (options.ExpectSuccess && options.AssertAppBundle)
  {
+ // Because we do relink in Release publish by default
+ if (options.Config == "Release")
+ options = options with { ExpectedFileType = NativeFilesType.Relinked };
+
  AssertBundle(res.Output, options with { IsPublish = true });
  }
 

src/mono/wasm/Wasm.Build.Tests/Templates/InterpPgoTests.cs

@@ -34,7 +34,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
 
  string id = $"browser_{config}_{GetRandomId()}";
  _testOutput.WriteLine("/// Creating project");
- string projectFile = CreateWasmTemplateProject(id, "wasmbrowser");
+ string projectFile = CreateWasmTemplateProject(id, "wasmbrowser", extraProperties: "<WasmDebugLevel>0</WasmDebugLevel>");
 
  _testOutput.WriteLine("/// Updating JS");
  UpdateBrowserMainJs((js) => {
@@ -53,6 +53,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
  // then call INTERNAL.interp_pgo_save_data() to save the interp PGO table
  js = js.Replace(
  "const text = exports.MyClass.Greeting();",
+ "console.log(`WASM debug level ${getConfig().debugLevel}`);\n" + 
  "let text = '';\n" +
  $"for (let i = 0; i < {iterationCount}; i++) {{ text = exports.MyClass.Greeting(); }};\n" +
  "await INTERNAL.interp_pgo_save_data();"

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppSettingsTests.cs

@@ -28,7 +28,7 @@ public async Task LoadAppSettingsBasedOnApplicationEnvironment(string applicatio
  CopyTestAsset("WasmBasicTestApp", "AppSettingsTests");
  PublishProject("Debug");
 
- var result = await RunSdkStyleApp(new(
+ var result = await RunSdkStyleAppForPublish(new(
  Configuration: "Debug",
  TestScenario: "AppSettingsTest",
  BrowserQueryString: new Dictionary<string, string> { ["applicationEnvironment"] = applicationEnvironment }

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppTestBase.cs

@@ -36,23 +36,29 @@ protected void CopyTestAsset(string assetName, string generatedProjectNamePrefix
  _projectDir = Path.Combine(_projectDir!, "App");
  }
 
- protected void BuildProject(string configuration)
+ protected void BuildProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
- protected void PublishProject(string configuration)
+ protected void PublishProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
  protected ToolCommand CreateDotNetCommand() => new DotNetCommand(s_buildEnv, _testOutput)
  .WithWorkingDirectory(_projectDir!)
  .WithEnvironmentVariable("NUGET_PACKAGES", _nugetPackagesDir);
 
- protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
+ protected Task<RunResult> RunSdkStyleAppForBuild(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.DotnetRun);
+
+ protected Task<RunResult> RunSdkStyleAppForPublish(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.WebServer);
+
+ private async Task<RunResult> RunSdkStyleApp(RunOptions options, BlazorRunHost host = BlazorRunHost.DotnetRun)
  {
  string queryString = "?test=" + options.TestScenario;
  if (options.BrowserQueryString != null)
@@ -67,9 +73,10 @@ protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
  CheckCounter: false,
  Config: options.Configuration,
  OnConsoleMessage: OnConsoleMessage,
- QueryString: queryString);
+ QueryString: queryString,
+ Host: host);
 
- await BlazorRunForBuildWithDotnetRun(blazorRunOptions);
+ await BlazorRunTest(blazorRunOptions);
 
  void OnConsoleMessage(IConsoleMessage msg)
  {

eng/testing/scenarios/BuildWasmAppsJobsList.txt

@@ -44,3 +44,4 @@ Wasm.Build.Tests.WasmTemplateTests
 Wasm.Build.Tests.WorkloadTests
 Wasm.Build.Tests.TestAppScenarios.DownloadResourceProgressTests
 Wasm.Build.Tests.MT.Blazor.SimpleMultiThreadedTests
+Wasm.Build.Tests.TestAppScenarios.DebugLevelTests

src/mono/browser/runtime/cwraps.ts

@@ -166,7 +166,7 @@ export interface t_Cwraps {
  mono_wasm_load_icu_data(offset: VoidPtr): number;
  mono_wasm_add_assembly(name: string, data: VoidPtr, size: number): number;
  mono_wasm_add_satellite_assembly(name: string, culture: string, data: VoidPtr, size: number): void;
- mono_wasm_load_runtime(unused: string, debugLevel: number): void;
+ mono_wasm_load_runtime(debugLevel: number): void;
  mono_wasm_change_debugger_log_level(value: number): void;
 
  mono_wasm_assembly_load(name: string): MonoAssembly;

src/mono/browser/runtime/driver.c

@@ -181,7 +181,7 @@ cleanup_runtime_config (MonovmRuntimeConfigArguments *args, void *user_data)
 }
 
 EMSCRIPTEN_KEEPALIVE void
-mono_wasm_load_runtime (const char *unused, int debug_level)
+mono_wasm_load_runtime (int debug_level)
 {
 	const char *interp_opts = "";
 

src/mono/browser/runtime/lazyLoading.ts

@@ -26,7 +26,7 @@ export async function loadLazyAssembly(assemblyNameToLoad: string): Promise<bool
  }
 
  const pdbNameToLoad = changeExtension(dllAsset.name, ".pdb");
- const shouldLoadPdb = loaderHelpers.hasDebuggingEnabled(loaderHelpers.config) && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
+ const shouldLoadPdb = loaderHelpers.config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
 
  const dllBytesPromise = loaderHelpers.retrieve_asset_download(dllAsset);
 

src/mono/browser/runtime/loader/assets.ts

@@ -266,7 +266,7 @@ export function prepareAssets() {
  }
  }
 
- if (config.debugLevel != 0 && resources.pdb) {
+ if (config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && resources.pdb) {
  for (const name in resources.pdb) {
  assetsToLoad.push({
  name,

src/mono/browser/runtime/loader/config.ts

@@ -197,13 +197,6 @@ export function normalizeConfig() {
  config.environmentVariables["MONO_SLEEP_ABORT_LIMIT"] = "5000";
  }
 
- // Default values (when WasmDebugLevel is not set)
- // - Build (debug) => debugBuild=true & debugLevel=-1 => -1
- // - Build (release) => debugBuild=true & debugLevel=0 => 0
- // - Publish (debug) => debugBuild=false & debugLevel=-1 => 0
- // - Publish (release) => debugBuild=false & debugLevel=0 => 0
- config.debugLevel = hasDebuggingEnabled(config) ? config.debugLevel : 0;
-
  if (BuildConfiguration === "Debug" && config.diagnosticTracing === undefined) {
  config.diagnosticTracing = true;
  }
@@ -264,14 +257,13 @@ export async function mono_wasm_load_config(module: DotnetModuleInternal): Promi
  }
 }
 
-export function hasDebuggingEnabled(config: MonoConfigInternal): boolean {
+export function isDebuggingSupported(): boolean {
  // Copied from blazor MonoDebugger.ts/attachDebuggerHotkey
  if (!globalThis.navigator) {
  return false;
  }
 
- const hasReferencedPdbs = !!config.resources!.pdb;
- return (hasReferencedPdbs || config.debugLevel != 0) && (loaderHelpers.isChromium || loaderHelpers.isFirefox);
+ return loaderHelpers.isChromium || loaderHelpers.isFirefox;
 }
 
 async function loadBootConfig(module: DotnetModuleInternal): Promise<void> {

src/mono/browser/runtime/loader/globals.ts

@@ -15,7 +15,7 @@ import { assertIsControllablePromise, createPromiseController, getPromiseControl
 import { mono_download_assets, resolve_single_asset_path, retrieve_asset_download } from "./assets";
 import { mono_log_error, set_thread_prefix, setup_proxy_console } from "./logging";
 import { invokeLibraryInitializers } from "./libraryInitializers";
-import { deep_merge_config, hasDebuggingEnabled } from "./config";
+import { deep_merge_config, isDebuggingSupported } from "./config";
 import { logDownloadStatsToConsole, purgeUnusedCacheEntriesAsync } from "./assetsCache";
 
 // if we are the first script loaded in the web worker, we are expected to become the sidecar
@@ -118,9 +118,9 @@ export function setLoaderGlobals(
  purgeUnusedCacheEntriesAsync,
  installUnhandledErrorHandler,
 
- hasDebuggingEnabled,
  retrieve_asset_download,
  invokeLibraryInitializers,
+ isDebuggingSupported,
 
  // from wasm-feature-detect npm package
  exceptions,

src/mono/browser/runtime/startup.ts

@@ -515,7 +515,7 @@ async function start_runtime() {
  if (runtimeHelpers.config.browserProfilerOptions)
  mono_wasm_init_browser_profiler(runtimeHelpers.config.browserProfilerOptions);
 
- mono_wasm_load_runtime("unused", runtimeHelpers.config.debugLevel);
+ mono_wasm_load_runtime();
 
  if (runtimeHelpers.config.interpreterPgo)
  setTimeout(maybeSaveInterpPgoTable, (runtimeHelpers.config.interpreterPgoSaveDelay || 15) * 1000);
@@ -534,17 +534,21 @@ async function maybeSaveInterpPgoTable() {
  await interp_pgo_save_data();
 }
 
-export function mono_wasm_load_runtime(unused?: string, debugLevel?: number): void {
+export function mono_wasm_load_runtime(): void {
  mono_log_debug("mono_wasm_load_runtime");
  try {
  const mark = startMeasure();
+ let debugLevel = runtimeHelpers.config.debugLevel;
  if (debugLevel == undefined) {
  debugLevel = 0;
  if (runtimeHelpers.config.debugLevel) {
  debugLevel = 0 + debugLevel;
  }
  }
- cwraps.mono_wasm_load_runtime(unused || "unused", debugLevel);
+ if (!loaderHelpers.isDebuggingSupported() || !runtimeHelpers.config.resources!.pdb) {
+ debugLevel = 0;
+ }
+ cwraps.mono_wasm_load_runtime(debugLevel);
  endMeasure(mark, MeasuredBlock.loadRuntime);
 
  } catch (err: any) {

src/mono/browser/runtime/types/internal.ts

@@ -155,7 +155,6 @@ export type LoaderHelpers = {
  out(message: string): void;
  err(message: string): void;
 
- hasDebuggingEnabled(config: MonoConfig): boolean,
  retrieve_asset_download(asset: AssetEntry): Promise<ArrayBuffer>;
  onDownloadResourceProgress?: (resourcesLoaded: number, totalResources: number) => void;
  logDownloadStatsToConsole: () => void;
@@ -166,6 +165,7 @@ export type LoaderHelpers = {
  invokeLibraryInitializers: (functionName: string, args: any[]) => Promise<void>,
  libraryInitializers?: { scriptName: string, exports: any }[];
 
+ isDebuggingSupported(): boolean,
  isChromium: boolean,
  isFirefox: boolean
 

src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

@@ -203,9 +203,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  <_BlazorWebAssemblyStartupMemoryCache>$(BlazorWebAssemblyStartupMemoryCache)</_BlazorWebAssemblyStartupMemoryCache>
  <_BlazorWebAssemblyJiterpreter>$(BlazorWebAssemblyJiterpreter)</_BlazorWebAssemblyJiterpreter>
  <_BlazorWebAssemblyRuntimeOptions>$(BlazorWebAssemblyRuntimeOptions)</_BlazorWebAssemblyRuntimeOptions>
- <_WasmDebugLevel>$(WasmDebugLevel)</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == ''">0</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == '0' and ('$(DebuggerSupport)' == 'true' or '$(Configuration)' == 'Debug')">-1</_WasmDebugLevel>
 
  <!-- Workaround for https://github.com/dotnet/sdk/issues/12114-->
  <PublishDir Condition="'$(AppendRuntimeIdentifierToOutputPath)' != 'true' AND '$(PublishDir)' == '$(OutputPath)$(RuntimeIdentifier)\$(PublishDirName)\'">$(OutputPath)$(PublishDirName)\</PublishDir>
@@ -364,7 +361,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmOutputWithHash)"
  DebugBuild="true"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="false"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(_WasmBuildBootJsonPath)"
@@ -380,7 +377,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="false" />
 
  <ItemGroup>
  <FileWrites Include="$(_WasmBuildBootJsonPath)" />
@@ -533,7 +531,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  </Target>
 
  <Target Name="GeneratePublishWasmBootJson" DependsOnTargets="$(GeneratePublishWasmBootJsonDependsOn)">
-
  <ItemGroup>
  <_WasmPublishAsset
  Include="@(StaticWebAsset)"
@@ -549,7 +546,6 @@ Copyright (c) .NET Foundation. All rights reserved.
 
  <!-- We remove the extensions since they are added to the list of static web assets but we need to compute the target path for them -->
  <_WasmPublishAsset Remove="@(_BlazorExtensionsCandidatesForPublish)" />
-
  </ItemGroup>
 
  <ComputeStaticWebAssetsTargetPaths
@@ -568,7 +564,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmPublishBootResourceWithHash)"
  DebugBuild="false"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="$(PublishTrimmed)"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(IntermediateOutputPath)blazor.publish.boot.json"
@@ -584,7 +580,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="true" />
 
  <ItemGroup>
  <FileWrites Include="$(IntermediateOutputPath)blazor.publish.boot.json" />

src/mono/wasi/build/WasiApp.targets

@@ -66,6 +66,11 @@
  Outputs="$(WasmAppDir)\.stamp"
  Condition="'$(WasmGenerateAppBundle)' == 'true'">
 
+ <PropertyGroup>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(CopyOutputSymbolsToPublishDirectory)' == 'true' and '$(_IsPublishing)' == 'true'">true</_WasmOutputSymbolsToAppBundle>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(_WasmOutputSymbolsToAppBundle)' == ''">false</_WasmOutputSymbolsToAppBundle>
+ </PropertyGroup>
+
  <WasiAppBuilder
  AppDir="$(WasmAppDir)"
  Assemblies="@(_WasmAssembliesInternal)"
@@ -80,7 +85,7 @@
  FilesToIncludeInFileSystem="@(WasmFilesToIncludeInFileSystem)"
  ExtraFilesToDeploy="@(WasmExtraFilesToDeploy)"
  NativeAssets="@(WasmNativeAsset)"
-DebugLevel="$(WasmDebugLevel)"
+OutputSymbolsToAppBundle="$(_WasmOutputSymbolsToAppBundle)"
  RuntimeConfigJsonPath="$(_WasmRuntimeConfigFilePath)"
  />
  </Target>

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorWasmTestBase.cs

@@ -63,7 +63,9 @@ public string CreateBlazorWasmTemplateProject(string id)
  (CommandResult res, string logPath) = BlazorBuildInternal(options.Id, options.Config, publish: false, setWasmDevel: false, expectSuccess: options.ExpectSuccess, extraArgs);
 
  if (options.ExpectSuccess && options.AssertAppBundle)
+ {
  AssertBundle(res.Output, options with { IsPublish = false });
+ }
 
  return (res, logPath);
  }
@@ -77,6 +79,10 @@ public string CreateBlazorWasmTemplateProject(string id)
 
  if (options.ExpectSuccess && options.AssertAppBundle)
  {
+ // Because we do relink in Release publish by default
+ if (options.Config == "Release")
+ options = options with { ExpectedFileType = NativeFilesType.Relinked };
+
  AssertBundle(res.Output, options with { IsPublish = true });
  }
 

src/mono/wasm/Wasm.Build.Tests/Templates/InterpPgoTests.cs

@@ -34,7 +34,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
 
  string id = $"browser_{config}_{GetRandomId()}";
  _testOutput.WriteLine("/// Creating project");
- string projectFile = CreateWasmTemplateProject(id, "wasmbrowser");
+ string projectFile = CreateWasmTemplateProject(id, "wasmbrowser", extraProperties: "<WasmDebugLevel>0</WasmDebugLevel>");
 
  _testOutput.WriteLine("/// Updating JS");
  UpdateBrowserMainJs((js) => {
@@ -53,6 +53,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
  // then call INTERNAL.interp_pgo_save_data() to save the interp PGO table
  js = js.Replace(
  "const text = exports.MyClass.Greeting();",
+ "console.log(`WASM debug level ${getConfig().debugLevel}`);\n" + 
  "let text = '';\n" +
  $"for (let i = 0; i < {iterationCount}; i++) {{ text = exports.MyClass.Greeting(); }};\n" +
  "await INTERNAL.interp_pgo_save_data();"

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppSettingsTests.cs

@@ -28,7 +28,7 @@ public async Task LoadAppSettingsBasedOnApplicationEnvironment(string applicatio
  CopyTestAsset("WasmBasicTestApp", "AppSettingsTests");
  PublishProject("Debug");
 
- var result = await RunSdkStyleApp(new(
+ var result = await RunSdkStyleAppForPublish(new(
  Configuration: "Debug",
  TestScenario: "AppSettingsTest",
  BrowserQueryString: new Dictionary<string, string> { ["applicationEnvironment"] = applicationEnvironment }

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppTestBase.cs

@@ -36,23 +36,29 @@ protected void CopyTestAsset(string assetName, string generatedProjectNamePrefix
  _projectDir = Path.Combine(_projectDir!, "App");
  }
 
- protected void BuildProject(string configuration)
+ protected void BuildProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
- protected void PublishProject(string configuration)
+ protected void PublishProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
  protected ToolCommand CreateDotNetCommand() => new DotNetCommand(s_buildEnv, _testOutput)
  .WithWorkingDirectory(_projectDir!)
  .WithEnvironmentVariable("NUGET_PACKAGES", _nugetPackagesDir);
 
- protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
+ protected Task<RunResult> RunSdkStyleAppForBuild(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.DotnetRun);
+
+ protected Task<RunResult> RunSdkStyleAppForPublish(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.WebServer);
+
+ private async Task<RunResult> RunSdkStyleApp(RunOptions options, BlazorRunHost host = BlazorRunHost.DotnetRun)
  {
  string queryString = "?test=" + options.TestScenario;
  if (options.BrowserQueryString != null)
@@ -67,9 +73,10 @@ protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
  CheckCounter: false,
  Config: options.Configuration,
  OnConsoleMessage: OnConsoleMessage,
- QueryString: queryString);
+ QueryString: queryString,
+ Host: host);
 
- await BlazorRunForBuildWithDotnetRun(blazorRunOptions);
+ await BlazorRunTest(blazorRunOptions);
 
  void OnConsoleMessage(IConsoleMessage msg)
  {

eng/testing/scenarios/BuildWasmAppsJobsList.txt

@@ -44,3 +44,4 @@ Wasm.Build.Tests.WasmTemplateTests
 Wasm.Build.Tests.WorkloadTests
 Wasm.Build.Tests.TestAppScenarios.DownloadResourceProgressTests
 Wasm.Build.Tests.MT.Blazor.SimpleMultiThreadedTests
+Wasm.Build.Tests.TestAppScenarios.DebugLevelTests

src/mono/browser/runtime/cwraps.ts

@@ -166,7 +166,7 @@ export interface t_Cwraps {
  mono_wasm_load_icu_data(offset: VoidPtr): number;
  mono_wasm_add_assembly(name: string, data: VoidPtr, size: number): number;
  mono_wasm_add_satellite_assembly(name: string, culture: string, data: VoidPtr, size: number): void;
- mono_wasm_load_runtime(unused: string, debugLevel: number): void;
+ mono_wasm_load_runtime(debugLevel: number): void;
  mono_wasm_change_debugger_log_level(value: number): void;
 
  mono_wasm_assembly_load(name: string): MonoAssembly;

src/mono/browser/runtime/driver.c

@@ -181,7 +181,7 @@ cleanup_runtime_config (MonovmRuntimeConfigArguments *args, void *user_data)
 }
 
 EMSCRIPTEN_KEEPALIVE void
-mono_wasm_load_runtime (const char *unused, int debug_level)
+mono_wasm_load_runtime (int debug_level)
 {
 	const char *interp_opts = "";
 

src/mono/browser/runtime/lazyLoading.ts

@@ -26,7 +26,7 @@ export async function loadLazyAssembly(assemblyNameToLoad: string): Promise<bool
  }
 
  const pdbNameToLoad = changeExtension(dllAsset.name, ".pdb");
- const shouldLoadPdb = loaderHelpers.hasDebuggingEnabled(loaderHelpers.config) && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
+ const shouldLoadPdb = loaderHelpers.config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && Object.prototype.hasOwnProperty.call(lazyAssemblies, pdbNameToLoad);
 
  const dllBytesPromise = loaderHelpers.retrieve_asset_download(dllAsset);
 

src/mono/browser/runtime/loader/assets.ts

@@ -266,7 +266,7 @@ export function prepareAssets() {
  }
  }
 
- if (config.debugLevel != 0 && resources.pdb) {
+ if (config.debugLevel != 0 && loaderHelpers.isDebuggingSupported() && resources.pdb) {
  for (const name in resources.pdb) {
  assetsToLoad.push({
  name,

src/mono/browser/runtime/loader/config.ts

@@ -197,13 +197,6 @@ export function normalizeConfig() {
  config.environmentVariables["MONO_SLEEP_ABORT_LIMIT"] = "5000";
  }
 
- // Default values (when WasmDebugLevel is not set)
- // - Build (debug) => debugBuild=true & debugLevel=-1 => -1
- // - Build (release) => debugBuild=true & debugLevel=0 => 0
- // - Publish (debug) => debugBuild=false & debugLevel=-1 => 0
- // - Publish (release) => debugBuild=false & debugLevel=0 => 0
- config.debugLevel = hasDebuggingEnabled(config) ? config.debugLevel : 0;
-
  if (BuildConfiguration === "Debug" && config.diagnosticTracing === undefined) {
  config.diagnosticTracing = true;
  }
@@ -264,14 +257,13 @@ export async function mono_wasm_load_config(module: DotnetModuleInternal): Promi
  }
 }
 
-export function hasDebuggingEnabled(config: MonoConfigInternal): boolean {
+export function isDebuggingSupported(): boolean {
  // Copied from blazor MonoDebugger.ts/attachDebuggerHotkey
  if (!globalThis.navigator) {
  return false;
  }
 
- const hasReferencedPdbs = !!config.resources!.pdb;
- return (hasReferencedPdbs || config.debugLevel != 0) && (loaderHelpers.isChromium || loaderHelpers.isFirefox);
+ return loaderHelpers.isChromium || loaderHelpers.isFirefox;
 }
 
 async function loadBootConfig(module: DotnetModuleInternal): Promise<void> {

src/mono/browser/runtime/loader/globals.ts

@@ -15,7 +15,7 @@ import { assertIsControllablePromise, createPromiseController, getPromiseControl
 import { mono_download_assets, resolve_single_asset_path, retrieve_asset_download } from "./assets";
 import { mono_log_error, set_thread_prefix, setup_proxy_console } from "./logging";
 import { invokeLibraryInitializers } from "./libraryInitializers";
-import { deep_merge_config, hasDebuggingEnabled } from "./config";
+import { deep_merge_config, isDebuggingSupported } from "./config";
 import { logDownloadStatsToConsole, purgeUnusedCacheEntriesAsync } from "./assetsCache";
 
 // if we are the first script loaded in the web worker, we are expected to become the sidecar
@@ -118,9 +118,9 @@ export function setLoaderGlobals(
  purgeUnusedCacheEntriesAsync,
  installUnhandledErrorHandler,
 
- hasDebuggingEnabled,
  retrieve_asset_download,
  invokeLibraryInitializers,
+ isDebuggingSupported,
 
  // from wasm-feature-detect npm package
  exceptions,

src/mono/browser/runtime/startup.ts

@@ -515,7 +515,7 @@ async function start_runtime() {
  if (runtimeHelpers.config.browserProfilerOptions)
  mono_wasm_init_browser_profiler(runtimeHelpers.config.browserProfilerOptions);
 
- mono_wasm_load_runtime("unused", runtimeHelpers.config.debugLevel);
+ mono_wasm_load_runtime();
 
  if (runtimeHelpers.config.interpreterPgo)
  setTimeout(maybeSaveInterpPgoTable, (runtimeHelpers.config.interpreterPgoSaveDelay || 15) * 1000);
@@ -534,17 +534,21 @@ async function maybeSaveInterpPgoTable() {
  await interp_pgo_save_data();
 }
 
-export function mono_wasm_load_runtime(unused?: string, debugLevel?: number): void {
+export function mono_wasm_load_runtime(): void {
  mono_log_debug("mono_wasm_load_runtime");
  try {
  const mark = startMeasure();
+ let debugLevel = runtimeHelpers.config.debugLevel;
  if (debugLevel == undefined) {
  debugLevel = 0;
  if (runtimeHelpers.config.debugLevel) {
  debugLevel = 0 + debugLevel;
  }
  }
- cwraps.mono_wasm_load_runtime(unused || "unused", debugLevel);
+ if (!loaderHelpers.isDebuggingSupported() || !runtimeHelpers.config.resources!.pdb) {
+ debugLevel = 0;
+ }
+ cwraps.mono_wasm_load_runtime(debugLevel);
  endMeasure(mark, MeasuredBlock.loadRuntime);
 
  } catch (err: any) {

src/mono/browser/runtime/types/internal.ts

@@ -155,7 +155,6 @@ export type LoaderHelpers = {
  out(message: string): void;
  err(message: string): void;
 
- hasDebuggingEnabled(config: MonoConfig): boolean,
  retrieve_asset_download(asset: AssetEntry): Promise<ArrayBuffer>;
  onDownloadResourceProgress?: (resourcesLoaded: number, totalResources: number) => void;
  logDownloadStatsToConsole: () => void;
@@ -166,6 +165,7 @@ export type LoaderHelpers = {
  invokeLibraryInitializers: (functionName: string, args: any[]) => Promise<void>,
  libraryInitializers?: { scriptName: string, exports: any }[];
 
+ isDebuggingSupported(): boolean,
  isChromium: boolean,
  isFirefox: boolean
 

src/mono/nuget/Microsoft.NET.Sdk.WebAssembly.Pack/build/Microsoft.NET.Sdk.WebAssembly.Browser.targets

@@ -203,9 +203,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  <_BlazorWebAssemblyStartupMemoryCache>$(BlazorWebAssemblyStartupMemoryCache)</_BlazorWebAssemblyStartupMemoryCache>
  <_BlazorWebAssemblyJiterpreter>$(BlazorWebAssemblyJiterpreter)</_BlazorWebAssemblyJiterpreter>
  <_BlazorWebAssemblyRuntimeOptions>$(BlazorWebAssemblyRuntimeOptions)</_BlazorWebAssemblyRuntimeOptions>
- <_WasmDebugLevel>$(WasmDebugLevel)</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == ''">0</_WasmDebugLevel>
- <_WasmDebugLevel Condition="'$(_WasmDebugLevel)' == '0' and ('$(DebuggerSupport)' == 'true' or '$(Configuration)' == 'Debug')">-1</_WasmDebugLevel>
 
  <!-- Workaround for https://github.com/dotnet/sdk/issues/12114-->
  <PublishDir Condition="'$(AppendRuntimeIdentifierToOutputPath)' != 'true' AND '$(PublishDir)' == '$(OutputPath)$(RuntimeIdentifier)\$(PublishDirName)\'">$(OutputPath)$(PublishDirName)\</PublishDir>
@@ -364,7 +361,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmOutputWithHash)"
  DebugBuild="true"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="false"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(_WasmBuildBootJsonPath)"
@@ -380,7 +377,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="false" />
 
  <ItemGroup>
  <FileWrites Include="$(_WasmBuildBootJsonPath)" />
@@ -533,7 +531,6 @@ Copyright (c) .NET Foundation. All rights reserved.
  </Target>
 
  <Target Name="GeneratePublishWasmBootJson" DependsOnTargets="$(GeneratePublishWasmBootJsonDependsOn)">
-
  <ItemGroup>
  <_WasmPublishAsset
  Include="@(StaticWebAsset)"
@@ -549,7 +546,6 @@ Copyright (c) .NET Foundation. All rights reserved.
 
  <!-- We remove the extensions since they are added to the list of static web assets but we need to compute the target path for them -->
  <_WasmPublishAsset Remove="@(_BlazorExtensionsCandidatesForPublish)" />
-
  </ItemGroup>
 
  <ComputeStaticWebAssetsTargetPaths
@@ -568,7 +564,7 @@ Copyright (c) .NET Foundation. All rights reserved.
  AssemblyPath="@(IntermediateAssembly)"
  Resources="@(_WasmPublishBootResourceWithHash)"
  DebugBuild="false"
- DebugLevel="$(_WasmDebugLevel)"
+ DebugLevel="$(WasmDebugLevel)"
  LinkerEnabled="$(PublishTrimmed)"
  CacheBootResources="$(BlazorCacheBootResources)"
  OutputPath="$(IntermediateOutputPath)blazor.publish.boot.json"
@@ -584,7 +580,8 @@ Copyright (c) .NET Foundation. All rights reserved.
  Extensions="@(WasmBootConfigExtension)"
  TargetFrameworkVersion="$(TargetFrameworkVersion)"
  ModuleAfterConfigLoaded="@(WasmModuleAfterConfigLoaded)"
- ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)" />
+ ModuleAfterRuntimeReady="@(WasmModuleAfterRuntimeReady)"
+ IsPublish="true" />
 
  <ItemGroup>
  <FileWrites Include="$(IntermediateOutputPath)blazor.publish.boot.json" />

src/mono/wasi/build/WasiApp.targets

@@ -66,6 +66,11 @@
  Outputs="$(WasmAppDir)\.stamp"
  Condition="'$(WasmGenerateAppBundle)' == 'true'">
 
+ <PropertyGroup>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(CopyOutputSymbolsToPublishDirectory)' == 'true' and '$(_IsPublishing)' == 'true'">true</_WasmOutputSymbolsToAppBundle>
+ <_WasmOutputSymbolsToAppBundle Condition="'$(_WasmOutputSymbolsToAppBundle)' == ''">false</_WasmOutputSymbolsToAppBundle>
+ </PropertyGroup>
+
  <WasiAppBuilder
  AppDir="$(WasmAppDir)"
  Assemblies="@(_WasmAssembliesInternal)"
@@ -80,7 +85,7 @@
  FilesToIncludeInFileSystem="@(WasmFilesToIncludeInFileSystem)"
  ExtraFilesToDeploy="@(WasmExtraFilesToDeploy)"
  NativeAssets="@(WasmNativeAsset)"
-DebugLevel="$(WasmDebugLevel)"
+OutputSymbolsToAppBundle="$(_WasmOutputSymbolsToAppBundle)"
  RuntimeConfigJsonPath="$(_WasmRuntimeConfigFilePath)"
  />
  </Target>

src/mono/wasm/Wasm.Build.Tests/Blazor/BlazorWasmTestBase.cs

@@ -63,7 +63,9 @@ public string CreateBlazorWasmTemplateProject(string id)
  (CommandResult res, string logPath) = BlazorBuildInternal(options.Id, options.Config, publish: false, setWasmDevel: false, expectSuccess: options.ExpectSuccess, extraArgs);
 
  if (options.ExpectSuccess && options.AssertAppBundle)
+ {
  AssertBundle(res.Output, options with { IsPublish = false });
+ }
 
  return (res, logPath);
  }
@@ -77,6 +79,10 @@ public string CreateBlazorWasmTemplateProject(string id)
 
  if (options.ExpectSuccess && options.AssertAppBundle)
  {
+ // Because we do relink in Release publish by default
+ if (options.Config == "Release")
+ options = options with { ExpectedFileType = NativeFilesType.Relinked };
+
  AssertBundle(res.Output, options with { IsPublish = true });
  }
 

src/mono/wasm/Wasm.Build.Tests/Templates/InterpPgoTests.cs

@@ -34,7 +34,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
 
  string id = $"browser_{config}_{GetRandomId()}";
  _testOutput.WriteLine("/// Creating project");
- string projectFile = CreateWasmTemplateProject(id, "wasmbrowser");
+ string projectFile = CreateWasmTemplateProject(id, "wasmbrowser", extraProperties: "<WasmDebugLevel>0</WasmDebugLevel>");
 
  _testOutput.WriteLine("/// Updating JS");
  UpdateBrowserMainJs((js) => {
@@ -53,6 +53,7 @@ public async Task FirstRunGeneratesTableAndSecondRunLoadsIt(string config)
  // then call INTERNAL.interp_pgo_save_data() to save the interp PGO table
  js = js.Replace(
  "const text = exports.MyClass.Greeting();",
+ "console.log(`WASM debug level ${getConfig().debugLevel}`);\n" + 
  "let text = '';\n" +
  $"for (let i = 0; i < {iterationCount}; i++) {{ text = exports.MyClass.Greeting(); }};\n" +
  "await INTERNAL.interp_pgo_save_data();"

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppSettingsTests.cs

@@ -28,7 +28,7 @@ public async Task LoadAppSettingsBasedOnApplicationEnvironment(string applicatio
  CopyTestAsset("WasmBasicTestApp", "AppSettingsTests");
  PublishProject("Debug");
 
- var result = await RunSdkStyleApp(new(
+ var result = await RunSdkStyleAppForPublish(new(
  Configuration: "Debug",
  TestScenario: "AppSettingsTest",
  BrowserQueryString: new Dictionary<string, string> { ["applicationEnvironment"] = applicationEnvironment }

src/mono/wasm/Wasm.Build.Tests/TestAppScenarios/AppTestBase.cs

@@ -36,23 +36,29 @@ protected void CopyTestAsset(string assetName, string generatedProjectNamePrefix
  _projectDir = Path.Combine(_projectDir!, "App");
  }
 
- protected void BuildProject(string configuration)
+ protected void BuildProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorBuild(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
- protected void PublishProject(string configuration)
+ protected void PublishProject(string configuration, params string[] extraArgs)
  {
- (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration));
+ (CommandResult result, _) = BlazorPublish(new BlazorBuildOptions(Id, configuration), extraArgs);
  result.EnsureSuccessful();
  }
 
  protected ToolCommand CreateDotNetCommand() => new DotNetCommand(s_buildEnv, _testOutput)
  .WithWorkingDirectory(_projectDir!)
  .WithEnvironmentVariable("NUGET_PACKAGES", _nugetPackagesDir);
 
- protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
+ protected Task<RunResult> RunSdkStyleAppForBuild(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.DotnetRun);
+
+ protected Task<RunResult> RunSdkStyleAppForPublish(RunOptions options)
+ => RunSdkStyleApp(options, BlazorRunHost.WebServer);
+
+ private async Task<RunResult> RunSdkStyleApp(RunOptions options, BlazorRunHost host = BlazorRunHost.DotnetRun)
  {
  string queryString = "?test=" + options.TestScenario;
  if (options.BrowserQueryString != null)
@@ -67,9 +73,10 @@ protected async Task<RunResult> RunSdkStyleApp(RunOptions options)
  CheckCounter: false,
  Config: options.Configuration,
  OnConsoleMessage: OnConsoleMessage,
- QueryString: queryString);
+ QueryString: queryString,
+ Host: host);
 
- await BlazorRunForBuildWithDotnetRun(blazorRunOptions);
+ await BlazorRunTest(blazorRunOptions);
 
  void OnConsoleMessage(IConsoleMessage msg)
  {