RESOLVED FIXED 147610
[INTL] Implement Number.prototype.toLocaleString in ECMA-402 
https://bugs.webkit.org/show_bug.cgi?id=147610
Summary [INTL] Implement Number.prototype.toLocaleString in ECMA-402
Andy VanWagoner
Reported 2015-08-03 17:32:41 PDT
Implement ECMA-402 2.0 13.2.1 Number.prototype.toLocaleString ([locales [, options ]]) http://ecma-international.org/publications/standards/Ecma-402.htm This definition supersedes the definition provided in ES2015, 20.1.3.4.
Attachments
Patch (19.97 KB, patch)
2015-12-02 21:34 PST, Andy VanWagoner 		no flags
DetailsFormatted DiffDiff
Patch (17.76 KB, patch)
2015-12-03 15:50 PST, Andy VanWagoner 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Andy VanWagoner
Comment 1 2015-12-02 21:34:28 PST
Created attachment 266511 [details] Patch
Benjamin Poulain
Comment 2 2015-12-03 14:27:45 PST
Can you please fix GTK/EFL first?
Andy VanWagoner
Comment 3 2015-12-03 14:30:47 PST
(In reply to comment #2) > Can you please fix GTK/EFL first? Yes, I just need to figure out why those platforms are not building NumberPrototype.js. Any ideas?
Andy VanWagoner
Comment 4 2015-12-03 15:50:23 PST
Created attachment 266570 [details] Patch
Andy VanWagoner
Comment 5 2015-12-03 16:49:59 PST
(In reply to comment #3) > (In reply to comment #2) > > Can you please fix GTK/EFL first? > > Yes, I just need to figure out why those platforms are not building > NumberPrototype.js. Any ideas? I had missed CMakeLists.txt. Latest patch fixes the omission.
WebKit Commit Bot
Comment 6 2015-12-04 17:14:19 PST
Comment on attachment 266570 [details] Patch Clearing flags on attachment: 266570 Committed r193493: <http://trac.webkit.org/changeset/193493>
WebKit Commit Bot
Comment 7 2015-12-04 17:14:22 PST
All reviewed patches have been landed. Closing bug.
Darin Adler
Comment 8 2015-12-13 16:35:34 PST
Seems likely this implementation of toLocaleString does not respect the number separator settings from the System Preferences page called Language & Region, under Advanced, on OS X. I don’t think it’s sufficient to call through to ICU without setting up ICU appropriately.
Darin Adler
Comment 9 2015-12-13 16:35:48 PST
*** Bug 77018 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.