Bug 1817201 Comment 5 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### Beta/Release Uplift Approval Request
* **User impact if declined**: Those enrolled in the cookie banner handling experiment as well as users foxfooding CBH may see inconsistent banner handling behavior between standard and private browsing modes
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: 1. Navigate to about:preferences#privacy
2. Scroll down to "cookie banner reduction"
3. Check (or uncheck) "Reduce Cookie Banners"
4. in about:config, verify that the checkbox correctly toggles both cookiebanners.service.mode and cookiebanners.service.mode.privateBrowsing to the same value (0=unchecked, 1=checked)
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Fixes a minor bug for an unreleased feature
* **String changes made/needed**: 
* **Is Android affected?**: No
### Beta/Release Uplift Approval Request
* **User impact if declined**: Those enrolled in the cookie banner handling experiment as well as users foxfooding CBH may see inconsistent banner handling behavior between standard and private browsing modes
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: 

 1. Navigate to `about:preferences#privacy`
 2. Scroll down to "cookie banner reduction"
 3. Check (or uncheck) "Reduce Cookie Banners"
 4. in `about:config`, verify that the checkbox correctly toggles both `cookiebanners.service.mode` and `cookiebanners.service.mode.privateBrowsing`  to the same value (`0`=unchecked, `1`=checked)

* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Fixes a minor bug for an unreleased feature
* **String changes made/needed**: 
* **Is Android affected?**: No

Back to Bug 1817201 Comment 5