On Thu, Oct 20, 2022, 7:37 AM 'Elad Alon' via blink-dev < blink-dev@chromium.org> wrote:
> Thanks for approving. > May this be a gapless ship? This would show consideration to our OT > partners and their effort. > Per a recent policy change, gapless is always allowed without approvals needed. > On Wednesday, October 19, 2022 at 5:58:25 PM UTC+2 Daniel Bratell wrote: > >> LGTM3 >> >> /Daniel >> >> *Sent with my Thunderbird* >> >> On 2022-10-19 17:55, 'Elad Alon' via blink-dev wrote: >> >> (Ack the nit.) >> >> >> On Wednesday, October 19, 2022 at 5:55:05 PM UTC+2 yoav...@chromium.org >> wrote: >> >>> LGTM2 >>> >>> On Wednesday, October 19, 2022 at 5:53:47 PM UTC+2 Chris Harrelson wrote: >>> >>>> LGTM1 >>>> >>>> On Wed, Oct 19, 2022 at 1:39 AM Yoav Weiss <yoav...@chromium.org> >>>> wrote: >>>> >>>>> OK, so we have WPTs covering the currently testable parts of the API, >>>>> as required by spec and browser tests covering the UA-specific parts. >>>>> Makes >>>>> sense. >>>>> >>>>> >>>>> On Wed, Oct 19, 2022 at 9:36 AM Elad Alon <elad...@google.com> wrote: >>>>> >>>>>> The main reason for partial WPT coverage is that the API is >>>>>> formulated with some key MAY-s, in order to accommodate Safari's >>>>>> different >>>>>> media-picker model. The errors should be the same on all platforms, and >>>>>> are >>>>>> tested. But whether the focus goes to the captured or capturing page is >>>>>> left to the UA to decide, and so cannot be effectively tested by WPT. >>>>>> (Chromium does have browser tests for that.) >>>>>> >>>>>> A strong secondary reason, btw, is the disproportionate difficulty of >>>>>> testing this API reliably using WPT. >>>>>> >>>>> >>>>> Might be worthwhile to file bugs against WPT/TestDriver for issues you >>>>> ran into, or missing features. >>>>> >>>>> >>>>>> >>>>>> On Wednesday, October 19, 2022 at 5:11:52 AM UTC+2 >>>>>> yoav...@chromium.org wrote: >>>>>> >>>>>>> On Thu, Oct 13, 2022 at 10:24 PM 'Elad Alon' via blink-dev < >>>>>>> blin...@chromium.org> wrote: >>>>>>> >>>>>>>> Contact emails elad...@chromium.org >>>>>>>> >>>>>>>> Explainer >>>>>>>> https://github.com/WICG/conditional-focus/blob/main/README.md >>>>>>>> >>>>>>>> Specification >>>>>>>> https://w3c.github.io/mediacapture-screen-share/#dom-capturecontroller-setfocusbehavior >>>>>>>> >>>>>>>> Design docs >>>>>>>> https://docs.google.com/document/d/1LHJRt-ry9hwzFTbPxKrmD0VvtEFEU6lvqsD7k6wwGKM >>>>>>>> (Note that this reflects the old API shape. There is no substantive >>>>>>>> difference; the same functionality is simply exposed on a different >>>>>>>> surface >>>>>>>> now.) >>>>>>>> >>>>>>>> Summary >>>>>>>> >>>>>>>> Extend the getDisplayMedia() API by adding a CaptureController >>>>>>>> object which can be passed in as a parameter. This object exposes a >>>>>>>> setFocusBehavior() method. By calling this method, an app can control >>>>>>>> whether the captured tab/window is focused when capture starts, or >>>>>>>> whether >>>>>>>> the capturing page should retain focus. >>>>>>>> >>>>>>>> >>>>>>>> Blink component Blink >>>>>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink> >>>>>>>> >>>>>>>> TAG review https://github.com/w3ctag/design-reviews/issues/679 >>>>>>>> >>>>>>>> TAG review status Pending >>>>>>>> >>>>>>>> Risks >>>>>>>> >>>>>>>> >>>>>>>> Interoperability and Compatibility >>>>>>>> >>>>>>>> *Gecko*: Positive ( >>>>>>>> https://github.com/mozilla/standards-positions/issues/697) >>>>>>>> Jan-Ivar Bruaroey from Mozilla, and Youenn Fablet from Apple, have both >>>>>>>> collaborated with us closely in shaping the PRs of this API. They have >>>>>>>> then >>>>>>>> approved merging the PRs into w3c/mediacapture-screen-share. This is >>>>>>>> implicit support, so I'd consider it POSITIVE even though, as of the >>>>>>>> time >>>>>>>> of this writing, the official request for position has not yet been >>>>>>>> answered. >>>>>>>> >>>>>>>> *WebKit*: Positive ( >>>>>>>> https://github.com/WebKit/standards-positions/issues/73) Jan-Ivar >>>>>>>> Bruaroey from Mozilla, and Youenn Fablet from Apple, have both >>>>>>>> collaborated >>>>>>>> with us closely in shaping the PRs of this API. They have then approved >>>>>>>> merging the PRs into w3c/mediacapture-screen-share. This is implicit >>>>>>>> support, so I'd consider it POSITIVE even though, as of the time of >>>>>>>> this >>>>>>>> writing, the official request for position has not yet been answered. >>>>>>>> >>>>>>> >>>> Nit: please don't record a result unless it was noted explicitly on the >>>> standards position issue. >>>> >>>> >>>>> >>>>>>>> *Web developers*: Positive ( >>>>>>>> https://github.com/WICG/proposals/issues/37) Support from Web >>>>>>>> developers inside+outside of Google. >>>>>>>> >>>>>>>> *Other signals*: >>>>>>>> >>>>>>>> Ergonomics >>>>>>>> >>>>>>>> N/A >>>>>>>> >>>>>>>> >>>>>>>> Activation >>>>>>>> >>>>>>>> N/A >>>>>>>> >>>>>>>> >>>>>>>> Security >>>>>>>> >>>>>>>> See design-doc. >>>>>>>> >>>>>>>> >>>>>>>> WebView application risks >>>>>>>> >>>>>>>> N/A >>>>>>>> >>>>>>>> >>>>>>>> Debuggability >>>>>>>> >>>>>>>> N/A >>>>>>>> >>>>>>>> >>>>>>>> Will this feature be supported on all six Blink platforms (Windows, >>>>>>>> Mac, Linux, Chrome OS, Android, and Android WebView)? No >>>>>>>> >>>>>>>> Supported on all Desktop platforms. >>>>>>>> >>>>>>>> >>>>>>>> Is this feature fully tested by web-platform-tests >>>>>>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>>>>>>> ? No >>>>>>>> >>>>>>> >>>>>>> Any particular reason? >>>>>>> >>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> Flag name ConditionalFocus >>>>>>>> >>>>>>>> Tracking bug >>>>>>>> https://bugs.chromium.org/p/chromium/issues/detail?id=1215480 >>>>>>>> >>>>>>>> Launch bug >>>>>>>> https://bugs.chromium.org/p/chromium/issues/detail?id=1214483 >>>>>>>> >>>>>>>> Sample links https://wicg.github.io/conditional-focus/demo >>>>>>>> >>>>>>>> Estimated milestones >>>>>>>> Expected full-ship milestone: 109 >>>>>>>> OriginTrial desktop last 108 (or up to and including 111 if delays >>>>>>>> are encountered; already approved >>>>>>>> <https://groups.google.com/a/chromium.org/g/blink-dev/c/CLmSojRQHoc/m/X-AroW-UAQAJ> >>>>>>>> .) >>>>>>>> OriginTrial desktop first 102 >>>>>>>> >>>>>>>> Anticipated spec changes >>>>>>>> >>>>>>>> N/A >>>>>>>> >>>>>>>> >>>>>>>> Link to entry on the Chrome Platform Status >>>>>>>> https://chromestatus.com/feature/5646614535340032 >>>>>>>> >>>>>>>> Links to previous Intent discussions Intent to prototype: >>>>>>>> https://groups.google.com/a/chromium.org/g/blink-dev/c/lbuqOGx07xY >>>>>>>> Intent to Experiment: >>>>>>>> https://groups.google.com/a/chromium.org/g/blink-dev/c/I4RE2pbocTg >>>>>>>> Intent to Extend Experiment: >>>>>>>> https://groups.google.com/a/chromium.org/g/blink-dev/c/CLmSojRQHoc/m/X-AroW-UAQAJ >>>>>>>> >>>>>>>> >>>>>>>> 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+...@chromium.org. >>>>>>>> To view this discussion on the web visit >>>>>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAMO6jDNmAMDVMxEKQRgVtHv7Cx_3%2BiHYWOCBi-VZ9%3DWqX6t5pw%40mail.gmail.com >>>>>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAMO6jDNmAMDVMxEKQRgVtHv7Cx_3%2BiHYWOCBi-VZ9%3DWqX6t5pw%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+...@chromium.org. >>>> >>>> >>>>> To view this discussion on the web visit >>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfWuNj1YwbJyhJU9Hiugn8nCYCKkNGdFfgAyScw7wtfHow%40mail.gmail.com >>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAL5BFfWuNj1YwbJyhJU9Hiugn8nCYCKkNGdFfgAyScw7wtfHow%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+...@chromium.org. >> >> To view this discussion on the web visit >> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/245a180e-eed0-4f93-8008-704cf383b536n%40chromium.org >> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/245a180e-eed0-4f93-8008-704cf383b536n%40chromium.org?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/09b9f36a-9364-42ab-addb-bf09457de92cn%40chromium.org > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/09b9f36a-9364-42ab-addb-bf09457de92cn%40chromium.org?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%2Bw8MOn7v2-S%2B14iDEYpQRFFgagg4ozodwUnv_tMKxZc%3D9g%40mail.gmail.com.