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 <yoavwe...@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 <elada...@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 emailselad...@chromium.org >>>>> >>>>> Explainerhttps://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 componentBlink >>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink> >>>>> >>>>> TAG reviewhttps://github.com/w3ctag/design-reviews/issues/679 >>>>> >>>>> TAG review statusPending >>>>> >>>>> 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 nameConditionalFocus >>>>> >>>>> 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 linkshttps://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 discussionsIntent 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+unsubscr...@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+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/716af50f-8274-4113-9801-55229861340fn%40chromium.org.