cosmetic blocking flag causes shields ad toggle to not work correctly
See original GitHub issueDescription
If you enable the cosmetic blocking flag (you must do this manually on 1.5.x, but it is enabled by default on 1.7.x), and you change the cross site trackers toggle to OFF, ads do not start appearing on web pages as expected. You must toggle shields off entirely.
Note - if you disable cosmetic blocking flag, this is not an issue, the Cross-site trackers toggle works as expected.
Steps to Reproduce
-
If using 1.5.x, enable the brave://flags/#brave-adblock-cosmetic-filtering flag. If using 1.7.x it should be enabled by default.
-
Navigate to a site like slashdot.org --> ads not displayed as expected.
-
Open shields, toggle off
Cross-site trackers blocked
. –> ads are not displayed on the page -
Toggle shields off entirely. –> now ads are displayed on the page
If you disable the brave://flags/#brave-adblock-cosmetic-filtering, then the Cross-site trackers blocked
toggle begins working as expected (after browser restart.
Actual result:
When cosmetic blocking flag is enabled and you toggle Cross-site trackers blocked
OFF, sites do not show ads as expected
Expected result:
Toggling Cross site trackers blocked
to OFF should show ads regardless of the cosmetic blocking flag setting:
Reproduces how often:
easily
Brave version (brave://version info)
Brave | 1.5.111 Chromium: 80.0.3987.132 (Official Build) (64-bit) |
---|---|
Revision | fcea73228632975e052eb90fcf6cd1752d3b42b4-refs/branch-heads/3987@{#974} |
OS | macOS Version 10.14.6 (Build 18G3020) |
Brave | 1.7.58 Chromium: 80.0.3987.132 (Official Build) dev (64-bit) |
---|---|
Revision | fcea73228632975e052eb90fcf6cd1752d3b42b4-refs/branch-heads/3987@{#974} |
OS | macOS Version 10.14.6 (Build 18G3020) |
Version/Channel Information:
- Can you reproduce this issue with the current release? 1.4.x n/a, but yes to 1.5.x
- Can you reproduce this issue with the beta channel? yes
- Can you reproduce this issue with the dev channel? yes
- Can you reproduce this issue with the nightly channel? unsure but probably
Other Additional Information:
- Does the issue resolve itself when disabling Brave Shields? disabling shields entirely is the solution
- Does the issue resolve itself when disabling Brave Rewards?
- Is the issue reproducible on the latest version of Chrome?
Miscellaneous Information:
reproduces on other sites like buzzfeed.com, etc.
Issue Analytics
- State:
- Created 4 years ago
- Comments:8 (5 by maintainers)
Top GitHub Comments
Closing. No longer valid with fix to https://github.com/brave/brave-browser/issues/8475.
Allowing all trackers and ads
in theAdvanced View
of Shields, yields the desired result of showing ads on Slashdot or respecting desire to show ads.Having a separate shields switch for cosmetic filtering would be helpful for debugging purposes if there an issue with a cosmetic filter we could track down more quickly