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.
Created attachment 266511 [details] Patch
Can you please fix GTK/EFL first?
(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?
Created attachment 266570 [details] Patch
(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.
Comment on attachment 266570 [details] Patch Clearing flags on attachment: 266570 Committed r193493: <http://trac.webkit.org/changeset/193493>
All reviewed patches have been landed. Closing bug.
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.
*** Bug 77018 has been marked as a duplicate of this bug. ***