Skip to content
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

Don't load widevine by default on linux #3172

Closed
simonhong opened this issue Jan 31, 2019 · 1 comment
Closed

Don't load widevine by default on linux #3172

simonhong opened this issue Jan 31, 2019 · 1 comment
Assignees
Labels
closed/invalid priority/P2 A bad problem. We might uplift this to the next planned release. privacy

Comments

@simonhong
Copy link
Member

simonhong commented Jan 31, 2019

Followup issue after #413
widevine cdm library should be loaded after user accepts.
Also, enabling widevine by component on linux would be better like other platforms do.
For now, it is loaded by default but not initialized.
Widevine initialization is done after user agrees to use it via content settings bubble.

@simonhong simonhong added this to the 1.x Backlog milestone Jan 31, 2019
@tildelowengrimm tildelowengrimm added privacy priority/P2 A bad problem. We might uplift this to the next planned release. labels Jan 31, 2019
@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@simonhong simonhong self-assigned this Feb 8, 2019
@simonhong
Copy link
Member Author

This issue isn't valid anymore because the PR that ships widevine by default wasn't merged and closed.
Instead, new PR that only downloads and enable widevine after user allowing is in-reviewing.

@rebron rebron added this to the Dupe / Invalid / Not actionable milestone Feb 28, 2019
@bbondy bbondy removed this from the Dupe / Invalid / Not actionable milestone May 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/invalid priority/P2 A bad problem. We might uplift this to the next planned release. privacy
Projects
None yet
Development

No branches or pull requests

4 participants