On Wed, Sep 1, 2021 at 8:22 PM Nidhi Jaju <nidhij...@chromium.org> wrote:
> > > On Wed, Sep 1, 2021 at 10:54 PM Alex Russell <slightly...@chromium.org> > wrote: > >> Incremental features often benefit from TAG guidance. I'd feel better if >> this intent at least cited prior TAG reviews in this area or filed an FYI >> (with potential for TAG to request full review). >> >> If it was covered in a previous review of WritableStreams, that would >> also be good to know. >> > > This feature wasn't covered in a previous review of WritableStreams, so I > have filed a new TAG review: > https://github.com/w3ctag/design-reviews/issues/672 > > >> >> On Wed, Sep 1, 2021, 2:28 PM Yoav Weiss <yoavwe...@chromium.org> wrote: >> >>> >>> >>> On Tue, Aug 31, 2021 at 5:14 PM Nidhi Jaju <nidhij...@chromium.org> >>> wrote: >>> >>>> Contact emailsnidhij...@chromium.org, ri...@chromium.org, >>>> yhir...@chromium.org >>>> >>>> Explainer >>>> https://github.com/whatwg/streams/blob/main/writable-stream-abort-signal-explainer.md >>>> >>>> >>> >>>> >>>> Specification >>>> https://streams.spec.whatwg.org/#writablestreamdefaultcontroller-signal >>>> >>>> Summary >>>> >>>> The streams APIs provide ubiquitous, interoperable primitives for >>>> creating, composing, and consuming streams of data. This change permits an >>>> underlying sink to rapidly abort an ongoing write or close when requested >>>> by the writer. >>>> >>>> >>>> Previously, when writer.abort() was called, a long-running write would >>>> still have to continue to completion before the stream could be aborted. >>>> With this change, the write can be aborted immediately. >>>> >>> >>> Am I correct in assuming that the stream developer would have to change >>> their code in order for that to happen, and if they won't, the current >>> behavior won't change? >>> >> > Yes, that's correct. > > >>> >>>> In addition to being exposed to streams authored in JavaScript, this >>>> facility will also be used by platform-provided streams such as >>>> WebTransport. >>>> >>>> Blink componentBlink>Network>StreamsAPI >>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3ENetwork%3EStreamsAPI> >>>> >>>> TAG reviewNot needed as this is an incremental improvement to an >>>> existing feature. >>>> >>>> TAG review statusNot applicable >>>> >>>> Risks >>>> Interoperability and Compatibility >>>> >>>> Low risk because streams has already been standardised for a long time >>>> (since around 2014). Other browsers have implemented other parts of the >>>> standard, and they will most likely also adapt this feature as well soon. >>>> >>>> Gecko: Positive ( >>>> https://github.com/whatwg/streams/pull/1132#issuecomment-867114529 >>>> <https://chromestatus.com/admin/features/launch/5698931422920704/5?intent=1>) >>>> Mozilla is supportive of Streams but has not yet shipped WritableStream. It >>>> is expected that they will include this feature when they do. They have >>>> also expressed interest in implementing WebTransport, and therefore are >>>> supportive of this feature. >>>> >>> >>> While such comments don't typically count as a position, combining them >>> with https://github.com/mozilla/standards-positions/issues/167 seems >>> reasonable, given the size of this change. >>> >> That said, could you comment on issue 167 giving at least an FYI that this intent is happening? > >>> >>>> >>>> WebKit: No signal >>>> >>> >>> Have you reached out? https://bit.ly/blink-signals >>> >> > I have requested for WebKit's position here: > https://lists.webkit.org/pipermail/webkit-dev/2021-September/031977.html > > >> >>> >>>> >>>> Web developers: No signals >>>> >>> >>> Have you reached out? https://goo.gle/developer-signals >>> >> > Mattias Buelens has shown positive support here: > https://github.com/whatwg/streams/issues/1015#issuecomment-531188273 > <https://github.com/whatwg/streams/issues/1015#issuecomment-531188273> > > >> >>> >>>> >>>> Ergonomics >>>> >>>> A lot of design efforts have been made into making the streams API easy >>>> to use. Additionally, this feature is only a small change to the existing >>>> streams API that developers are used to. An underlying sink which doesn't >>>> observe the controller.signal will continue to have the existing behavior. >>>> >>>> >>>> Debuggability >>>> >>>> No special support needed. >>>> >>>> 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 >>>> Requires code in //chrome?False >>>> >>>> Tracking bug >>>> https://bugs.chromium.org/p/chromium/issues/detail?id=1215992 >>>> >>>> Patch >>>> https://chromium-review.googlesource.com/c/chromium/src/+/3127824 >>>> >>>> Estimated milestonesM95 >>>> >>>> Link to entry on the Chrome Platform Status >>>> https://chromestatus.com/feature/5698931422920704 >>>> >>>> This intent message was generated by Chrome Platform Status >>>> <https://www.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/CAMZNYAP-Of819oOp4yq8vRdzx78UUCPj9YKJqx4ZwLj_YsN%3D-Q%40mail.gmail.com >>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAMZNYAP-Of819oOp4yq8vRdzx78UUCPj9YKJqx4ZwLj_YsN%3D-Q%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/CAL5BFfXWMCdEAA51FQ6tJyG8CfuU_oOb-x0-mmBp5qFkDpFP8A%40mail.gmail.com >>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfXWMCdEAA51FQ6tJyG8CfuU_oOb-x0-mmBp5qFkDpFP8A%40mail.gmail.com?utm_medium=email&utm_source=footer> >>> . >> >> > Hopefully, this addresses your questions/concerns. Thank you! > > Best regards, > Nidhi > > -- > 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/CAMZNYANY35OcJeP6h1cOEF6FOU%3D_rAVnq2Rk%2BXeQ3OU0AT0wnQ%40mail.gmail.com > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAMZNYANY35OcJeP6h1cOEF6FOU%3D_rAVnq2Rk%2BXeQ3OU0AT0wnQ%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%2Bw8BuVi_JeX8PxcOWX3Y_AXi2TasLmY495YQDFnq6wLCQA%40mail.gmail.com.