Thanks Domenic and Mike for the reply! I see. It looks like we have an LGTM on the Mozilla review <https://github.com/mozilla/standards-positions/issues/1019>. Could we please add some reviewers for the TAG review as well? If not, I'm happy to ping this thread again in two weeks :]
Best, Jiaming On Tue, May 14, 2024 at 10:16 PM Domenic Denicola <dome...@chromium.org> wrote: > Hi Jiaming, > > Per our process > <https://www.chromium.org/blink/launching-features/#new-feature-prepare-to-ship>, > we give the TAG and other vendors at least one month to comment on changes. > (This is why it is recommended to start these reviewers earlier, before > sending the Intent to Ship.) So it might be a bit more time before we can > consider this feature for shipping. Of course, if you get responses and > engagement before that point, we can proceed earlier. > > -Domenic > > On Tuesday, May 14, 2024 at 11:06:55 AM UTC+9 Jiaming Cheng wrote: > >> Hi team, >> >> Those reviews have been posted for 10 days, there's no opposing comments >> on any of them so far. Could you please take another look at this intent? >> >> Let me know if you have any further questions or concerns. >> >> Thanks, >> Jiaming >> >> On Fri, May 3, 2024 at 6:25 PM Jiaming Cheng <jiami...@google.com> wrote: >> >>> Hi Alex, Chris and Daniel, >>> >>> Thank you for your valuable feedback! >>> >>> I've addressed your comments and taken the following updates: >>> >>> Hey Alex, the ChapterInformation *does* apply to audio as well as >>> video, since MediaSession is for both audio and video. I've updated the >>> Chrome status to reflect this. >>> >>> Additionally, I've taken the following actions: >>> >>> - Added WPT test: >>> https://chromium-review.googlesource.com/c/chromium/src/+/5516503 >>> - Filed TAG review: >>> https://github.com/w3ctag/design-reviews/issues/952 >>> - Filed WebKit review: >>> https://github.com/WebKit/standards-positions/issues/344 >>> - Filed Gecko review: >>> https://github.com/mozilla/standards-positions/issues/1019 >>> >>> >>> I will keep you updated on the progress of these reviews and notify you >>> once they are approved. Let me know if you have any questions :] >>> >>> Best, >>> Jiaming >>> >>> On Wed, May 1, 2024 at 8:57 AM Alex Russell <slightly...@chromium.org> >>> wrote: >>> >>>> Hey folks, >>>> >>>> On reviewing this, I'm concerned that this isn't also addressing the >>>> same needs for Audio. This would have come up in a TAG review, and probably >>>> would have been fleshed out in an Explainer. Would like to see those before >>>> this progresses. >>>> >>>> Best, >>>> >>>> Alex >>>> >>>> On Tuesday, April 30, 2024 at 3:35:45 PM UTC-7 dan...@microsoft.com >>>> wrote: >>>> >>>>> I was curious about WPT coverage for this and found >>>>> https://wpt.fyi/results/mediasession/mediametadata.html >>>>> >>>>> >>>>> >>>>> Maybe that could be updated to check for the basics of the new >>>>> attribute? >>>>> >>>>> >>>>> >>>>> -- Dan >>>>> >>>>> >>>>> >>>>> *From:* 'Jiaming Cheng' via blink-dev <blink-dev@chromium.org> >>>>> *Sent:* Tuesday, April 30, 2024 1:50 PM >>>>> *To:* blink-dev@chromium.org >>>>> *Cc:* Alex Newcomer <newco...@google.com>; Megan Fu < >>>>> megan...@google.com>; Tommy Steimel <stei...@google.com>; Andrew Xu < >>>>> andre...@google.com> >>>>> *Subject:* [blink-dev] Intent to Ship: Support Video Chapter in >>>>> MediaMetadata >>>>> >>>>> >>>>> Contact emails >>>>> >>>>> jiami...@google.com >>>>> >>>>> >>>>> Explainer >>>>> >>>>> https://github.com/w3c/mediasession/pull/308 >>>>> >>>>> >>>>> Specification >>>>> >>>>> https://www.w3.org/TR/mediasession/#the-chapterinformation-interface >>>>> >>>>> >>>>> Summary >>>>> >>>>> The corresponding implementation on the blink layer based on the w3c >>>>> api change, which is to add the `ChapterInformation` attribute in the >>>>> existing `MediaMetadata` See the propose: >>>>> https://github.com/w3c/mediasession/issues/273 >>>>> >>>>> >>>>> >>>>> >>>>> Blink component >>>>> >>>>> Blink>Media>Session >>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EMedia%3ESession> >>>>> >>>>> >>>>> TAG review >>>>> >>>>> None >>>>> >>>>> >>>>> TAG review status >>>>> >>>>> Not applicable >>>>> >>>>> >>>>> Risks >>>>> >>>>> >>>>> >>>>> >>>>> Interoperability and Compatibility >>>>> >>>>> It’s low risk as it's a small addition to an existing API that both >>>>> Gecko and WebKit approve of >>>>> >>>>> >>>>> >>>>> *Gecko*: No signal >>>>> >>>>> *WebKit*: No signal >>>>> >>>>> *Web developers*: No signals >>>>> >>>>> *Other signals*: >>>>> >>>>> >>>>> WebView application risks >>>>> >>>>> *Does this intent deprecate or change behavior of existing APIs, such >>>>> that it has potentially high risk for Android WebView-based applications?* >>>>> >>>>> None >>>>> >>>>> >>>>> >>>>> >>>>> Debuggability >>>>> >>>>> None >>>>> >>>>> >>>>> >>>>> >>>>> Will this feature be supported on all six Blink platforms (Windows, >>>>> Mac, Linux, ChromeOS, Android, and Android WebView)? >>>>> >>>>> No >>>>> >>>>> >>>>> Is this feature fully tested by web-platform-tests >>>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>>>> ? >>>>> >>>>> No >>>>> >>>>> >>>>> Flag name on chrome://flags >>>>> >>>>> MediaSessionChapterInformation >>>>> >>>>> >>>>> Finch feature name >>>>> >>>>> None >>>>> >>>>> >>>>> Non-finch justification >>>>> >>>>> None >>>>> >>>>> >>>>> Requires code in //chrome? >>>>> >>>>> False >>>>> >>>>> >>>>> Sample links >>>>> >>>>> >>>>> https://googlechrome.github.io/samples/media-session/video.html >>>>> >>>>> >>>>> Estimated milestones >>>>> >>>>> Shipping on desktop >>>>> >>>>> 126 >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> Anticipated spec changes >>>>> >>>>> *Open questions about a feature may be a source of future web compat >>>>> or interop issues. Please list open issues (e.g. links to known github >>>>> issues in the project for the feature specification) whose resolution may >>>>> introduce web compat/interop risk (e.g., changing to naming or structure >>>>> of >>>>> the API in a non-backward-compatible way).* >>>>> >>>>> None >>>>> >>>>> >>>>> Link to entry on the Chrome Platform Status >>>>> >>>>> https://chromestatus.com/feature/6682585059295232?gate=5003115407605760 >>>>> >>>>> 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/CABE8buQTGirUaRRsr9ooud9S%3Dg0OquQy6rGy%2BvnrDtT7T%2BqK%2BQ%40mail.gmail.com >>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CABE8buQTGirUaRRsr9ooud9S%3Dg0OquQy6rGy%2BvnrDtT7T%2BqK%2BQ%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/CABE8buRKoe2zoSz5tm98i8yFfDMxDGD-1F051DZwOJr773tv7w%40mail.gmail.com.
