Skip to content

[browser] Trim globalization code #93473

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Oct 26, 2023
Merged

[browser] Trim globalization code #93473

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
da239c3
Correct trimming errors.
ilonatommy Oct 13, 2023
90f5166
Feedback
ilonatommy Oct 16, 2023
37c3ad4
Fix WBT
ilonatommy Oct 18, 2023
7a41bf7
Allow trimming Settings class on Unix.
ilonatommy Oct 18, 2023
e65de59
Feature switches setting matter, not properties. Improve the approach…
ilonatommy Oct 18, 2023
c49e2ea
Users set properties, not features, so this is what we should test.
ilonatommy Oct 19, 2023
b8df428
Revert.
ilonatommy Oct 19, 2023
86263b2
`DebuggerSupport` is false by default
ilonatommy Oct 19, 2023
efd62a3
`EventSourceSupport` is trimmed when not enabled.
ilonatommy Oct 19, 2023
451708c
Fix `WasmTestOnBrowser` scenario lib tests.
ilonatommy Oct 19, 2023
8c3595b
Fixing another batch of lib failures.
ilonatommy Oct 19, 2023
2d26700
Fix `System.Runtime` tests + partial feedback.
ilonatommy Oct 19, 2023
10a7269
@radical's feedback
ilonatommy Oct 20, 2023
09936a4
Moving common properties to manifest.
ilonatommy Oct 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion src/libraries/System.Private.CoreLib/src/ILLink/ILLink.Substitutions.Browser.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,10 @@
<linker>
<assembly fullname="System.Private.CoreLib">
<type fullname="System.Globalization.GlobalizationMode">
<method signature="System.Boolean get_Hybrid()" body="stub" value="true" feature="System.Globalization.Hybrid" featurevalue="false" />
<method signature="System.Boolean get_Hybrid()" body="stub" value="false" feature="System.Globalization.Hybrid" featurevalue="false" />
</type>
<type fullname="System.Globalization.GlobalizationMode">
<method signature="System.Boolean get_Hybrid()" body="stub" value="true" feature="System.Globalization.Hybrid" featurevalue="true" />
</type>
</assembly>
</linker>
6 changes: 3 additions & 3 deletions src/mono/wasm/Wasm.Build.Tests/IcuTestsBase.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,9 +108,9 @@ protected void TestIcuShards(BuildArgs buildArgs, string shardName, string teste
bool dotnetWasmFromRuntimePack = !(buildArgs.AOT || buildArgs.Config == "Release");

buildArgs = buildArgs with { ProjectName = projectName };
string extraProperties = onlyPredefinedCultures ?
$"<WasmIcuDataFileName>{shardName}</WasmIcuDataFileName><PredefinedCulturesOnly>true</PredefinedCulturesOnly>" :
$"<WasmIcuDataFileName>{shardName}</WasmIcuDataFileName>";
string extraProperties = $"<WasmIcuDataFileName>{shardName}</WasmIcuDataFileName><UseSystemResourceKeys>false</UseSystemResourceKeys>";
if (onlyPredefinedCultures)
extraProperties = $"{extraProperties}<PredefinedCulturesOnly>true</PredefinedCulturesOnly>";
buildArgs = ExpandBuildArgs(buildArgs, extraProperties: extraProperties);

string programText = GetProgramText(testedLocales, onlyPredefinedCultures);
Expand Down
15 changes: 14 additions & 1 deletion src/mono/wasm/build/WasmApp.targets
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -154,10 +154,23 @@
<WasmStripILAfterAOT Condition="'$(WasmStripILAfterAOT)' == ''">false</WasmStripILAfterAOT>

<WasmRuntimeAssetsLocation Condition="'$(WasmRuntimeAssetsLocation)' == ''">_framework</WasmRuntimeAssetsLocation>
<BlazorCacheBootResources Condition="'$(BlazorCacheBootResources)' == ''">true</BlazorCacheBootResources>

<!-- Default values for build properties -->
<InvariantTimezone Condition="'$(InvariantTimezone)' == ''">false</InvariantTimezone>
</PropertyGroup>

<ItemGroup>
<!-- Runtime feature defaults to trim unnecessary code -->
<RuntimeHostConfigurationOption Condition="'$(HybridGlobalization)' == ''" Include="System.Globalization.Hybrid" Value="false"/>
<RuntimeHostConfigurationOption Condition="'$(InvariantGlobalization)' == ''" Include="System.Globalization.Invariant" Value="false" />
<RuntimeHostConfigurationOption Condition="'$(EventSourceSupport)' == ''" Include="System.Diagnostics.Tracing.EventSource.IsSupported" Value="false" />
<RuntimeHostConfigurationOption Condition="'$(UseSystemResourceKeys)' == ''" Include="System.Resources.UseSystemResourceKeys" Value="true" />
<RuntimeHostConfigurationOption Condition="'$(EnableUnsafeUTF7Encoding)' == ''" Include="System.Text.Encoding.EnableUnsafeUTF7Encoding" Value="false" />
<RuntimeHostConfigurationOption Condition="'$(HttpActivityPropagationSupport)' == ''" Include="System.Net.Http.EnableActivityPropagation" Value="false" />
<RuntimeHostConfigurationOption Condition="'$(NullabilityInfoContextSupport)' == ''" Include="System.Reflection.NullabilityInfoContext.IsSupported" Value="false" />
<RuntimeHostConfigurationOption Condition="'$(_AggressiveAttributeTrimming)' == ''" Include="System.AggressiveAttributeTrimming" Value="true" />
<RuntimeHostConfigurationOption Condition="'$(DebuggerSupport)' == ''and '$(Configuration)' != 'Debug'" Include="System.Diagnostics.Debugger.IsSupported" Value="false" />

<!-- Allow running/debugging from VS -->
<ProjectCapability Include="DotNetCoreWeb"/>

Expand Down