Dave Hunt [:davehunt] [he/him] ⌚BST Reporter
	Description • 6 years ago

When navigating to a website that has content blocking enabled, Firefox shows a popup to explain the feature. We should disable this when Firefox is launched with Marionette enabled.

The intro panel is shown a maximum number of times, and we can disable it entirely by setting two preferences to indicate that it's already exceeded this number.

The preferences are defined here: https://searchfox.org/mozilla-central/rev/5a18fb5aeeec99f1ca1c36a697082c221189a3b9/browser/base/content/browser-contentblocking.js#168-169

I would suggest setting both "browser.contentblocking.introCount" and "privacy.trackingprotection.introCount" to 99. The current maximum is set to 20.

These should be added in the following places:

https://searchfox.org/mozilla-central/rev/5a18fb5aeeec99f1ca1c36a697082c221189a3b9/testing/marionette/components/marionette.js#54

https://searchfox.org/mozilla-central/rev/5a18fb5aeeec99f1ca1c36a697082c221189a3b9/testing/marionette/client/marionette_driver/geckoinstance.py#33

https://searchfox.org/mozilla-central/rev/5a18fb5aeeec99f1ca1c36a697082c221189a3b9/testing/geckodriver/src/prefs.rs

	Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Comment 2 • 6 years ago

Dave, is this a preference which needs a restart of Firefox, or is it active right away once set?

	Dave Hunt [:davehunt] [he/him] ⌚BST Reporter
	Comment 3 • 6 years ago

(In reply to Henrik Skupin (:whimboo) from comment #2)
> Dave, is this a preference which needs a restart of Firefox, or is it active
> right away once set?

It worked right away for me.

	Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Comment 4 • 6 years ago

In that case we only have to modify those two locations:

https://searchfox.org/mozilla-central/rev/5a18fb5aeeec99f1ca1c36a697082c221189a3b9/testing/marionette/components/marionette.js#54

https://searchfox.org/mozilla-central/rev/5a18fb5aeeec99f1ca1c36a697082c221189a3b9/testing/marionette/client/marionette_driver/geckoinstance.py#504 (DesktopInstance, and not GeckoInstance)

If you don't mind I would take over the mentorship for this bug.

	gurungrahul2
	Comment 5 • 6 years ago

I am working on this bug, currently, I'm reading the https://firefox-source-docs.mozilla.org/testing/marionette/marionette/NewContributors.html , please assign it to me, it's going to be my first bug in Mozilla, it will take a while but this looks do-able to me.

	Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Comment 6 • 6 years ago

Hi gurungrahul2, and thank you for your interest in working on this bug. It's noted, but we generally assign bugs once a first patch has been uploaded. Please let me know when you have questions, or troubles.

	Kajal Kumari Sah (:kajalsah)
	Comment 7 • 6 years ago

Hello :Henrik,

I am a beginner and this looks like a very good bug for me. However :gurungrahul2 is working on it. Please let me know if I am allowed to submit a patch. Otherwise, let me know if :gurungrahul2 is not able to come up with a patch.

	gurungrahul2
	Comment 8 • 6 years ago

(In reply to Kajal Kumari Sah (:kajalsah) from comment #7)
> Hello :Henrik,
> 
> I am a beginner and this looks like a very good bug for me. However
> :gurungrahul2 is working on it. Please let me know if I am allowed to submit
> a patch. Otherwise, let me know if :gurungrahul2 is not able to come up with
> a patch.

I got a little busy on work and academics, have you made any progress ? We can help each other if you have made changes.

	Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Comment 9 • 6 years ago

This is actually kinda nasty when investigating problems on websites which are using all kinds of tracking methods. I will just add those prefs myself.

	Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Comment 10 • 6 years ago

(In reply to Dave Hunt [:davehunt] ⌚️UTC+1 from comment #0)
> I would suggest setting both "browser.contentblocking.introCount" and
> "privacy.trackingprotection.introCount" to 99. The current maximum is set to
> 20.

When implementing this feature I noticed that "privacy.trackingprotection.introCount" doesn't exist anymore. Was this a temporary addition, and got removed again? Not a single result is returned by searchfox when searching for it.

As such this bug will only cover the content blocking introduction panel.

	Pulsebot
	Comment 12 • 6 years ago

Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/86787c8d2a26
[marionette] Disable content blocking introduction panel. r=ato

	Natalia Csoregi [:nataliaCs]
	Comment 13 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/86787c8d2a26

	Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Comment 14 • 6 years ago

Please uplift this test-only patch to mozilla-beta, so we can prevent the introduction panel for people testing real websites.

	Ryan VanderMeulen [:RyanVM]
	Comment 15 • 6 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/5b42d14602aa