1561734 - Add trackingAnnotationTable to content blocking problem reports

Status: RESOLVED FIXED

(Firefox :: Protections UI, enhancement)

Description

[Arthur Edelstein [:arthur]]

We would like to know which problem reports from Content Blocking are when the user has the strict-list content blocking enabled. So we will need the trackingAnnotationTable pref included in the reports. Please see: https://searchfox.org/mozilla-central/source/browser/base/content/browser-contentblocking.js#990

Comment 1

[(no longer active)]

That is a static pref, see https://searchfox.org/mozilla-central/rev/0671407b7b9e3ec1ba96676758b33316f26887a4/modules/libpref/init/all.js#5224.

The pref you probably want is privacy.annotate_channels.strict_list.enabled.

Comment 2

[(no longer active)]

Attachment: Bug 1561734 - Add privacy.annotate_channels.strict_list.enabled to content blocking problem reports;

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/b3c13d7442a8
Add privacy.annotate_channels.strict_list.enabled to content blocking problem reports; r=ewright

Comment 4

[Oana Pop-Rus]

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

Comment 5

[(no longer active)]

Comment on attachment 9075470 [details]
Bug 1561734 - Add privacy.annotate_channels.strict_list.enabled to content blocking problem reports;

Beta/Release Uplift Approval Request

• User impact if declined: We'd like to uplift this to Beta in order to improve the quality of the content blocking site breakage reports we receive from users, in order to improve the quality of the Enhanced Tracking Protection feature which we hope to enable for all users soon.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): It adds a line to the site breakage report that gets sent when the user uses the respective UI, and has an automated test. It otherwise has no impact at all.
• String changes made/needed: None

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9075470 [details]
Bug 1561734 - Add privacy.annotate_channels.strict_list.enabled to content blocking problem reports;

This was auto-merged to Beta69 already. No need for uplift.
https://hg.mozilla.org/releases/mozilla-beta/rev/b3c13d7442a8

Comment 7

[(no longer active)]

Great, thanks!

Comment 8

[Arthur Edelstein [:arthur]]

Thanks, everyone!