Don't throw FatalInternalGlobalizationError on _no_icu -- only (INTL_ICU || INTL_WINGLOB)

Author: dilijev

lib/Parser/CharClassifier.cpp

@@ -411,11 +411,13 @@ Js::CharClassifier::CharClassifier(void)
     bool isES6UnicodeModeEnabled = CONFIG_FLAG(ES6Unicode);
     bool isFullUnicodeSupportAvailable = PlatformAgnostic::UnicodeText::IsExternalUnicodeLibraryAvailable();
 
+#if INTL_ICU || INTL_WINGLOB // don't assert in _no_icu builds (where there is no i18n library, by design)
     AssertMsg(isFullUnicodeSupportAvailable, "Windows.Globalization needs to present with IUnicodeCharacterStatics support for Chakra.dll to work");
     if (!isFullUnicodeSupportAvailable)
     {
         Js::Throw::FatalInternalGlobalizationError();
     }
+#endif
 
     // If we're in ES6 mode, and we have full support for Unicode character classification
     // from an external library, then use the ES6/Surrogate pair supported versions of the functions

lib/Runtime/PlatformAgnostic/Platform/Windows/UnicodeText.cpp

@@ -407,6 +407,7 @@ namespace PlatformAgnostic
                     {
                         return true;
                     }
+#if INTL_ICU || INTL_WINGLOB // don't assert in _no_icu builds (where there is no i18n library, by design)
                     else
                     {
                         // did not find winGlobCharApi
@@ -417,9 +418,10 @@ namespace PlatformAgnostic
                 {
                     // failed to initialize Windows Globalization
                     Js::Throw::FatalInternalGlobalizationError();
+#endif
                 }
 
-#ifndef DBG
+#if (INTL_ICU || INTL_WINGLOB) && !defined(DBG)
                 return false; // in debug builds, this is unreachable code
 #endif
             }, false);