Sorry for the slow reply here. Glad to see this works for Audio too. This might have been a bit more obvious of there were an explainer in the usual format. I expect the TAG will ask for one of those too. Please pay particular attention to considered alternatives and example code, both for the proposed design and for discarded alternatives.
https://w3ctag.org/explainers/ Best, Alex On Wednesday, May 15, 2024 at 12:14:18 PM UTC-7 Jiaming Cheng wrote: > 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 <[email protected]> > 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 <[email protected]> >>> 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 <[email protected]> >>>> 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 [email protected] >>>>> 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 <[email protected]> >>>>>> *Sent:* Tuesday, April 30, 2024 1:50 PM >>>>>> *To:* [email protected] >>>>>> *Cc:* Alex Newcomer <[email protected]>; Megan Fu < >>>>>> [email protected]>; Tommy Steimel <[email protected]>; Andrew Xu < >>>>>> [email protected]> >>>>>> *Subject:* [blink-dev] Intent to Ship: Support Video Chapter in >>>>>> MediaMetadata >>>>>> >>>>>> >>>>>> Contact emails >>>>>> >>>>>> [email protected] >>>>>> >>>>>> >>>>>> 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 [email protected]. >>>>>> 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 [email protected]. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/ccf33370-ace1-413f-9a35-852045dd736en%40chromium.org.
