1486092 - Make the reject tracker cookie behavior honor the content blocking pref

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement)

Description

[(no longer active)]

This option is being exposed as part of the Content Blocking UI, so we would like to make it honor the same pref as well.

Comment 1

[(no longer active)]

Attachment: Part 1: Move the browser.contentblocking.enabled pref to StaticPrefs

Comment 2

[(no longer active)]

Attachment: Part 2: Make the reject tracker cookie behavior also depend on the browser.contentblocking.enabled pref

The semantics implemented are as follows: the feature requires this new
pref to also be set to true, otherwise cookies from trackers will be
accepted.

Comment 3

[(no longer active)]

Attachment: Part 3: Add tests to ensure that the reject tracker cookie behavior also depends on the browser.contentblocking.enabled pref

The antitracking test mini-framework is extended to run the non-blocking
callback of the tests three times, each time with a unique combination
of the network.cookie.cookieBehavior and browser.contentblocking.enabled
prefs that would cause the reject tracker cookie behavior feature to get
disabled.  This works seamlessly for all test cases that do not make any
assumptions around how many times their callbacks are executed.

Note that browser_imageCache.js depends on the number of times its
non-blocking callback gets executed.  As a result of this, this test
is split into three tests to test the three possible combinations of
the pref for the non-blocking callback.

Comment 4

[(no longer active)]

Attachment: Part 2: Make the reject tracker cookie behavior also depend on the browser.contentblocking.enabled pref

The semantics implemented are as follows: the feature requires this new
pref to also be set to true, otherwise cookies from trackers will be
accepted.

Comment 5

[(no longer active)]

Attachment: Part 3: Add tests to ensure that the reject tracker cookie behavior also depends on the browser.contentblocking.enabled pref

The antitracking test mini-framework is extended to run the non-blocking
callback of the tests three times, each time with a unique combination
of the network.cookie.cookieBehavior and browser.contentblocking.enabled
prefs that would cause the reject tracker cookie behavior feature to get
disabled.  This works seamlessly for all test cases that do not make any
assumptions around how many times their callbacks are executed.

Note that browser_imageCache.js depends on the number of times its
non-blocking callback gets executed.  As a result of this, this test
is split into three tests to test the three possible combinations of
the pref for the non-blocking callback.

Comment 6

[(no longer active)]

Attachment: Part 3: Add tests to ensure that the reject tracker cookie behavior also depends on the browser.contentblocking.enabled pref

The antitracking test mini-framework is extended to run the non-blocking
callback of the tests three times, each time with a unique combination
of the network.cookie.cookieBehavior and browser.contentblocking.enabled
prefs that would cause the reject tracker cookie behavior feature to get
disabled.  This works seamlessly for all test cases that do not make any
assumptions around how many times their callbacks are executed.

Note that browser_imageCache.js depends on the number of times its
non-blocking callback gets executed.  As a result of this, this test
is split into three tests to test the three possible combinations of
the pref for the non-blocking callback.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/537688d78965
Part 1: Move the browser.contentblocking.enabled pref to StaticPrefs; r=baku
https://hg.mozilla.org/integration/mozilla-inbound/rev/b6c0d8e69ddc
Part 2: Make the reject tracker cookie behavior also depend on the browser.contentblocking.enabled pref; r=baku
https://hg.mozilla.org/integration/mozilla-inbound/rev/389b7c456a91
Part 3: Add tests to ensure that the reject tracker cookie behavior also depends on the browser.contentblocking.enabled pref; r=baku

Comment 8

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/537688d78965
https://hg.mozilla.org/mozilla-central/rev/b6c0d8e69ddc
https://hg.mozilla.org/mozilla-central/rev/389b7c456a91