LGTM3
On Tue, Dec 14, 2021 at 8:40 AM Mike Taylor <miketa...@chromium.org> wrote: > LGTM2 > > On 12/13/21 6:27 PM, Chris Harrelson wrote: > > LGTM1 > > On Mon, Dec 13, 2021 at 3:20 PM Frank Tang <ft...@chromium.org> wrote: > >> Dear API owners: >> [Five days after my I2S email] >> Any questions or feedback I should/need to take action before you grant >> me a lgtm? >> >> Thanks >> >> Regards, >> Frank Tang >> >> On Wed, Dec 8, 2021 at 10:42 AM Frank Tang <ft...@chromium.org> wrote: >> >>> [for m99, AFTER branching off m98 Dec 9, 2021] >>> >>> Contact emails ft...@google.com, s...@google.com >>> >>> Specification https://tc39.es/proposal-intl-locale-info/ >>> >>> Design docs >>> >>> https://docs.google.com/document/d/1OwEsvs8VQwvR-ug01xLyIwpgcvUfaP24u9owc7aBKJ4 >>> Update slides for TC39 2021-Dec-15 >>> <https://chromestatus.com/admin/features/launch/5566859262820352/Update%20slides%20for%20TC39%202021-Dec-15> >>> >>> https://docs.google.com/presentation/d/1PZ0_WiE9PNInY2bgyHGJH0DbKd0PKL9RApXxVPKJjUY >>> >>> Summary >>> >>> Intl Locale Info API is a new Stage ECMAScript TC39 proposal to enhance >>> the Intl.Locale object by exposing Locale information, such as week data >>> (first day in a week, weekend start day, weekend end day, minimun day in >>> the first week), and text direction hour cycle used in the locale. >>> https://github.com/tc39/proposal-intl-locale-info >>> >>> >>> Blink component Blink>JavaScript>Internationalization >>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EJavaScript%3EInternationalization> >>> >>> TAG review >>> >>> TAG review status Not applicable >>> >>> Risks >>> >>> >>> Interoperability and Compatibility >>> >>> Low- Intl.Locale object is already pre-exist in ECMA402 and implemented >>> by Mozilla. This launch adds 7 get methods on the object. >>> >>> >>> Gecko: Shipped/Shipping ( >>> https://bugzilla.mozilla.org/show_bug.cgi?id=1693576) Ship in 93 >>> >>> WebKit: Shipped/Shipping ( >>> https://developer.apple.com/safari/technology-preview/release-notes/) >>> Shipped in Safari Technology Preview 132 >>> >>> Web developers: Positive ( >>> https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/Locale) >>> MDN >>> >>> Other signals: >>> >>> Ergonomics >>> >>> Low risk. Intl.Locale is already in ECMA402, adding new methods should >>> be easy to discover. >>> >>> >>> Activation >>> >>> Low risk. Intl.Locale is already in ECMA402, adding new methods should >>> be easy to use. >>> >>> >>> Security >>> >>> Thew newly added method are getter only. Low risk for security. >>> >>> >>> Debuggability >>> >>> This feature help to debug avaiable values in the system >>> >>> >>> Is this feature fully tested by web-platform-tests >>> <https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md> >>> ? Yes >>> >>> Flag name harmony_intl_locale_info >>> >>> Requires code in //chrome? False >>> >>> Tracking bug https://bugs.chromium.org/p/v8/issues/detail?id=11638 >>> >>> Estimated milestones >>> DevTrial on desktop 92 >>> DevTrial on android 92 >>> >>> Link to entry on the Chrome Platform Status >>> https://chromestatus.com/feature/5566859262820352 >>> >>> Links to previous Intent discussions Ready for Trial: >>> https://groups.google.com/a/chromium.org/g/blink-dev/c/5spmAncbooE/m/NdwZGjLpAgAJ >>> >>> >>> This intent message was generated by Chrome Platform Status >>> <https://chromestatus.com/>. >>> >> -- >> You received this message because you are subscribed to the Google Groups >> "blink-dev" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to blink-dev+unsubscr...@chromium.org. >> To view this discussion on the web visit >> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOcELL95DVVeMQ38S3XOmCXZwdbz4_hp5M6XwTCbUru7Jt%3DZXw%40mail.gmail.com >> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOcELL95DVVeMQ38S3XOmCXZwdbz4_hp5M6XwTCbUru7Jt%3DZXw%40mail.gmail.com?utm_medium=email&utm_source=footer> >> . >> > > -- TAMURA Kent Software Engineer, Google -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGH7WqG1yfm6n4c_HuTov170gEbcXG6ftZ9YBBoWRzkTVBzidQ%40mail.gmail.com.