Suggestion: Add option to disable notifications

[Jacalz]

• Did you search for similar issues before submitting this one?

• Describe the issue you encountered:

• Platform (Win7, 8, 10? macOS? Linux distro?):

• Brave Version (revision SHA):

• Steps to reproduce:
  1.
  2.
  3.

• Actual result:

• Expected result:

• Will the steps above reproduce in a fresh profile? If not what other info can be added?

• Is this an issue in the currently released version?

• Can this issue be consistently reproduced?

• Extra QA steps:
  1.
  2.
  3.

• Screenshot if needed:

• Any related issues:

[Jacalz]

+1 from community: https://community.brave.com/t/always-deny-notifications/5831/2

[luixxiul]

is it ok to close this in favor of #2215?

[ryanml]

I've laid out the initial work for such a feature on this branch.

Where there will need to be some larger discussion:

1. Naming, Help Text, and Placement within settings for this toggle.
2. Exactly what type of notifications should be covered in this settings' scope. With the work on this branch, all notifications would be disabled should they be switched off. This may be good, but for things like browser restart notifications we probably want to show those regardless. Per Issue Preference to always block website notifications and other requests #2215 it sounds like we may want this to only cover site-specific notifications.

cc: @Jacalz @luixxiul @bridiver @bsclifton

See also:

[bsclifton]

Closing as a duplicate of #12467