Revert "[wasm] Throw exception if culture data does not exist in icu (d…

…otnet#47301)"

This reverts commit 128c757.

src/libraries/Common/tests/Tests/System/StringTests.cs

@@ -2567,7 +2567,8 @@ public void Equals_Encyclopaedia_ReturnsExpected(StringComparison comparison, bo
         {
             string source = "encyclop\u00e6dia";
             string target = "encyclopaedia";
-            using (new ThreadCultureChange(PlatformDetection.IsBrowser ?"pl-PL" : "se-SE"))
+
+            using (new ThreadCultureChange("se-SE"))
             {
                 Assert.Equal(expected, string.Equals(source, target, comparison));
                 Assert.Equal(expected, source.AsSpan().Equals(target.AsSpan(), comparison));

src/libraries/System.ComponentModel.TypeConverter/tests/CultureInfoConverterTests.cs

@@ -33,10 +33,8 @@ public override IEnumerable<ConvertTest> ConvertFromTestData()
                 yield return ConvertTest.Valid("nl-B", new CultureInfo("nl--B"), CultureInfo.InvariantCulture);
                 yield return ConvertTest.Valid("nl-B", new CultureInfo("nl--B"), new CultureInfo("en-US"));
             }
-            if (PlatformDetection.IsNotBrowser)
-            {
-                yield return ConvertTest.Valid("Afrikaans", new CultureInfo("af"));
-            }
+
+            yield return ConvertTest.Valid("Afrikaans", new CultureInfo("af"));
 
             yield return ConvertTest.CantConvertFrom(CultureInfo.CurrentCulture);
             yield return ConvertTest.CantConvertFrom(1);

src/libraries/System.Globalization/tests/CompareInfo/CompareInfoTests.Compare.cs

@@ -254,11 +254,8 @@ public static IEnumerable<object[]> Compare_TestData()
             yield return new object[] { s_hungarianCompare, "dzsdzs", "ddzs", CompareOptions.None, useNls ? 0 : -1 };
             yield return new object[] { s_invariantCompare, "Test's", "Tests", CompareOptions.None, useNls ? 1 : -1 };
             yield return new object[] { new CultureInfo("de-DE").CompareInfo, "\u00DC", "UE", CompareOptions.None, -1 };
-            if (PlatformDetection.IsNotBrowser)
-            {
-                yield return new object[] { new CultureInfo("de-DE_phoneb").CompareInfo, "\u00DC", "UE", CompareOptions.None, useNls ? 0 : -1 };
-                yield return new object[] { new CultureInfo("es-ES_tradnl").CompareInfo, "llegar", "lugar", CompareOptions.None, useNls ? 1 : -1 };
-            }
+            yield return new object[] { new CultureInfo("de-DE_phoneb").CompareInfo, "\u00DC", "UE", CompareOptions.None, useNls ? 0 : -1 };
+            yield return new object[] { new CultureInfo("es-ES_tradnl").CompareInfo, "llegar", "lugar", CompareOptions.None, useNls ? 1 : -1 };
         }
 
         // There is a regression in Windows 190xx version with the Kana comparison. Avoid running this test there.

src/libraries/System.Globalization/tests/CultureInfo/CultureInfoCurrentCulture.cs

@@ -22,14 +22,11 @@ public void CurrentCulture()
                 Assert.Equal(CultureInfo.CurrentCulture, newCulture);
             }
 
-            if (PlatformDetection.IsNotBrowser)
+            newCulture = new CultureInfo("de-DE_phoneb");
+            using (new ThreadCultureChange(newCulture))
             {
-                newCulture = new CultureInfo("de-DE_phoneb");
-                using (new ThreadCultureChange(newCulture))
-                {
-                    Assert.Equal(CultureInfo.CurrentCulture, newCulture);
-                    Assert.Equal("de-DE_phoneb", newCulture.CompareInfo.Name);
-                }
+                Assert.Equal(CultureInfo.CurrentCulture, newCulture);
+                Assert.Equal("de-DE_phoneb", newCulture.CompareInfo.Name);
             }
         }
 
@@ -48,14 +45,11 @@ public void CurrentUICulture()
                 Assert.Equal(CultureInfo.CurrentUICulture, newUICulture);
             }
 
-            if (PlatformDetection.IsNotBrowser)
+            newUICulture = new CultureInfo("de-DE_phoneb");
+            using (new ThreadCultureChange(null, newUICulture))
             {
-                newUICulture = new CultureInfo("de-DE_phoneb");
-                using (new ThreadCultureChange(null, newUICulture))
-                {
-                    Assert.Equal(CultureInfo.CurrentUICulture, newUICulture);
-                    Assert.Equal("de-DE_phoneb", newUICulture.CompareInfo.Name);
-                }
+                Assert.Equal(CultureInfo.CurrentUICulture, newUICulture);
+                Assert.Equal("de-DE_phoneb", newUICulture.CompareInfo.Name);
             }
         }
 

src/libraries/System.Globalization/tests/CultureInfo/CultureInfoParent.cs

@@ -35,20 +35,10 @@ public void Parent(string name, string expectedParentName)
         [Fact]
         public void Parent_ParentChain()
         {
-            if (PlatformDetection.IsNotBrowser)
-            {
-                CultureInfo myExpectParentCulture = new CultureInfo("uz-Cyrl-UZ");
-                Assert.Equal("uz-Cyrl", myExpectParentCulture.Parent.Name);
-                Assert.Equal("uz", myExpectParentCulture.Parent.Parent.Name);
-                Assert.Equal("", myExpectParentCulture.Parent.Parent.Parent.Name);
-            }
-            else
-            {
-                CultureInfo myExpectParentCulture = new CultureInfo("zh-Hans-CN");
-                Assert.Equal("zh-Hans", myExpectParentCulture.Parent.Name);
-                Assert.Equal("zh", myExpectParentCulture.Parent.Parent.Name);
-                Assert.Equal("", myExpectParentCulture.Parent.Parent.Parent.Name);
-            }
+            CultureInfo myExpectParentCulture = new CultureInfo("uz-Cyrl-UZ");
+            Assert.Equal("uz-Cyrl", myExpectParentCulture.Parent.Name);
+            Assert.Equal("uz", myExpectParentCulture.Parent.Parent.Name);
+            Assert.Equal("", myExpectParentCulture.Parent.Parent.Parent.Name);
         }
     }
 }

src/libraries/System.Globalization/tests/CultureInfo/GetCultureInfo.cs

@@ -2,16 +2,13 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 
 using System.Collections.Generic;
-using Microsoft.DotNet.RemoteExecutor;
-using System.Diagnostics;
 using Xunit;
 
 namespace System.Globalization.Tests
 {
     public class GetCultureInfoTests
     {
-        public static bool PlatformSupportsFakeCulture => (!PlatformDetection.IsWindows || (PlatformDetection.WindowsVersion >= 10 && !PlatformDetection.IsNetFramework)) && PlatformDetection.IsNotBrowser;
-        public static bool PlatformSupportsFakeCultureAndRemoteExecutor => PlatformSupportsFakeCulture && RemoteExecutor.IsSupported;
+        public static bool PlatformSupportsFakeCulture => !PlatformDetection.IsWindows || (PlatformDetection.WindowsVersion >= 10 && !PlatformDetection.IsNetFramework);
 
         public static IEnumerable<object[]> GetCultureInfoTestData()
         {
@@ -113,31 +110,5 @@ public void TestFakeCultureNames(string name)
             Assert.Equal(name, CultureInfo.GetCultureInfo(name, predefinedOnly: false).Name);
             Assert.Throws<CultureNotFoundException>(() => CultureInfo.GetCultureInfo(name, predefinedOnly: true));
         }
-
-        [ConditionalTheory(nameof(PlatformSupportsFakeCultureAndRemoteExecutor))]
-        [InlineData("1", "xx-XY")]
-        [InlineData("1", "zx-ZY")]
-        [InlineData("0", "xx-XY")]
-        [InlineData("0", "zx-ZY")]
-        public void PredefinedCulturesOnlyEnvVarTest(string predefinedCulturesOnlyEnvVar, string cultureName)
-        {
-            var psi = new ProcessStartInfo();
-            psi.Environment.Clear();
-
-            psi.Environment.Add("DOTNET_SYSTEM_GLOBALIZATION_PREDEFINED_CULTURES_ONLY", predefinedCulturesOnlyEnvVar);
-
-            RemoteExecutor.Invoke((culture, predefined) =>
-            {
-                if (predefined == "1")
-                {
-                    AssertExtensions.Throws<CultureNotFoundException>(() => new CultureInfo(culture));
-                }
-                else
-                {
-                    CultureInfo ci = new CultureInfo(culture);
-                    Assert.Equal(culture, ci.Name);
-                }
-            }, cultureName, predefinedCulturesOnlyEnvVar, new RemoteInvokeOptions { StartInfo = psi }).Dispose();
-        }
     }
 }

src/libraries/System.Globalization/tests/System/Globalization/TextInfoTests.cs

@@ -288,7 +288,7 @@ public static IEnumerable<object[]> ToLower_TestData()
             // ICU has special tailoring for the en-US-POSIX locale which treats "i" and "I" as different letters
             // instead of two letters with a case difference during collation.  Make sure this doesn't confuse our
             // casing implementation, which uses collation to understand if we need to do Turkish casing or not.
-            if (!PlatformDetection.IsWindows && PlatformDetection.IsNotBrowser)
+            if (!PlatformDetection.IsWindows)
             {
                 yield return new object[] { "en-US-POSIX", "I", "i" };
             }
@@ -411,7 +411,7 @@ public static IEnumerable<object[]> ToUpper_TestData()
             // ICU has special tailoring for the en-US-POSIX locale which treats "i" and "I" as different letters
             // instead of two letters with a case difference during collation.  Make sure this doesn't confuse our
             // casing implementation, which uses collation to understand if we need to do Turkish casing or not.
-            if (!PlatformDetection.IsWindows && PlatformDetection.IsNotBrowser)
+            if (!PlatformDetection.IsWindows)
             {
                 yield return new object[] { "en-US-POSIX", "i", "I" };
             }

src/libraries/System.Private.CoreLib/src/System.Private.CoreLib.Shared.projitems

@@ -307,6 +307,8 @@
     <Compile Include="$(MSBuildThisFileDirectory)System\Globalization\CultureData.Icu.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\Globalization\CultureData.Nls.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\Globalization\CultureInfo.cs" />
+    <Compile Include="$(MSBuildThisFileDirectory)System\Globalization\CultureInfo.Icu.cs" />
+    <Compile Include="$(MSBuildThisFileDirectory)System\Globalization\CultureInfo.Nls.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\Globalization\CultureNotFoundException.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\Globalization\CultureTypes.cs" />
     <Compile Include="$(MSBuildThisFileDirectory)System\Globalization\DateTimeFormat.cs" />

src/libraries/System.Private.CoreLib/src/System/Globalization/CultureData.Icu.cs

@@ -63,8 +63,11 @@ private bool InitIcuCultureDataCore()
             {
                 _iLanguage = CultureInfo.LOCALE_CUSTOM_UNSPECIFIED;
             }
+
             _bNeutral = TwoLetterISOCountryName.Length == 0;
+
             _sSpecificCulture = _bNeutral ? IcuLocaleData.GetSpecificCultureName(_sRealName) : _sRealName;
+
             // Remove the sort from sName unless custom culture
             if (index > 0 && !_bNeutral && !IsCustomCultureId(_iLanguage))
             {
@@ -107,6 +110,7 @@ private string IcuGetLocaleInfo(LocaleStringData type)
         {
             Debug.Assert(!GlobalizationMode.Invariant);
             Debug.Assert(!GlobalizationMode.UseNls);
+
             Debug.Assert(_sWindowsName != null, "[CultureData.IcuGetLocaleInfo] Expected _sWindowsName to be populated already");
             return IcuGetLocaleInfo(_sWindowsName, type);
         }
@@ -134,6 +138,7 @@ private unsafe string IcuGetLocaleInfo(string localeName, LocaleStringData type)
                 Debug.Fail("[CultureData.IcuGetLocaleInfo(LocaleStringData)] Failed");
                 return string.Empty;
             }
+
             return new string(buffer);
         }
 
@@ -221,12 +226,6 @@ private static string IcuGetLanguageDisplayName(string cultureName)
             return null;
         }
 
-        internal static bool IcuIsEnsurePredefinedLocaleName(string name)
-        {
-            Debug.Assert(!GlobalizationMode.UseNls);
-            return Interop.Globalization.IsPredefinedLocale(name);
-        }
-
         private static string ConvertIcuTimeFormatString(ReadOnlySpan<char> icuFormatString)
         {
             Debug.Assert(icuFormatString.Length < ICU_ULOC_FULLNAME_CAPACITY);

src/libraries/System.Private.CoreLib/src/System/Globalization/CultureData.Nls.cs

@@ -85,12 +85,6 @@ private int[] NlsGetLocaleInfo(LocaleGroupingData type)
             return ConvertWin32GroupString(GetLocaleInfoFromLCType(_sWindowsName, (uint)type, _bUseOverrides));
         }
 
-        internal static bool NlsIsEnsurePredefinedLocaleName(string name)
-        {
-            Debug.Assert(GlobalizationMode.UseNls);
-            return CultureData.GetLocaleInfoExInt(name, Interop.Kernel32.LOCALE_ICONSTRUCTEDLOCALE) != 1;
-        }
-
         private string? NlsGetTimeFormatString()
         {
             Debug.Assert(ShouldUseUserOverrideNlsData);

src/libraries/System.Private.CoreLib/src/System/Globalization/CultureData.cs

@@ -422,14 +422,17 @@ internal partial class CultureData
             {
                 return CultureData.Invariant;
             }
-            CultureData? retVal = null;
+
             // First check if GetCultureData() can find it (ie: its a real culture)
-            retVal = GetCultureData(cultureName, useUserOverride);
+            CultureData? retVal = GetCultureData(cultureName, useUserOverride);
             if (retVal != null && !retVal.IsNeutralCulture)
             {
                 return retVal;
             }
 
+            // Not a specific culture, perhaps it's region-only name
+            // (Remember this isn't a core clr path where that's not supported)
+
             // If it was neutral remember that so that RegionInfo() can throw the right exception
             CultureData? neutral = retVal;
 
@@ -673,12 +676,6 @@ private static CultureData CreateCultureWithInvariantData()
                 return CultureData.Invariant;
             }
 
-            if (GlobalizationMode.PredefinedCulturesOnly && !GlobalizationMode.Invariant)
-            {
-                if (GlobalizationMode.UseNls ? !NlsIsEnsurePredefinedLocaleName(cultureName): !IcuIsEnsurePredefinedLocaleName(cultureName))
-                    return null;
-            }
-
             // Try the hash table first
             string hashName = AnsiToLower(useUserOverride ? cultureName : cultureName + '*');
             Dictionary<string, CultureData>? tempHashTable = s_cachedCultures;
@@ -701,6 +698,7 @@ private static CultureData CreateCultureWithInvariantData()
                     return retVal;
                 }
             }
+
             // Not found in the hash table, need to see if we can build one that works for us
             CultureData? culture = CreateCultureData(cultureName, useUserOverride);
             if (culture == null)

src/libraries/System.Private.CoreLib/src/System/Globalization/CultureInfo.Icu.cs

@@ -0,0 +1,22 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using System.Diagnostics;
+
+namespace System.Globalization
+{
+    public partial class CultureInfo : IFormatProvider
+    {
+        private static CultureInfo IcuGetPredefinedCultureInfo(string name)
+        {
+            Debug.Assert(!GlobalizationMode.UseNls);
+
+            if (!Interop.Globalization.IsPredefinedLocale(name))
+            {
+                throw new CultureNotFoundException(nameof(name), SR.Format(SR.Argument_InvalidPredefinedCultureName, name));
+            }
+
+            return GetCultureInfo(name);
+        }
+    }
+}

src/libraries/System.Private.CoreLib/src/System/Globalization/CultureInfo.Nls.cs

@@ -0,0 +1,22 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using System.Diagnostics;
+
+namespace System.Globalization
+{
+    public partial class CultureInfo : IFormatProvider
+    {
+        private static CultureInfo NlsGetPredefinedCultureInfo(string name)
+        {
+            Debug.Assert(GlobalizationMode.UseNls);
+
+            if (CultureData.GetLocaleInfoExInt(name, Interop.Kernel32.LOCALE_ICONSTRUCTEDLOCALE) == 1)
+            {
+                throw new CultureNotFoundException(nameof(name), SR.Format(SR.Argument_InvalidPredefinedCultureName, name));
+            }
+
+            return GetCultureInfo(name);
+        }
+    }
+}

src/libraries/System.Private.CoreLib/src/System/Globalization/CultureInfo.cs

@@ -1169,10 +1169,9 @@ public static CultureInfo GetCultureInfo(string name, bool predefinedOnly)
 
             if (predefinedOnly && !GlobalizationMode.Invariant)
             {
-                if (GlobalizationMode.UseNls ? !CultureData.NlsIsEnsurePredefinedLocaleName(name): !CultureData.IcuIsEnsurePredefinedLocaleName(name))
-                {
-                    throw new CultureNotFoundException(nameof(name), name, SR.Format(SR.Argument_InvalidPredefinedCultureName, name));
-                }
+                return GlobalizationMode.UseNls ?
+                    NlsGetPredefinedCultureInfo(name) :
+                    IcuGetPredefinedCultureInfo(name);
             }
 
             return GetCultureInfo(name);

src/libraries/System.Private.CoreLib/src/System/Globalization/GlobalizationMode.cs

@@ -14,9 +14,6 @@ private static bool GetInvariantSwitchValue() =>
         private static bool TryGetAppLocalIcuSwitchValue([NotNullWhen(true)] out string? value) =>
             TryGetStringValue("System.Globalization.AppLocalIcu", "DOTNET_SYSTEM_GLOBALIZATION_APPLOCALICU", out value);
 
-        internal static bool PredefinedCulturesOnly { get; } =
-            GetSwitchValue("System.Globalization.PredefinedCulturesOnly", "DOTNET_SYSTEM_GLOBALIZATION_PREDEFINED_CULTURES_ONLY");
-
         private static bool GetSwitchValue(string switchName, string envVariable)
         {
             if (!AppContext.TryGetSwitch(switchName, out bool ret))

src/libraries/System.Reflection.MetadataLoadContext/tests/src/TestUtils/TestData.cs

@@ -985,7 +985,7 @@ internal static class TestData
             "ABYAAAEAAAACAAAAAQAAAAIAAAACAAAAAQAAAAMAAAAAAG4AAQAAAAAABgAqAAoABgBQAAoAAAAAAAEAAAAAAAEAAQAJAEoAAQARAEoABgAuAAsAEwAuABMA" +
             "HAAEgAAAAAAAAAAAAAAAAAAAAAChAAAABAAAAAAAAAAAAAAACgCIAAAAAAAAAAAAAAAAAAAAAAAAAJEAAAAAAAEAAgADAAQAAAAAAAAAlgCbAAAAAAAAPE1v" +
             "ZHVsZT4AU3lzdGVtLlJ1bnRpbWUuQ29tcGlsZXJTZXJ2aWNlcwBDb21waWxhdGlvblJlbGF4YXRpb25zQXR0cmlidXRlAC5jdG9yAFJ1bnRpbWVDb21wYXRp" +
-            "YmlsaXR5QXR0cmlidXRlAEFzc2VtYmx5UmVmZXJlbmNlVGVzdC5kbGwAbXNjb3JsaWIAZGVwMQBkZXAyAHBsLVBMAEFzc2VtYmx5UmVmZXJlbmNlVGVzdAAA" +
+            "YmlsaXR5QXR0cmlidXRlAEFzc2VtYmx5UmVmZXJlbmNlVGVzdC5kbGwAbXNjb3JsaWIAZGVwMQBkZXAyAGFyLUxZAEFzc2VtYmx5UmVmZXJlbmNlVGVzdAAA" +
             "AAMgAAAAAAD+latRaKErR6lFRbe04zMTAAQgAQEIAyAAAQi3elxWGTTgiQgBAAgAAAAAAB4BAAEAVAIWV3JhcE5vbkV4Y2VwdGlvblRocm93cwEAtCIAAAAA" +
             "AAAAAAAAziIAAAAgAAAAAAAAAAAAAAAAAAAAAAAAAAAAAMAiAAAAAAAAAAAAAAAAX0NvckRsbE1haW4AbXNjb3JlZS5kbGwAAAAAAP8lACAAEAAAAAAAAAAA" +
             "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA" +

src/libraries/System.Reflection.MetadataLoadContext/tests/src/Tests/Assembly/AssemblyTests.cs

@@ -171,6 +171,7 @@ public static void AssemblyGetReferencedAssemblies()
                 Assembly a = lc.LoadFromByteArray(TestData.s_AssemblyReferencesTestImage);
                 AssemblyName[] ans = a.GetReferencedAssemblies();
                 Assert.Equal(3, ans.Length);
+
                 {
                     AssemblyName an = ans.Single(an2 => an2.Name == "mscorlib");
                     Assert.Equal(default(AssemblyNameFlags), an.Flags);
@@ -210,7 +211,7 @@ public static void AssemblyGetReferencedAssemblies()
                     Assert.Equal(2, v.Minor);
                     Assert.Equal(3, v.Build);
                     Assert.Equal(4, v.Revision);
-                    Assert.Equal("pl-PL", an.CultureName);
+                    Assert.Equal("ar-LY", an.CultureName);
                     Assert.Null(an.GetPublicKey());
                     Assert.Equal(0, an.GetPublicKeyToken().Length);
                 }