LGTM3 On Fri, Feb 11, 2022 at 8:28 AM Yoav Weiss <yoavwe...@chromium.org> wrote:
> LGTM2 - thanks for helping us catching up! > > On Fri, Feb 11, 2022 at 4:57 PM Mike Taylor <miketa...@chromium.org> > wrote: > >> LGTM1 - seems like a useful feature and good to see that Gecko and WebKit >> have already shipped. >> >> On 2/10/22 5:54 PM, Scott Haseley wrote: >> >> Contact emails shase...@chromium.org >> >> Explainer Discussion: https://github.com/whatwg/dom/issues/927 >> Examples: >> https://developer.mozilla.org/en-US/docs/Web/API/AbortSignal/throwIfAborted >> >> Specification https://dom.spec.whatwg.org/#dom-abortsignal-throwifaborted >> >> Summary >> >> Throws an AbortSignal's reason if the signal is aborted. This convenience >> method can be used by signal-handling functions to check a signal's abort >> status and propagate the abort reason, e.g. after async operations that >> might change a signal's state. >> >> Blink component Blink>DOM >> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EDOM> >> >> TAG review N/A: the feature has been merged into the DOM standard and >> has been shipped in at least one other browser, in line with the criteria >> in >> https://groups.google.com/a/chromium.org/g/blink-dev/c/naqmDmy1iM8/m/lQAJ17CRAQAJ >> . >> >> TAG review status Not applicable >> >> Risks >> >> Interoperability and Compatibility >> >> Low risk. This feature is already part of the DOM standard, has web >> platform tests, and is implemented by Safari and Firefox. We'll improve >> eventual interop by shipping this feature. >> >> Gecko: Shipped/Shipping ( >> https://bugzilla.mozilla.org/show_bug.cgi?id=1745372) >> >> WebKit: Shipped/Shipping (https://bugs.webkit.org/show_bug.cgi?id=234127) >> >> Web developers: Positive. Minor positive feedback (likes) from >> announcement tweets: >> - Safari: https://twitter.com/chris_dumez/status/1469566206516424704 >> - Node: https://twitter.com/simonplend/status/1487053107720859648 and >> https://twitter.com/jasnell/status/1466093594129756166 >> >> Other signals: >> >> Ergonomics >> >> None; this feature is an ergonomic improvement for AbortSignal users. >> >> Activation >> >> The feature has already been implemented in both Safari and Firefox, but >> it would benefit from a polyfill for use in older browser versions. >> >> Security >> >> None. >> >> Debuggability >> >> Basic tooling only, i.e. autocomplete support for the new AbortSignal >> method will be provided. >> >> Is this feature fully tested by web-platform-tests >> <https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md> >> ? Yes ( >> https://wpt.fyi/results/dom/abort/event.any.html?label=master&label=experimental&aligned&q=dom%2Fabort >> ) >> >> Flag name --enable-blink-features=AbortSignalThrowIfAborted >> >> Requires code in //chrome? False >> >> Tracking bug >> https://bugs.chromium.org/p/chromium/issues/detail?id=1273458 >> >> Launch bug https://bugs.chromium.org/p/chromium/issues/detail?id=1290443 >> >> Measurement UseCounter: AbortSignalThrowIfAborted >> >> Sample links >> https://developer.mozilla.org/en-US/docs/Web/API/AbortSignal/throwIfAborted#examples >> >> Estimated milestones >> DevTrial on desktop 100 >> DevTrial on android 100 >> Link to entry on the Chrome Platform Status >> https://chromestatus.com/feature/5029737100476416 >> >> Links to previous Intent discussions Intent to prototype: >> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAKXGoJ3tUkTGZ1VBQm4139zL0n%3De-DO5emVpZE3ukya4Akyu2w%40mail.gmail.com >> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAKXGoJ3tUkTGZ1VBQm4139zL0n=e-do5emvpze3ukya4aky...@mail.gmail.com> >> >> >> 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/CAKXGoJ1w2WYK4X6fA7V4C0xNJBetNb%3D%2BCMywobhzzc9q9xSRxg%40mail.gmail.com >> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAKXGoJ1w2WYK4X6fA7V4C0xNJBetNb%3D%2BCMywobhzzc9q9xSRxg%40mail.gmail.com?utm_medium=email&utm_source=footer> >> . >> >> >> -- >> 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/469859d1-1f5a-b3f9-f725-26563c112bb7%40chromium.org >> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/469859d1-1f5a-b3f9-f725-26563c112bb7%40chromium.org?utm_medium=email&utm_source=footer> >> . >> > -- > 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/CAL5BFfXQznt_8ygdDsbANe-mCd%3DPkPNAYHkxgJtpmkexkEDmpg%40mail.gmail.com > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfXQznt_8ygdDsbANe-mCd%3DPkPNAYHkxgJtpmkexkEDmpg%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > -- 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/CAOMQ%2Bw_kM3eRarNOHVP%3DL%2BJZPJfwR6mnuRmNKq7wbg%3DRW92-0A%40mail.gmail.com.