search

brave / muon

[DEPRECATED] Build browsers and browser like applications with HTML, CSS, and JavaScript
https://discord.gg/TcT5tX2
MIT License
971 stars 114 forks source link

Issue: 15232 AllowScript should use atom::ContentSettingsManager like other Allow* methods in the observer #651

Closed jumde closed 6 years ago

jumde commented 6 years ago

fix https://github.com/brave/browser-laptop/issues/15232

Test Plan

  1. Open Brave and navigate to https://peltate-worth.000webhostapp.com/world.html
  2. Enable Block Scripts for the domain
  3. Click on Show Message - alert box should not be displayed.
Saloha commented 6 years ago

https://github.com/brave/muon/commit/c18663aa171c6cdf03da3e8c70df8663645b97c4