Make GeckoView get content blocking log from the parent process
Categories
(GeckoView :: General, task, P1)
Tracking
(firefox74 fixed)
Tracking | Status | |
---|---|---|
firefox74 | --- | fixed |
People
(Reporter: dlee, Assigned: timhuang)
References
Details
Attachments
(3 files)
GeckoView accesses content blocking log here via docshell.getContentBlockingLog.
There are two approaches we can take to make this anti-tracking fission compatible:
- Add a IPC message from the child to the parent to get the log.
- GeckoView gets content blocking log directly from the parent.
The better way would be #2 because the log is already in the parent process, so we don't have to involve an IPC call. If #2 is not doable, then we will use #1 approach.
Reporter | ||
Comment 1•4 years ago
|
||
Dylan, needinfo you since you implemented Bug 1580201
We are current moving content blocking log from the child process to the parent process to make anti-tracking fission compatiable. APIs that access content blocking log also need to update accordingly. One thing related to geckoview is the use case here
My question is, for this use case, can GeckoView access ContentBlockingLog directly from the parent process through WindowGlobalParent(will be implemented in Bug 1599043). If not, I'll fix this with approach #1., thx
Comment 2•4 years ago
|
||
(In reply to Dimi Lee [:dimi][:dlee] from comment #1)
Dylan, needinfo you since you implemented Bug 1580201
We are current moving content blocking log from the child process to the parent process to make anti-tracking fission compatiable. APIs that access content blocking log also need to update accordingly. One thing related to geckoview is the use case hereMy question is, for this use case, can GeckoView access ContentBlockingLog directly from the parent process through WindowGlobalParent(will be implemented in Bug 1599043). If not, I'll fix this with approach #1., thx
Yes, I think we can get the WindowGlobalParent
in parent process js -- it will probably actually be an improvement over our current implementation.
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Reporter | ||
Comment 3•4 years ago
|
||
Reporter | ||
Comment 4•4 years ago
|
||
Depends on D56737
Assignee | ||
Comment 5•4 years ago
|
||
This patch is doing two things.
- Make GeckoView directly gets the ContentBlockingLog in the parent
process when it gets the bundle event 'ContentBlocking:RequestLog'. It
will get the top-level browsingContext and get the log from the
WindowGlobal of this browsingContext. - Remove the GeckoViewContentBlockingChild. The child module of
ContentBlocking is no longer needed since it serves nothing after we move
the functionality of getting log to the parent process.
Updated•4 years ago
|
Pushed by dlee@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/06001b30f6cd Part 1: Make GeckoView to get the content blocking log in the parent process. r=geckoview-reviewers,Ehsan,snorp https://hg.mozilla.org/integration/autoland/rev/ec5d3b078a88 Part 2: Remove getContentBlockingLog from nsIDocShell.idl r=Ehsan
Backout by opoprus@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e2677d8ef770 Backed out 2 changesets for gv-junit failures in ContentBlockingControllerTest. on a CLOSED TREE
Comment 8•4 years ago
|
||
Backed out 2 changesets (bug 1601660) for gv-junit failures in ContentBlockingControllerTest
Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=superseded%2Ctestfailed%2Cbusted%2Cexception%2Crunnable&revision=ec5d3b078a88d9733b5a1d82ccc24360811baf65&selectedJob=285929897
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=285929897&repo=autoland&lineNumber=1476
Backout: https://hg.mozilla.org/integration/autoland/rev/e2677d8ef7700cb4b023a64be2dae32145d57329
Pushed by dlee@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/970c96636f40 Part 1: Make GeckoView to get the content blocking log in the parent process. r=geckoview-reviewers,Ehsan,snorp https://hg.mozilla.org/integration/autoland/rev/ebff6fae285e Part 2: Remove getContentBlockingLog from nsIDocShell.idl r=Ehsan
Comment 10•4 years ago
|
||
Pushed by nerli@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bcd450deb288 Fix lint failure r=fix CLOSED TREE
Comment 11•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/970c96636f40
https://hg.mozilla.org/mozilla-central/rev/ebff6fae285e
https://hg.mozilla.org/mozilla-central/rev/bcd450deb288
Assignee | ||
Updated•4 years ago
|
Comment 12•2 years ago
|
||
Move GeckoView::Tracking Protection
bugs to the GeckoView::General
component.
Description
•