Closed Bug 1304830 Opened 3 years ago Closed 3 years ago

Cleanup unused search test cruft

Categories

(Firefox :: Search, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 52
Tracking Status
firefox52 --- fixed

People

(Reporter: mkaply, Unassigned)

Details

Attachments

(1 file)

See:

https://bugzilla.mozilla.org/show_bug.cgi?id=1294201#c27

We have some unused test cruft hanging around. We should remove it.
Comment on attachment 8794356 [details]
Bug 1304830 - Remove unused search test file.

https://reviewboard.mozilla.org/r/80840/#review79674

Thanks!

getSearchMetadata at http://searchfox.org/mozilla-central/rev/c31ad35f39c6187b2e121aa6d3a39b7f67397010/toolkit/components/search/tests/xpcshell/head_search.js#179 is also unused (you can remove it and keep my r+).
Attachment #8794356 - Flags: review?(florian) → review+
https://hg.mozilla.org/mozilla-central/rev/0bbbaab0e95d
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
You need to log in before you can comment on or make changes to this bug.