1500455 - Let's use cookieBehavior = 4 in beta by default

Status: RESOLVED FIXED

(Core :: DOM: Security, enhancement)

Description

[Andrea Marchesini [:baku]]

This has been discussed during the Default Cookie Restrictions Meeting.
We would like to have 12 weeks of testing in beta in order to see if we have regressions and to have enough time to fix them.

Comment 1

[Andrea Marchesini [:baku]]

Attachment: pref.patch

Comment 2

[(no longer active)]

Comment on attachment 9019439 [details] [diff] [review]
pref.patch

r=me.  Please request uplift too.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/006df234545b
Let's use cookieBehavior = 4 in beta by default, r=ehsan

Comment 4

[Andrea Marchesini [:baku]]

Comment on attachment 9019439 [details] [diff] [review]
pref.patch

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: n/a

User impact if declined: We want to have more data about cookieBehavior 4 and we would like to use the beta population.

Is this code covered by automated tests?: Yes

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: No

If yes, steps to reproduce: n/a

List of other uplifts needed: n/a

Risk to taking this patch: Medium

Why is the change risky/not risky? (and alternatives if risky): Medium because the purpose of this patch is to collect data: we want to collect regressions.

String changes made/needed: n/a

Comment 5

[Andrea Marchesini [:baku]]

Peter, maybe you want to add something in the beta uplift request? Thanks

Comment 6

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/006df234545b

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Just to be clear, we flip EARLY_BETA_OR_EARLIER roughly halfway through the Beta cycle. For 64, it'll likely be after we ship b7 on 6-Nov.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9019439 [details] [diff] [review]
pref.patch

Approved for 64.0b4. I'm also updating the commit message for the uplift to make clear that this is only being changed for early betas. I think that's an important point of clarification and the commit message for m-c is misleading.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/0df77c2cce9f