-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
brave://settings has two Widevine settings and it's not obvious what they do #14130
Comments
Per Slack discussion https://bravesoftware.slack.com/archives/C7VLGSR55/p1612981235355800?thread_ts=1612956954.340600&cid=C7VLGSR55 I think we can probably remove the
|
We should keep showing the Widevine prompt unless they have clicked on "Don't show again" checkbox in the prompt. |
…reference from UI Preference itself is still used- but not displayed to user. If they click "Don't show again" and then change their mind, they can enable regular Widevine preference Fixes brave/brave-browser#14130
Verified
Install happy path
Deny-prompt path
Verification passed on
Install happy pathDeny-prompt path
Verification passed on
Install happy pathDeny-prompt path
|
Test plan
Install happy path
Widevine
is disabledWidevine
is enabledDeny prompt path
Widevine
is disabledNever show this again
box and deny accessWidevine
Description
Discovered by @GeetaSarvadnya:
With the recent addition of
Enable Widevine
by @simonhong with brave/brave-core#7554, there are now two toggles:A few ways to resolve:
The text was updated successfully, but these errors were encountered: