1592286 - Investigate whether we can add hooks for the webcompat add-on to modify the position and visibility of the Picture-in-Picture toggle

Status: RESOLVED FIXED

(Toolkit :: Video/Audio Controls, task, P3)

Description

[Mike Conley (:mconley) (:⚙️)]

There are certain sites where the toggle interferes with elements on screen in ways that users might find jarring or unintuitive.

Our hypothesis is that we can special-case these sites using the webcompat add-ons intervention capability. We need to investigate this, and if it's possible, add the hooks necessary to do this special-casing.

Comment 1

[Mike Conley (:mconley) (:⚙️)]

Attachment: Bug 1592286 - Add URL-mapped policy support to the Picture-in-Picture toggle. r?mstriemer

Comment 2

[Dennis Schubert [:denschub]]

The webcompat-addon is not only used in Firefox Desktop, but we also ship the very same code to Fennec and android-components (and the component is already enabled in Fenix), which is why development on that addon does usually not happen in m-c, but on GitHub. It's fine for patches to be submitted against m-c, but we have to keep in mind that the code has to run on more than one platform, unless this is a very short-lived experiment.

This looks fine, but could you please do us the favor of designing the experimental API in a way that just gracefully fails if the APIs in question are not there? aboutConfigPrefs.getPref(), for example, eats all exceptions and returns undefined, resulting in the addon progressing as if nothing ever happened. The other functions are async, resulting in a failed promise, but nothing more serious. pictureInPicture.getAPI() looks like it could be unhappy inside android-components the way it's currently build.

Thank you :)

Comment 3

[Mike Conley (:mconley) (:⚙️)]

Attachment: Bug 1592286 - Add a WebCompat experimental API for mapping MatchPatterns to Picture-in-Picture toggle position policies. r?denschub

Depends on D57177

Comment 4

[Dennis Schubert [:denschub]]

That was fast! Thank you.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/38ad56dbac9b
Add URL-mapped policy support to the Picture-in-Picture toggle. r=mstriemer
https://hg.mozilla.org/mozilla-central/rev/a3701e47b259
Add a WebCompat experimental API for mapping MatchPatterns to Picture-in-Picture toggle position policies. r=denschub

Comment 6

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/38ad56dbac9b
https://hg.mozilla.org/mozilla-central/rev/a3701e47b259