Thanks Domenic :] On Thu, May 30, 2024 at 6:28 PM Domenic Denicola <dome...@chromium.org> wrote:
> LGTM1. > > This feature and its spec fits well with the existing MediaMetadata. It > has received a positive position from Mozilla. The TAG review solicited a > good discussion of considered alternatives. > > I agree with Alex that this would have gone better with a proper > explainer, including considered alternatives and example code. But the > feature is simple enough, and thankfully we got a chance to spell out the > reasoning about alternatives in the TAG review, so I think we can proceed. > > On Saturday, May 18, 2024 at 1:53:09 AM UTC+9 Alex Russell wrote: > >> 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 <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/CABE8buRO0uw_Z_tj_nvwYG_Gt5tGG-h0C_qu%3Dv%3DeN6hFfsqj9g%40mail.gmail.com.