Thanks for marking it for review! I submitted a request to review this change to the chromium enterprise mailing list.
Thanks, Thomas On Wed, Jan 31, 2024 at 1:08 PM Mike Taylor <miketa...@chromium.org> wrote: > Yep - seems that's the cause of confusion. In your first email, > https://chromestatus.com/feature/5259513871466496 is linked from the > bottom, so our review tooling is presenting that to us. But I've just > flagged the new one so it will show up as well. > > thanks! > On 1/31/24 2:41 PM, Thomas Guilbert wrote: > > I requested privacy/security/debuggability on the video element > fullscreen API deprecation feature > <https://chromestatus.com/feature/5111638103687168?gate=5204167302053888> last > week. Privacy and debuggability are approved, just waiting on security. > > Mike, are you talking about requesting those gates on the original > Prefixed Fullscreen API feature > <https://chromestatus.com/feature/5259513871466496>? I don't have edit > rights on that Chrome status entry, and upon closer look, it relates to > `webkitRequestFullscreen`, which is not covered by this deprecation intent. > > > [...] requesting enterprise signoff [...] > Is this a field on the chrome status entry? It doesn't show up for me. Or > is this about emailing the list mentioned here > <https://www.chromium.org/developers/enterprise-changes/#1-give-enterprises-visibility> > ? > > Thanks, > Thomas > > > > On Wed, Jan 31, 2024 at 8:54 AM Philip Jägenstedt <foo...@chromium.org> > wrote: > >> Apologies in advance for excessive paperwork, but can you also put >> https://chromestatus.com/feature/5111638103687168 through the process, >> requesting enterprise signoff in particular? Enterprise folks could depend >> on this and might need to take some extra action, and a "Feature >> deprecation" entry is the only way we can flag that. >> >> On Wed, Jan 31, 2024 at 5:44 PM Mike Taylor <miketa...@chromium.org> >> wrote: >> >>> Gentle reminder to follow up on requesting >>> privacy/security/debuggability approvals in chromestatus (which is >>> currently blocking LGTMs). >>> >>> On Wednesday, January 24, 2024 at 7:23:28 AM UTC-5 Mike Taylor wrote: >>> >>>> Would you mind requesting reviews for the various gates (privacy, >>>> security, debuggability) for an OT/DT in your chromestatus entry? >>>> On 1/19/24 10:43 PM, Thomas Guilbert wrote: >>>> >>>> Contact emails >>>> >>>> tguilb...@chromium.org >>>> >>>> Explainer >>>> >>>> None >>>> >>>> Specification >>>> >>>> https://fullscreen.spec.whatwg.org/#dom-document-fullscreenenabled >>>> >>>> Summary >>>> There was an attempt in 2014 >>>> <https://groups.google.com/a/chromium.org/g/blink-dev/c/Bxe7DnDVRZ0/m/5K61HQPrNK4J> >>>> to deprecate and remove the HTMLVideoElement-specific Prefixed Fullscreen >>>> APIs. This meant removing the following APIs from HTMLVideoElement: >>>> >>>> readonly attribute boolean webkitSupportsFullscreen; >>>> readonly attribute boolean webkitDisplayingFullscreen; >>>> void webkitEnterFullscreen(); >>>> void webkitExitFullscreen(); >>>> // Note the different capitalization of the "S" in FullScreen. >>>> void webkitEnterFullScreen(); >>>> void webkitExitFullScreen(); >>>> >>>> The overall usage of these APIs is low, and has trended downwards over >>>> time. Here are the latest usage numbers, as a % of total page loads: >>>> >>>> PrefixedVideoSupportsFullscreen: 0.025% >>>> PrefixedVideoDisplayingFullscreen: 0.082% >>>> PrefixedVideoEnterFullscreen: 0.001% >>>> PrefixedVideoExitFullscreen: 0.010% >>>> PrefixedVideoEnterFullScreen: 0.001% >>>> PrefixedVideoExitFullScreen: 0.000% >>>> >>>> >>>> There has been an unfortunate uptick in the past 2 years for the two >>>> following APIs, which means that it's best to remove them now, before they >>>> see a wider adoption. These numbers might be going up because the prefixed >>>> APIs are also present on iOS. >>>> >>>> https://chromestatus.com/metrics/feature/timeline/popularity/166 >>>> https://chromestatus.com/metrics/feature/timeline/popularity/167 >>>> >>>> There is an alternative set of APIs supported by all browsers that web >>>> authors can use. >>>> >>>> The full history of the removal attempt is here: crbug.com/346236 >>>> >>>> >>>> Goals for experimentation >>>> >>>> The primary goal of the deprecation trial is to reduce the amount of >>>> broken user-visible experiences as the prefixed fullscreen APIs are >>>> removed, and to give time to web authors to transition to the modern API >>>> (which has been available for 5 years). >>>> >>>> >>>> The un-prefixed fullscreen APIs have been available since Chrome M71. >>>> >>>> Experiment timeline >>>> >>>> TBD, with a proposed 3 months duration >>>> >>>> Blink component >>>> >>>> Blink>Fullscreen >>>> Blink>Media>Video >>>> >>>> TAG review >>>> >>>> None >>>> >>>> TAG review status >>>> >>>> Not applicable >>>> >>>> Risks >>>> Interoperability and Compatibility >>>> >>>> Web Compatibility: >>>> >>>> Removing non-standard APIs should overall help web compatibility, and >>>> encourage web authors to use the unprefixed APIs. Some experiences might be >>>> broken by this change, thus justifying this deprecation trial. The API has >>>> been deprecated for a significant amount of time however, and usage has >>>> gone down. >>>> >>>> This would only be an issue for websites that *only* support the >>>> prefixed APIs. >>>> >>>> >>>> Interoperability: >>>> >>>> >>>> All browsers have shipped the new APIs, most of them using an >>>> unprefixed version (Safari on iOS being the only remaining prefixed-only >>>> API). See also >>>> https://developer.mozilla.org/en-US/docs/Web/API/Element/requestFullscreen#browser_compatibility >>>> >>>> >>>> Gecko: >>>> >>>> >>>> WebKit: >>>> >>>> Web developers: >>>> >>>> Other signals: >>>> >>>> Activation >>>> >>>> Impact on the Ads ecosystem: >>>> >>>> N/A >>>> >>>> >>>> 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? >>>> >>>> Potentially. The deprecation trial should give a heads up and >>>> appropriate time for apps to switch over to the unprefixed APIs. >>>> >>>> >>>> >>>> Ongoing technical constraints >>>> >>>> None >>>> >>>> >>>> Debuggability >>>> >>>> N/A >>>> >>>> Will this feature be supported on all six Blink platforms (Windows, >>>> Mac, Linux, Chrome OS, Android, and Android WebView)? >>>> >>>> Yes - the prefixed API will be removed across all platforms. >>>> >>>> Is this feature fully tested by web-platform-tests >>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>>> ? >>>> >>>> Yes >>>> >>>> WPTs testing the prefixes are removed: >>>> https://github.com/web-platform-tests/wpt/blob/master/fullscreen/api/historical.html >>>> >>>> WPTs testing the new API: >>>> https://github.com/web-platform-tests/wpt/tree/master/fullscreen/api >>>> >>>> >>>> Flag name on chrome://flags >>>> >>>> None >>>> >>>> Finch feature name >>>> >>>> PrefixedVideoFullscreen >>>> >>>> Non-finch justification >>>> >>>> None >>>> >>>> Requires code in //chrome? >>>> >>>> False >>>> >>>> Launch bug >>>> >>>> None >>>> >>>> Estimated milestones >>>> >>>> DevTrial on desktop >>>> >>>> 123 >>>> >>>> DevTrial on Android >>>> >>>> 123 >>>> >>>> >>>> Link to entry on the Chrome Platform Status >>>> >>>> https://chromestatus.com/feature/5259513871466496 >>>> >>>> -- >>>> >>>> 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/CABrVPoa373%3Dnxuc%2BTe_h9e0WdS53_oAyUEa%2B4j0v2xWgJ2MFcw%40mail.gmail.com >>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CABrVPoa373%3Dnxuc%2BTe_h9e0WdS53_oAyUEa%2B4j0v2xWgJ2MFcw%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/b1babe80-69c0-4f5e-b6f8-9d6c1ca20d9an%40chromium.org >>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/b1babe80-69c0-4f5e-b6f8-9d6c1ca20d9an%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/CABrVPobzETKd6%2B0XjAMzPXHrTx%2BZ2a9GZBNbaH8fWicE3O7Dgw%40mail.gmail.com.