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

Allow <site> to <action>? #8826

Closed
mrose17 opened this issue May 10, 2017 · 2 comments · Fixed by #8828
Closed

Allow <site> to <action>? #8826

mrose17 opened this issue May 10, 2017 · 2 comments · Fixed by #8828

Comments

@mrose17
Copy link
Member

mrose17 commented May 10, 2017

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:

    1. go to a site that will cause the "Allow ... to run Flash Player?" notification
    2. click on "Remember this decision"
    3. click on "Deny"
  • 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

1
2

@mrose17 mrose17 added the bug label May 10, 2017
@mrose17
Copy link
Member Author

mrose17 commented May 10, 2017

@bsclifton - here is the first issue we discussed.

@diracdeltas
Copy link
Member

pretty sure that 'remember this decision' + 'deny' never worked (only 'remember' + 'allow') so this isn't a regression

@diracdeltas diracdeltas self-assigned this May 10, 2017
@diracdeltas diracdeltas added this to the 0.15.400 milestone May 10, 2017
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
@mrose17 mrose17 closed this as completed May 12, 2017
@mrose17 mrose17 reopened this May 12, 2017
@mrose17 mrose17 changed the title Allow ... to run Flash Player? Allow <site> to <action>? May 12, 2017
@diracdeltas diracdeltas removed their assignment May 18, 2017
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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants