Thank you Thomas! As far as I'm aware that's all of the paperwork completed, so LGTM1 to disable the APIs by default and at the same time start a reverse origin trial to re-enable them for 6 months. If you hear feedback requesting an extension towards the end of those 6 months, please request an extension for another 6 months.
On Thu, Feb 1, 2024 at 12:43 AM Thomas Guilbert <tguilb...@google.com> wrote: > 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/CAARdPYc_Aqk54G%3DYihQvTbP%3DtnWkMDMSsTxLNYzTjc5JrSgb6A%40mail.gmail.com.