Closed Bug 1661234 Opened 1 year ago Closed 1 year ago

Remove head_search.js useTestEngines in favour of SearchTestUtils.useTestEngines

Categories

(Firefox :: Search, task, P2)

task
Points:
2

Tracking

()

RESOLVED FIXED
82 Branch
Iteration:
82.1 - Aug 24 - Sep 6
Tracking Status
firefox82 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(2 files)

We currently have two implementations of useTestEngines - I think we should just consolidate on the SearchTestUtils version.

Rather than using NetUtils, we can swap to OS.File.read and TextDecoder().

Depends on D88273

Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3a13258fbd51
Remove useTestEngines from head_search.js, use SearchTestUtils version instead. r=daleharvey
https://hg.mozilla.org/integration/autoland/rev/f16918c10a6b
Simplify head_search.js' readJSONFile. r=daleharvey
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d8e8744729e1
Remove useTestEngines from head_search.js, use SearchTestUtils version instead. r=daleharvey
https://hg.mozilla.org/integration/autoland/rev/8da488a82f50
Simplify head_search.js' readJSONFile. r=daleharvey
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in before you can comment on or make changes to this bug.