Bug 236121 - Passing undefined for the locale in the Intl.NumberFormatter constructor uses the system locale
Summary: Passing undefined for the locale in the Intl.NumberFormatter constructor uses...
Status: RESOLVED MOVED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-02-03 20:20 PST by Myles C. Maxfield
Modified: 2022-02-03 21:13 PST (History)
2 users (show)

See Also:


Attachments
Example (196 bytes, text/html)
2022-02-03 20:22 PST, Myles C. Maxfield
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Myles C. Maxfield 2022-02-03 20:20:39 PST
1. This doesn't match Chrome (but does match Firefox)
2. This causes bugs on some sites which may receive numerals they weren't expecting (See the prices in https://www.westelm.com/search/results.html?words=floor%20lamp with your system set to Arabic)
3. There are fingerprinting concerns
Comment 1 Myles C. Maxfield 2022-02-03 20:22:59 PST
Created attachment 450860 [details]
Example

Open the example in Chrome and WebKit with your system set to Arabic.
Comment 2 Radar WebKit Bug Importer 2022-02-03 20:25:40 PST
<rdar://problem/88471306>
Comment 3 Yusuke Suzuki 2022-02-03 20:52:41 PST
OK, this is because upstream CLDR has two options for numbering-system in ar locale.

1. arab
2. latn

By default, it is pointing "arab". But it looks like there is an option "latn".
And according to CLDR issue (https://unicode-org.atlassian.net/browse/CLDR-10825)

> And add: <defaultNumberingSystem alt=”latndigi”>latn</…> → Google (and other companies like Netflix) which want to default non-country-specific Arabic to ASCII can use this...we can then test it out, see how it works, and based on that experience see if we should switch default and alt
> Stock ICU uses “standard” variant for ar.xml (Arab); Google etc will filter the data so that ar.xml gets <defaultNumberingSystem>Latn</…>

So, Stock ICU (& current Apple ICU) uses arab. Google's fork is using latn.
Comment 4 Myles C. Maxfield 2022-02-03 21:13:15 PST
Okay, because this bug is about Apple's internal implementation of ICU, let's move this to radar and continue there. Marking as RESOLVED MOVED.