-
Notifications
You must be signed in to change notification settings - Fork 2.6k
[CR 126] Add Safety check section back to brave://settings/privacy #38850
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Safety check will soon be replaced with Safety Hub upstream: https://issues.chromium.org/issues/40277421. Just wanted to make sure that we're still OK with this change, in light of that. cc: @fmarier |
Is this just a rebrand, or is there something else in Safety Hub? Maybe @ShivanKaul has seen this. |
Ok, we'll have to take the "Brave can check your passwords" card because we don't enable that service. |
The above requires |
Verification PASSED on
Using the STR/Cases outlined via brave/brave-core#24162 (comment) & #38850 (comment), ensured that
|
Description
The Safety check section was removed from brave://settings/privacy in 1.66.x with this issue #36859
The Safety check section is needed for the unused permissions check.
https://issues.chromium.org/issues/40232296
Steps to reproduce
Actual result
In 1.66.x:

Expected result
Safety check section and functionality after Tor windows section.
and:
Reproduces how often
Easily reproduced
Brave version (brave://version info)
1.68.x with CR 126
Channel information
Reproducibility
Miscellaneous information
Related issue: #38827 and #38825
cc: @fmarier @LaurenWags
The text was updated successfully, but these errors were encountered: