Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Create Plugins preference page #7157

Merged
merged 1 commit into from
Feb 15, 2017
Merged

Create Plugins preference page #7157

merged 1 commit into from
Feb 15, 2017

Conversation

cezaraugusto
Copy link
Contributor

@cezaraugusto cezaraugusto commented Feb 10, 2017

Test Plan:

  • Go to preferences
  • Go to new Plugins tab
  • Toggling both Widevine and Flash should work on sites that require them

Details

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

Fixes #7101

@bsclifton
Copy link
Member

#7155 was merged; can you rebase @cezaraugusto? Also, I moved this to 0.13.4 😄

@cezaraugusto
Copy link
Contributor Author

thanks, rebased

Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! This is nice and clean 😄

I verified that going to netflix.com pops up the Widevine CDM as expected and everything works. With regards to Flash though, you missed a spot which links back to preferences (these should be updated to the new about:preferences#plugins route)

https://github.com/cezaraugusto/browser-laptop/blob/a4148b0bf020ab961b0420b1d881c58bf80e9d54/js/contextMenus.js#L477

@cezaraugusto
Copy link
Contributor Author

done, thanks!

Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++++

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants