Skip to content
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

Global Shields toggle for cookies does not work #6804

Closed
jumde opened this issue Nov 6, 2019 · 4 comments · Fixed by brave/brave-core#3903
Closed

Global Shields toggle for cookies does not work #6804

jumde opened this issue Nov 6, 2019 · 4 comments · Fixed by brave/brave-core#3903

Comments

@jumde
Copy link
Contributor

jumde commented Nov 6, 2019

Steps to Reproduce

  1. Toggle global shield settings to block all cookies
  2. Navigate to https://www.whatismybrowser.com/detect/are-third-party-cookies-enabled
  3. You should see No
  4. Re-toggle the global shield settings to block-only-cross-site setting
  5. Navigate to https://www.whatismybrowser.com/detect/are-third-party-cookies-enabled

Test Plan:

Specified here - brave/brave-core#3903

Actual result:

Third party cookies are not blocked

Expected result:

Third party cookies should be blocked

Reproduces how often:

Easily

Brave version (brave://version info)

0.73.60 Chromium: 78.0.3904.87 (Official Build) nightly (64-bit) - Nightly
0.70.122 Chromium: 78.0.3904.87 (Official Build) (64-bit) - Release

@jumde jumde self-assigned this Nov 6, 2019
@bridiver
Copy link
Contributor

bridiver commented Nov 6, 2019

I can't replicate this in 0.70.122

@jumde
Copy link
Contributor Author

jumde commented Nov 6, 2019

Transition from Block all -> Block 3rd parties does not work

@bridiver
Copy link
Contributor

bridiver commented Nov 7, 2019

I was able to replicate that specific transition so it does depend on the exact sequence and other transitions work correctly.

@btlechowski
Copy link

btlechowski commented Nov 28, 2019

Verification passed on

Brave 1.3.38 Chromium: 78.0.3904.108 (Official Build) nightly (64-bit)
Revision 4b26898a39ee037623a72fcfb77279fce0e7d648-refs/branch-heads/3904@{#889}
OS Ubuntu 18.04 LTS

Verified test plan from brave/brave-core#3903

Only block only cross-site cookies
image
Allow all cookies
image

Verified passed with

Brave 1.3.88 Chromium: 79.0.3945.117 (Official Build) beta (64-bit)
Revision 04f0a055010adab4484f7497fbfdbf312c307f1d-refs/branch-heads/3945@{#1019}
OS macOS Version 10.14.6 (Build 18G103)

3rd party cookies blocked:
Screen Shot 2020-01-10 at 1 33 25 PM

all cookies blocked:
Screen Shot 2020-01-10 at 1 33 41 PM

all cookies allowed:
Screen Shot 2020-01-10 at 1 33 55 PM

Verification passed on

Brave 1.3.91 Chromium: 79.0.3945.117 (Official Build) beta (64-bit)
Revision 04f0a055010adab4484f7497fbfdbf312c307f1d-refs/branch-heads/3945@{#1019}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment