1492005 - Unship prefixed Fullscreen API

Status: REOPENED

(Core :: DOM: Core & HTML, task, P2)

Description

[Xidorn Quan [:xidorn] UTC+11]

See https://hg.mozilla.org/mozilla-central/rev/04c9910699f7 for previous deprecation. The code is still usable.

We may want to do this only after bug 1269276 reaches release.

Comment 1

[Xidorn Quan [:xidorn] UTC+11]

We can probably also put the prefixed API behind a pref and try disabling it on Nightly in this bug.

Comment 2

[Kagami Rosylight [:saschanaz] (they/them)]

Do we need a use counter here? I have a strong feeling that the data would be largely from feature detection.

Comment 3

[Anne (:annevk)]

I share your worry that it might be hard to count something productive here. Even if we counted successful fullscreen attempts from these methods, that would not tell us whether the user would still have gone fullscreen if we did not support them.

However, given that the prefixed variants have been around for a long time, I do think we want to be somewhat cautious with removing support for them. I think we should at least have console warnings for a couple releases. Comment 1 also seems like a good idea (and the preference will make it easier to revert later) and I think we could do that at the same time as the warnings.

An intent to unship that outlines the steps to be taken also seems good in case others have useful input on that plan.