LGTM3 On Mon, Mar 24, 2025, 7:32 PM Domenic Denicola <dome...@chromium.org> wrote:
> LGTM2, since you are sure future changes will be compatible. And I agree > with Alex; the OT feedback is very persuasive that this API serves web > developers. > > On Tuesday, March 25, 2025 at 3:22:12 AM UTC+9 Alex Russell wrote: > >> Thanks for all the context, Elad. Evidence from OT is persuasive here. >> LGTM1. >> >> On Monday, March 24, 2025 at 2:05:50 AM UTC-7 elad...@google.com wrote: >> >>> The remaining open issues are: >>> >>> 1. >>> >>> Whether the API shape should already support gestures other than >>> wheel, or whether such a future extension should only be addressed in the >>> future. >>> 2. >>> >>> Whether the API should allow concurrent forwarding events from >>> multiple source elements or just a single one. >>> 3. >>> >>> Whether the API should explicitly limit forwarding to the >>> intersection of the source element and a relevant rendering element, or >>> whether this is up to the user agent. >>> 4. >>> >>> Whether the API should be exposed on CaptureController, or exposed >>> on HTMLVideoElement and HTMLCanvasElement. >>> >>> >>> During multiple OT extensions, we have made multiple compromises that >>> have closed other issues. However, when it comes to these 4 issues, our >>> suggested compromise that completely accommodated 1-3, was rejected on the >>> grounds that it does not also fix #4. We do not believe that further >>> progress on these issues is possible at this time. >>> >>> We are confident that any future changes to this API will be compatible >>> with what we are shipping now. >>> >>> >>> On Mon, Mar 24, 2025 at 6:55 AM Domenic Denicola <dome...@chromium.org> >>> wrote: >>> >>>> >>>> >>>> On Friday, March 21, 2025 at 4:42:21 AM UTC+9 elad...@google.com wrote: >>>> >>>> Contact emails >>>> >>>> elada...@chromium.org, gui...@chromium.org >>>> >>>> Explainer >>>> >>>> https://github.com/w3c/mediacapture-surface-control/blob/main/README.md >>>> >>>> Specification >>>> >>>> https://w3c.github.io/mediacapture-surface-control >>>> >>>> Design docs >>>> >>>> https://docs.google.com/document/d/10UojDvTJ6ojBEOP7cgBIIaE7WZEfe >>>> s_Qv1eN3A2A0nM/edit?usp=sharing >>>> >>>> Summary >>>> >>>> A Web API that allows Web applications to: >>>> 1. Forward wheel events to a captured tab. >>>> 2. Read and change the zoom level of a captured tab. >>>> >>>> Blink component >>>> >>>> Blink>GetDisplayMedia >>>> <https://issues.chromium.org/issues?q=customfield1222907:%22Blink%3EGetDisplayMedia%22> >>>> >>>> TAG review >>>> >>>> https://github.com/w3ctag/design-reviews/issues/962 >>>> >>>> TAG review status >>>> >>>> Issues addressed >>>> >>>> Origin Trial Name >>>> >>>> Captured Surface Control >>>> >>>> Chromium Trial Name >>>> >>>> CapturedSurfaceControl >>>> >>>> >>>> WebFeature UseCounter name >>>> >>>> kCapturedSurfaceControl >>>> >>>> Risks >>>> >>>> Interoperability and Compatibility >>>> >>>> Gecko: No signal (https://github.com/mozilla/ >>>> standards-positions/issues/1061) The request for a standards position >>>> has no official position. Both on that issue and in the WebRTC WG, >>>> Mozilla's representative supported the use case; but of the issues he had >>>> filed against the spec, some remain without consensus. >>>> >>>> WebKit: No signal (https://github.com/WebKit/ >>>> standards-positions/issues/388) The request for a standards position >>>> has no official position. In the WebRTC WG, Apple's representative >>>> supported the use case; but of the issues he had filed against the spec, >>>> some remain without consensus. >>>> >>>> >>>> Can you expand on these issues that remain open without consensus, and >>>> how they might impact interoperability and compatibility risks? >>>> >>>> >>>> >>>> Web developers: Strongly positive (https://docs.google.com/ >>>> document/d/1CyKBeo2tGsp_k2HhK6WjDM4dciWEbejUTflKrOVYFv >>>> M/edit?usp=sharing) >>>> >>>> Security >>>> >>>> https://w3c.github.io/mediacapture-surface-control/# >>>> privacy-and-security-considerations >>>> >>>> >>>> WebView application risks >>>> >>>> N/A - this feature is built on top of screen-capture, which is not >>>> currently supported on mobile platforms. >>>> >>>> >>>> Debuggability >>>> >>>> N/A >>>> >>>> >>>> Will this feature be supported on all six Blink platforms (Windows, >>>> Mac, Linux, ChromeOS, Android, and Android WebView)? >>>> >>>> No >>>> >>>> Supported on all desktop platforms. (Screen-sharing is not currently >>>> supported on mobile platforms.) >>>> >>>> >>>> Is this feature fully tested by web-platform-tests >>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>>> ? >>>> >>>> No. Given the interaction with the user, the browser's own UX, and >>>> between the capturing and captured tabs, browser tests produced more >>>> meaningful coverage. Substantial coverage of that form exists over multiple >>>> unit tests and end-to-end browser tests. >>>> >>>> >>>> Flag name on about://flags >>>> >>>> captured-surface-control >>>> >>>> Finch feature name >>>> >>>> CapturedDisplaySurface >>>> >>>> Tracking bug >>>> >>>> https://bugs.chromium.org/p/chromium/issues/detail?id=1466247 >>>> >>>> Launch bug >>>> >>>> https://launch.corp.google.com/launch/4268170 >>>> >>>> Sample links >>>> >>>> https://captured-surface-control.glitch.me >>>> >>>> Estimated milestones >>>> >>>> Origin trial desktop first >>>> >>>> 122 >>>> >>>> Origin trial desktop last >>>> >>>> 127 >>>> >>>> Origin trial extension 1 end milestone >>>> >>>> 135 >>>> >>>> Origin trial extension 2 end milestone >>>> >>>> 132 >>>> >>>> Origin trial extension 3 end milestone >>>> >>>> 129 >>>> >>>> DevTrial on desktop >>>> >>>> 122 >>>> >>>> >>>> Anticipated spec changes >>>> >>>> Additional gestures (in addition to scrolling) may be supported in the >>>> future, if the need is articulated by Web developers, and subject to a >>>> separate privacy/security review of those particular gestures. >>>> >>>> >>>> Link to entry on the Chrome Platform Status >>>> >>>> https://chromestatus.com/feature/5092615678066688?gate=5158502187663360 >>>> >>>> Links to previous Intent discussions >>>> >>>> Intent to Prototype: https://groups.google.com/a/ >>>> chromium.org/d/msgid/blink-dev/CAMO6jDPSgR3kX39drHd9t- >>>> JvTKBk%2B7Dg03O6dvowzw-LjQ__1A%40mail.gmail.com >>>> >>>> Intent to Experiment: https://groups.google.com/a/ >>>> chromium.org/d/msgid/blink-dev/CAOMQ%2Bw-DF_ >>>> BCtdwiaYqrXsan3YNBfT2pAeboT0f1F9wmViPPWQ%40mail.gmail.com >>>> >>>> Intent to Extend Experiment 1: https://groups.google.com/a/ >>>> chromium.org/d/msgid/blink-dev/CAMO6jDN%3DJteucLtF5RU%2Bvx9Fbc_ >>>> CBifdWzjGjtW1Z5zqhcvqgA%40mail.gmail.com >>>> >>>> Intent to Extend Experiment 2: https://groups.google.com/a/ >>>> chromium.org/d/msgid/blink-dev/CAM0wra9vLNpe45swTf1Bgm1ZbH% >>>> 3DhBt0iAjN_FK49Y%3DznEea3vg%40mail.gmail.com >>>> >>>> Intent to Extend Experiment 3: https://groups.google.com/a/ >>>> chromium.org/g/blink-dev/c/7MbIEBdHMck/m/UBbWTX-hAgAJ >>>> >>>> >>>> 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 visit > https://groups.google.com/a/chromium.org/d/msgid/blink-dev/c3a16382-e24a-42c9-9155-4e51905cbad3n%40chromium.org > <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/c3a16382-e24a-42c9-9155-4e51905cbad3n%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 visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw8h322m1G2TxJ%3DCCvx07T3BBC%2Bab3e%3DRZ_6eCnDaYphnw%40mail.gmail.com.