1194864 - about:privatebrowsing built-in page links to incorrect SUMO page

Status: VERIFIED FIXED

(Firefox :: Private Browsing, defect, P1)

Description

[Javaun Moradi [:javaun]]

There is a blue "See how this works" button that links to the old SUMO page (TP activated for normal mode via about:config)

The correct link should be the new page: 
https://support.mozilla.org/1/firefox/%VERSION%/%OS%/%LOCALE%/tracking-protection-pbm

Let's edit this and uplift for DevEdition build2. When the UI tour is ready, we'll link to that from the blue button

Comment 1

[Panos Astithas (he/him) [:past] (please ni?)]

Attachment: Patch v1

Simple fix.

Comment 2

[:Paolo Amadini]

Comment on attachment 8650999 [details] [diff] [review]
Patch v1

The link we need to change is the one for the button, while the Learn More link would still link to the current private browsing help page.

I think we'd simply need to change the privacy.trackingprotection.introURL preference instead of using the support base URL for the startTour button, because when the UI tour is ready we'd like to change the preference again to point to the external entry point for the tour rather than a SUMO page.

Comment 3

[Panos Astithas (he/him) [:past] (please ni?)]

Attachment: Patch v2

D'oh! Yes, this makes more sense.

Comment 4

[:Paolo Amadini]

Comment on attachment 8651794 [details] [diff] [review]
Patch v2

Looks good to me, and we can remove the duplication in the preferences file.

Comment 5

[Panos Astithas (he/him) [:past] (please ni?)]

Thanks, I'll remove the first instance in the version that I will land.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/cfc5cf4fdc51

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/cfc5cf4fdc51

Comment 8

[Marco Mucci [:MarcoM]]

Hi Panos, can you set this bug to qe-verify- or qe-verify+

Comment 9

[Paul Silaghi, QA [:pauly]]

Verified fixed FF 43.0a1 (2015-09-10) Win 7