1616775 - Support running ContentBlocking::AllowAccessFor in both parent and child

Status: RESOLVED FIXED

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

Description

[Dimi Lee [:dimi]]

We are going to implement two versions of AddFirstPartyStorageAccessGrantedFor:

• When the parent window and tracker’s window are in the same process, we use the current implementation(check permission in the child) to guarantee permission will be updated synchronously.
• When the parent window and tracker’s window are NOT in the same process, we use the same API but implement the logic in the parent.

See design document for more details.

Comment 1

[Chris Peterson [:cpeterson]]

Tracking for Fission Nightly (M6)

Comment 2

[Chris Peterson [:cpeterson]]

Tentatively moving P1 Fission M6 bugs to M6a.

Comment 3

[Dimi Lee [:dimi]]

Attachment: Bug 1616775 - P1. Add IsThirdPartyWindow and IsThirdPartyTrackingResourceWindow fields to WindowContext. r=timhuang

Comment 4

[Dimi Lee [:dimi]]

Attachment: Bug 1616775 - P2. Support getting principal and tracking origin by BrowsingContext. r=timhuang

Depends on D71010

Comment 5

[Dimi Lee [:dimi]]

Attachment: Bug 1616775 - P3. Add CompleteAllowAccessFor to support running in the parent. r=timhuang

Depends on D71011

Comment 6

[Dimi Lee [:dimi]]

Attachment: Bug 1616775 - P4. Add CompleteAllowAccessFor IPDL r=timhuang

Depends on D71012

Comment 7

[Dimi Lee [:dimi]]

Attachment: Bug 1616775 - P5. Get tracking origin in CompleteAllowAccesFor when it is not available in the child processes. r=timhuang

Depends on D71013

Comment 8

[Dimi Lee [:dimi]]

Attachment: Bug 1616775 - P6. Add OnAllowAccessFor to process tasks that need a parent window. r=timhuang

Depends on D71014

Comment 9

[Dimi Lee [:dimi]]

Attachment: Bug 1616775 - P7. Add IPC for ContentBlockingNotifier::OnDecision r=timhuang

Depends on D71015

Comment 10

[Dimi Lee [:dimi]]

Attachment: Bug 1616775 - P8. Make HasStorageAccessGranted fission-compatible r=timhuang

Depends on D71016

Comment 11

[Dimi Lee [:dimi]]

Attachment: Bug 1616775 - P9. Enable running AllowAccesFor in parent r=timhuang

Depends on D71101

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ae01b3db6250
P1. Add IsThirdPartyWindow and IsThirdPartyTrackingResourceWindow fields to WindowContext. r=timhuang,baku,farre
https://hg.mozilla.org/integration/autoland/rev/414a605d677a
P2. Support getting principal and tracking origin by BrowsingContext. r=timhuang,baku
https://hg.mozilla.org/integration/autoland/rev/123b6c3a0ede
P3. Add CompleteAllowAccessFor to support running in the parent. r=timhuang,baku
https://hg.mozilla.org/integration/autoland/rev/3b1dbe909aaf
P4. Add CompleteAllowAccessFor IPDL r=timhuang,baku
https://hg.mozilla.org/integration/autoland/rev/577f090dae54
P5. Get tracking origin in CompleteAllowAccesFor when it is not available in the child processes.  r=timhuang,baku
https://hg.mozilla.org/integration/autoland/rev/9be5ae499d84
P6. Add OnAllowAccessFor to process tasks that need a parent window. r=timhuang,baku
https://hg.mozilla.org/integration/autoland/rev/312b4abacc2d
P7. Add IPC for ContentBlockingNotifier::OnDecision r=timhuang,baku

Comment 13

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/ae01b3db6250
https://hg.mozilla.org/mozilla-central/rev/414a605d677a
https://hg.mozilla.org/mozilla-central/rev/123b6c3a0ede
https://hg.mozilla.org/mozilla-central/rev/3b1dbe909aaf
https://hg.mozilla.org/mozilla-central/rev/577f090dae54
https://hg.mozilla.org/mozilla-central/rev/9be5ae499d84
https://hg.mozilla.org/mozilla-central/rev/312b4abacc2d