1547114 - Simplify nsCookiePermission

Status: RESOLVED FIXED

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

Comment 1

[(no longer active)]

Attachment: Bug 1547114 - Part 1: Remove nsICookiePermission::CanAccessURI();

Comment 2

[(no longer active)]

Attachment: Bug 1547114 - Part 2: Read the cookie lifetime policy from nsContentUtils;

Comment 3

[(no longer active)]

Attachment: Bug 1547114 - Part 3: Remove nsCookiePermission::mThirdPartyUtil;

Comment 4

[(no longer active)]

Attachment: Bug 1547114 - Part 4: Mark nsICookiePermission as non-scriptable;

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fc4e5936eeae
Part 1: Remove nsICookiePermission::CanAccessURI(); r=baku
https://hg.mozilla.org/integration/autoland/rev/c92df939a4d6
Part 2: Read the cookie lifetime policy from nsContentUtils; r=baku
https://hg.mozilla.org/integration/autoland/rev/0a4180b074ea
Part 3: Remove nsCookiePermission::mThirdPartyUtil; r=baku
https://hg.mozilla.org/integration/autoland/rev/4167117f548f
Part 4: Mark nsICookiePermission as non-scriptable; r=baku

Comment 6

[(no longer active)]

Attachment: Bug 1547114 - Part 5: Remove nsICookiePermission::CanAccess();

Comment 7

[(no longer active)]

Attachment: Bug 1547114 - Part 6: Convert network.cookie.lifetimePolicy into a static pref;

Comment 8

[(no longer active)]

Attachment: Bug 1547114 - Part 7: Get rid of nsCookieModule.h/cpp;

Comment 9

[Andreea Pavel [:apavel]]

Backed out for multiple failures e.g. xpcshell test_SitePermissions.js

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&selectedJob=242878527&revision=4167117f548f41a9755a6710fc543674c8b2cc6b

failure logs:
xpcshell: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=242894533&repo=autoland&lineNumber=2193
bc: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=242893023&repo=autoland&lineNumber=3319
mochitest: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=242894304&repo=autoland&lineNumber=1963

Backout: https://hg.mozilla.org/integration/autoland/rev/f13945ab88b10a0ab3fbbef21d667d30caebba8a

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8cfaa18c3b3b
Part 1: Remove nsICookiePermission::CanAccessURI(); r=baku
https://hg.mozilla.org/integration/autoland/rev/405b90db2106
Part 2: Read the cookie lifetime policy from nsContentUtils; r=baku
https://hg.mozilla.org/integration/autoland/rev/14ad847bd477
Part 3: Remove nsCookiePermission::mThirdPartyUtil; r=baku

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e28229a014fa
Part 5: Remove nsICookiePermission::CanAccess(); r=baku
https://hg.mozilla.org/integration/autoland/rev/246b613845d3
Part 6: Convert network.cookie.lifetimePolicy into a static pref; r=baku
https://hg.mozilla.org/integration/autoland/rev/fb9c7644874e
Part 7: Get rid of nsCookieModule.h/cpp; r=baku

Comment 12

[(no longer active)]

Attachment: Bug 1547114 - Part 8: Move nsCookiePermission to netwerk/cookie;

Comment 13

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/8cfaa18c3b3b
https://hg.mozilla.org/mozilla-central/rev/405b90db2106
https://hg.mozilla.org/mozilla-central/rev/14ad847bd477

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/52b13dded51f
Part 8: Move nsCookiePermission to netwerk/cookie; r=baku

Comment 15

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/e28229a014fa
https://hg.mozilla.org/mozilla-central/rev/246b613845d3
https://hg.mozilla.org/mozilla-central/rev/fb9c7644874e
https://hg.mozilla.org/mozilla-central/rev/52b13dded51f