Remove unused dom.testing.sync-content-blocking-notifications pref

👁 Image	Gregory Pappas [:gregp] Reporter
	Description • 1 year ago

This pref became unused in bug 1599046. It's being set in a bunch of tests, but it doesn't actually do anything. Let's get rid of it

👁 Image	Gregory Pappas [:gregp] Reporter
	Comment 1 • 1 year ago

• modules/libpref/init/StaticPrefList.yaml#4323-4326
• browser/base/content/test/protectionsUI/browser_protectionsUI_icon_state.js#22,29,33
• browser/base/content/test/protectionsUI/browser_protectionsUI_pbmode_exceptions.js#11,18,94
• browser/base/content/test/protectionsUI/browser_protectionsUI_state.js#22,48,284
• browser/base/content/test/protectionsUI/browser_protectionsUI_telemetry.js#6,23,36
• netwerk/cookie/test/mochitest/test_xmlDocument.html#17
• toolkit/components/antitracking/test/browser/browser-blocking.toml#11
• toolkit/components/antitracking/test/browser/browser.toml#10
• toolkit/components/url-classifier/tests/mochitest/test_threathit_report.html#211

👁 Image	tannal2409 Assignee
	Comment 2 • 1 year ago

👁 Image	Phabricator Automation
	Updated • 1 year ago

👁 Image	Pulsebot
	Comment 3 • 1 year ago

Pushed by gp3033@protonmail.com: https://hg.mozilla.org/integration/autoland/rev/be1041192aff Remove unused dom.testing.sync-content-blocking-notifications pref. r=gregp,anti-tracking-reviewers,pbz

👁 Image	Cristina Horotan [:chorotan]
	Comment 4 • 1 year ago
bugherder

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