This repository has been archived by the owner on Dec 11, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 972
Allow <site> to <action>? #8826
Labels
Milestone
Comments
@bsclifton - here is the first issue we discussed. |
pretty sure that 'remember this decision' + 'deny' never worked (only 'remember' + 'allow') so this isn't a regression |
diracdeltas
added a commit
that referenced
this issue
May 10, 2017
fix #8826 Test Plan: 1. enable flash 2. go to http://homestarrunner.com/ 3. you should see a Flash notification bar. click 'remember this decision' and 'deny' 4. close the tab and open http://homestarrunner.com/ in a new tab 5. you should not see a Flash notification bar this time
bbondy
pushed a commit
that referenced
this issue
May 19, 2017
fix #8826 Test Plan: 1. enable flash 2. go to http://homestarrunner.com/ 3. you should see a Flash notification bar. click 'remember this decision' and 'deny' 4. close the tab and open http://homestarrunner.com/ in a new tab 5. you should not see a Flash notification bar this time
This was referenced Jun 19, 2017
Closed
94 tasks
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
originally this was about flash, but #8828 fixes that.
however, i am pretty sure than whenever someone answers "No" to an "Allow to " and selects "Remember this decision", then it is never remembered regardless of the choice. i know this is also the case with autoplay.
can we please have a systematic search of the code for all cases and ensure it works right?
Test plan
#8828 (comment)
Did you search for similar issues before submitting this one? yes.
Describe the issue you encountered: "Allow ... to run Flash Player?" with "Remember this decision" doesn't seem
Platform (Win7, 8, 10? macOS? Linux distro?): MacOS
Brave Version (revision SHA): 4e86cab
Steps to reproduce:
Actual result: the setting is not remembered
Expected result: the setting is remembered
Will the steps above reproduce in a fresh profile? yes
Is this an issue in the currently released version? yes
Can this issue be consistently reproduced? yes
Extra QA steps: none
Screenshot if needed: see below
Any related issues: none
The text was updated successfully, but these errors were encountered: