Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Add ui under settings page for controlling exceptions for auto play #16846

Closed
Amejia481 opened this issue Dec 3, 2020 · 4 comments
Closed

Add ui under settings page for controlling exceptions for auto play #16846

Amejia481 opened this issue Dec 3, 2020 · 4 comments
Assignees
Labels
eng:qa:verified QA Verified feature request 🌟 New functionality and improvements Feature:SitePermissions needs:triage Issue needs triage

Comments

@Amejia481
Copy link
Contributor

Amejia481 commented Dec 3, 2020

Video demo

Related issue #11325

image

┆Issue is synchronized with this Jira Task

@Amejia481 Amejia481 added feature request 🌟 New functionality and improvements Feature:SitePermissions labels Dec 3, 2020
@Amejia481 Amejia481 self-assigned this Dec 3, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Dec 3, 2020
@Amejia481 Amejia481 changed the title Add ui under settings for controlling exceptions for auto play Add ui under settings page for controlling exceptions for auto play Dec 10, 2020
Amejia481 added a commit to Amejia481/fenix that referenced this issue Dec 19, 2020
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 13, 2021
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 15, 2021
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 15, 2021
@Amejia481 Amejia481 added the eng:qa:needed QA Needed label Jan 15, 2021
@Amejia481
Copy link
Contributor Author

This feature has landed in master, QA team please help us to verify that is working as expected. Thanks in advance!

@sflorean
Copy link
Contributor

@Amejia481 tested on latest Nightly and when I add a site to exceptions, (using camera, or microphone), all the permissions are displayed: camera, location, microphone, notification, persistent storage, DRM, autoplay, not only the ones that were allowed as described in your screenshot. Is this expected?

@sflorean sflorean removed the eng:qa:needed QA Needed label Jan 18, 2021
@Amejia481
Copy link
Contributor Author

@sflorean thanks for testing :)
Yes, it's expected as we take the default permission values to add the exception :)

@sflorean
Copy link
Contributor

Thank you. Closing as verified.

@sflorean sflorean added the eng:qa:verified QA Verified label Jan 18, 2021
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
eng:qa:verified QA Verified feature request 🌟 New functionality and improvements Feature:SitePermissions needs:triage Issue needs triage
Projects
None yet
Development

No branches or pull requests

2 participants