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

Verify removal of CookieListDefaultStudy on main #435

Closed
antonok-edm opened this issue Nov 5, 2022 · 1 comment
Closed

Verify removal of CookieListDefaultStudy on main #435

antonok-edm opened this issue Nov 5, 2022 · 1 comment
Assignees

Comments

@antonok-edm
Copy link
Collaborator

CookieListDefaultStudy is being removed from production in #432. It's already not in main for some reason, but we'd like to double-check that everything works as expected on main before merging #432.

@kjozwiak
Copy link
Member

kjozwiak commented Nov 7, 2022

QA has verified BraveAdblockCookieListOptInReleaseStudy pretty extensively while CookieListDefaultStudy was already removed from main as per the #377 (comment) & #377 (comment). Quickly went through a spot check and ensured that BraveAdblockCookieListOptInReleaseStudy was working as per the following:

Verification PASSED on Win 11 x64 using the following build(s):

Brave | 1.45.118 Chromium: 107.0.5304.91 (Official Build) (64-bit)
-- | --
Revision | 3d5948960d62418160796d5831a4d2d7d6c90fa8-refs/branch-heads/5304@{#1097}
OS | Windows 11 Version 22H2 (Build 22621.755)
Example Example
desktopRelease image

Verification PASSED on Samsung Galaxy S10+ running Android 12 using the following build(s):

Brave | 1.45.120 Chromium: 107.0.5304.91 (Official Build) (64-bit)
--- | ---
Revision | 3d5948960d62418160796d5831a4d2d7d6c90fa8-refs/branch-heads/5304@{#1097}
OS | Android 12; Build/SP1A.210812.016
Example Example Example
Screenshot_20221107-013003_Brave Screenshot_20221107-013043_Brave Screenshot_20221107-013025_Brave

@kjozwiak kjozwiak closed this as completed Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants