1616570 - Syncing CookieSettings to the WindowContext from Document.

Status: RESOLVED FIXED

(Core :: Privacy: Anti-Tracking, task, P1)

Description

[Tim Huang[:timhuang]]

As part of the ETP-Fission work, we need to make the CookieSettings to be accessible through the browsingContext. So, we can access it across process boundaries.

Comment 2

[Chris Peterson [:cpeterson]]

Tracking for Fission Nightly (M6)

Comment 3

[Tim Huang[:timhuang]]

Attachment: Bug 1616570 - Part 1: Rename CookieSettings to CookieJarSettings. r?Ehsan

Given that we are going to add ContentBlockingAllowList in
CookieSettings, so CookieSettings will be responsible for more stuff than the
cookie behavior and cookie permission. We should use a proper name to
reflect the purpose of it. The name 'CookieSettings' is misleading that
this is only for cookie related stuff. So, we decide to rename
'CookieSettins' to 'CookieJarSettings' which serves better meaning here.

Comment 4

[Tim Huang[:timhuang]]

Attachment: Bug 1616570 - Part 2: Add the CookieJarSettingsArgs as a field in the window context. r?Ehsan,farre

This patch adds a new field CookieJarSettings in the browsing context.
This makes the CookieJarSettings can be accssible across the process
boundaries, which is needed for the work of making the ETP
Fission-compatible.

Depends on D63935

Comment 5

[Tim Huang[:timhuang]]

Attachment: Bug 1616570 - Part 4: Sync the CookieBehavior of the CookieJarSettings into the WindowContext. r?Ehsan

This patch makes the cookie jar settings to be populated in the browsing
context. The cookie jar settings will be synced to the browsing context
as it has been set in the loadInfo.

Depends on D63937

Comment 6

[Tim Huang[:timhuang]]

Attachment: Bug 1616570 - Part 3: Change the CookieJarSettings::CreateBlockingAll to be a singleton and rename it to GetBlockingAll(). r?Ehsan

We don't need to create the blocking-all cookieJarSettings every time we
request it since it is basically the same. Instead, we should make it as
a singleton.

The patch makes this change the rename the
CookieJarSettings::CreateBlockingAll() to GetBlockingAll() which
reflects the fact that it is a singleton.

Depends on D63937

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7a18a0d2e114
Part 1: Rename CookieSettings to CookieJarSettings. r=Ehsan
https://hg.mozilla.org/integration/autoland/rev/3855ea76bc29
Part 2: Add the CookieJarSettingsArgs as a field in the window context. r=Ehsan,farre
https://hg.mozilla.org/integration/autoland/rev/1bad5855c84f
Part 3: Change the CookieJarSettings::CreateBlockingAll to be a singleton and rename it to GetBlockingAll(). r=Ehsan
https://hg.mozilla.org/integration/autoland/rev/073bf81ba3cd
Part 4: Sync the CookieBehavior of the CookieJarSettings into the WindowContext. r=Ehsan

Comment 8

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/7a18a0d2e114
https://hg.mozilla.org/mozilla-central/rev/3855ea76bc29
https://hg.mozilla.org/mozilla-central/rev/1bad5855c84f
https://hg.mozilla.org/mozilla-central/rev/073bf81ba3cd