866474 - Make JSRuntime::setDefaultLocale check whether internationalization API can use the locale

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Norbert Lindenberg]

The JSRuntime::setDefaultLocale function currently accepts any string as a locale; it doesn't check whether it's a valid BCP 47 language tag and supported by the implementation of the Internationalization API. It's up to the DefaultLocale function in Intl.js whether it actually uses the runtime's default locale.

It might be better to have JSRuntime::setDefaultLocale check whether the string is acceptable, and reject it if not.

Comment 1

[Zibi Braniecki [:zbraniecki][:gandalf]]

The only caller of this method is now using LocaleService which canonicalizes the locale code.

I think we can close this as fixed.