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/bd448dde-10a9-4391-acfa-05857c0e7263n%40chromium.org.
