LGTM *From: *Frank Tang <ft...@chromium.org> *Date: *Tue, May 14, 2019 at 5:33 PM *To: *blink-dev, v8-users, <v8-dev@googlegroups.com>, Adam Klein, Sathya Gunasekaran, Jakob Kummerow, Nebojša Ćirić
Contact emails ft...@chromium.org,js...@chromium.org Explainer > https://github.com/tc39/ecma402/pull/175 Design docs/spec Specification: > https://github.com/tc39/ecma402/pull/175 > https://github.com/tc39/ecma402/pull/175 TAG review No need for TAG > review since the TC39 / ECMA402 process provide oversight. Summary Allows > calendar and numberingSystem to be specified in the options bag of the > DateTimeFormat and NumberFormat constructors. Motivation One use case for > these options would be, when working with locales which have two numbering > systems and calendars in use--the UA default may be the non-Western one, > but in some contexts, it may be appropriate to use the Western one. > Currently, without the patch, it would be necessary to parse the BCP 47 > language tag in the application. This feature make it simpler for the > developer. See https://github.com/tc39/ecma402/pull/175 for more details. > Risks > Interoperability and Compatibility The PR in > https://github.com/tc39/ecma402/pull/175 is in last stage to be merged > into ECMA402. *Firefox*: No public signals *Edge*: No public signals > *Safari*: No public signals *Web developers*: No signals Ergonomics > Small. Both Intl object already provide the underline format functionality > by reading calendar ("ca") and numberingSystem ("nu") from the locale. This > feature only add a tiny option reading step Activation Low. A lot of > users already use these two API . New Javascript code which follow this > change running on an old version of browser which does not support this > browser will format the number and/or date in different result. But users > should still be able to understand these output. Security None > Debuggability No additional debugging support required. Will this feature > be supported on all six Blink platforms (Windows, Mac, Linux, Chrome OS, > Android, and Android WebView)? Yes Is this feature fully tested by > web-platform-tests > <https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md> > ? Yes Tests will be added to test262 before launching Link to entry on > the Chrome Platform Status > https://www.chromestatus.com/features/5440249461211136 > > -- > -- > v8-users mailing list > v8-us...@googlegroups.com > http://groups.google.com/group/v8-users > --- > You received this message because you are subscribed to the Google Groups > "v8-users" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to v8-users+unsubscr...@googlegroups.com. > To view this discussion on the web visit > https://groups.google.com/d/msgid/v8-users/CAOcELL_AamuY_jQBmqQjLUW7rswNNih3AQ2SxJ%2BG8CP4r%2Bn9GQ%40mail.gmail.com > <https://groups.google.com/d/msgid/v8-users/CAOcELL_AamuY_jQBmqQjLUW7rswNNih3AQ2SxJ%2BG8CP4r%2Bn9GQ%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > For more options, visit https://groups.google.com/d/optout. > -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups "v8-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to v8-dev+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/v8-dev/CADizRgZjxUqvVs9CP9MhknDD5Lc%3DrCwdGK33WtXsbo%2B3SiMtYg%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.