Remove commented-out code in SearchService.sys.mjs

👁 Image	Stephanie Cunnane [:scunnane] Reporter
	Description • 1 month ago

I noticed this commented-out line was accidentally left in the search service code. This would be a nice "hello world" type bug for a new contributor.

👁 Image	Akeem Omosanya Assignee
	Comment 1 • 1 month ago

Hi, I’m a new contributor and I’d like to work on this bug.
I’ve set up the Firefox source locally and identified the commented-out line in SearchService.sys.mjs. I’m preparing a patch and will submit it for review shortly.

👁 Image	Stephanie Cunnane [:scunnane] Reporter
	Comment 2 • 1 month ago

Sounds great, thank you. You can add me (scunnane) as a reviewer when you submit the patch.

👁 Image	Akeem Omosanya Assignee
	Comment 3 • 1 month ago

👁 Image	Phabricator Automation
	Updated • 1 month ago

👁 Image	Pulsebot
	Comment 4 • 1 month ago

Pushed by scunnane@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/55a1d10f7eea https://hg.mozilla.org/integration/autoland/rev/e779fb77f0bf Remove commented-out code in SearchService.sys.mjs. r=scunnane,search-reviewers DONTBUILD

👁 Image	Sandor Molnar[:smolnar]
	Comment 5 • 1 month ago
bugherder

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

👁 Image	Ardelean Oana, Desktop Test Engineering [:oardelean]
	Updated • 1 month ago