1706604 - Implement URL query string stripping for navigations

Status: RESOLVED FIXED

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

Description

[Tim Huang[:timhuang]]

Basic idea is that we will strip certain parts of the query string before we create the channel for navigation.

Comment 1

[Tim Huang[:timhuang]]

Attachment: Bug 1706604 - Part 1: Implement the URLQueryStringStripper. r?dimi

This patch implements a URLQueryStringStripper which provides a static
function Strip() to strip the target query parameters. The function
accepts a URI and reutrn the stripped URI.

Depends on D113841

Comment 2

[Tim Huang[:timhuang]]

Attachment: Bug 1706604 - Part 2: Strip the URI before create the channel in nsDocShell. r?smaug

The target query parameters will be stripped from the loading URI in
nsDocShell before creating the channel for the loading. We will only
strip the URI for top-level loading but not for frames. It's because
there is no need to strip frames if we've stripped in top-level already.

Depends on D113842

Comment 3

[Tim Huang[:timhuang]]

Attachment: Bug 1706604 - Part 3: Strip the target query parameters in BrowsingContext::LoadURI(). r?nika

We strip the target query parameters in BrowsingContext::LoadURI(). This
is needed for the loading from the RemoteWebNavigation. The parent
process will create the channel before the loading happens in the
nsDocShell in content process. So we have to strip before the channel
has been created in the parent process.

Depends on D113843

Comment 4

[Tim Huang[:timhuang]]

Attachment: Bug 1706604 - Part 4: Add tests to test the query stripping. r?dimi

Depends on D113844

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cce6d3f75b62
Part 1: Implement the URLQueryStringStripper. r=dimi
https://hg.mozilla.org/integration/autoland/rev/a7d285e73387
Part 2: Strip the URI before create the channel in nsDocShell. r=smaug
https://hg.mozilla.org/integration/autoland/rev/5509494fddc7
Part 3: Strip the target query parameters in BrowsingContext::LoadURI(). r=nika
https://hg.mozilla.org/integration/autoland/rev/2a9e4619bcd0
Part 4: Add tests to test the query stripping. r=dimi,smaug

Comment 6

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/cce6d3f75b62
https://hg.mozilla.org/mozilla-central/rev/a7d285e73387
https://hg.mozilla.org/mozilla-central/rev/5509494fddc7
https://hg.mozilla.org/mozilla-central/rev/2a9e4619bcd0