Georg Fritzsche [:gfritzsche] Reporter
	Description • 10 years ago

We have two unused plugin infobar preferences: plugins.hide_infobar_for_blocked_plugin & plugins.hide_infobar_for_outdated_plugin.

http://hg.mozilla.org/mozilla-central/annotate/37f08ddaea48/browser/app/profile/firefox.js#l649

We should remove them.

	sriteja
	Comment 1 • 10 years ago

yeah, I am interested to do that..
how can I proceed....???

	sriteja
	Comment 2 • 10 years ago

Since I was new to this ...Can you help me how to proceed...?

	Georg Fritzsche [:gfritzsche] Reporter
	Comment 3 • 10 years ago

There is a general developer guide here: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide
If you don't have yet, you should check out the source & build Firefox: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions

Comment 0 here describes the lines that need to be removed. Once you have done that, it would be good to confirm that the basic browser tests still run through fine using:
> mach mochitest-browser browser/base/content/test/

Then you should create a patch with your changes that you can submit here: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/How_to_Submit_a_Patch

Feel free to contact me via IRC or e-mail if you get stuck.

	vikneshwar Assignee
	Comment 4 • 10 years ago

Hope this looks good , can you assign me this bug . Thank you :)

	Georg Fritzsche [:gfritzsche] Reporter
	Comment 5 • 10 years ago

Sure, thanks for the patch!

	Georg Fritzsche [:gfritzsche] Reporter
	Comment 6 • 10 years ago

Comment on attachment 8443437 [details] [diff] [review]
plugin-pref.patch

This looks fine, thanks. Looking for a peer for an actual r+.

	sriteja
	Comment 7 • 10 years ago

thank you Georg...

	Georg Fritzsche [:gfritzsche] Reporter
	Comment 8 • 10 years ago

And checked in: https://hg.mozilla.org/integration/fx-team/rev/840334f37ae9

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 9 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/840334f37ae9