1676692 - Move contentSearch*UI.* and associated test to browser/components/search (from browser/base/content) and clean up the tests

Status: RESOLVED FIXED

(Firefox :: Search, task, P3)

Description

[Mark Banner (:standard8)]

I think we should move contentSearch*UI.* from browser/base/content to browser/components/search. This is because:

• They are not used by anything in browser/base/content.
• They are used by about:newtab and about:privatebrowsing, however it is normally the search team that looks after them.
• Search is the more logical place to find them.

Whilst I'm in the area, I'm also going to clean up some of the setup for the test - this is part of an experiment for other tests, to see if we can get extensions being installed into mochitests and avoid the opensearch / xml style setup.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1676692 - Move contentSearchUI and related files to browser/components/search. r?daleharvey!

Comment 2

[Mark Banner (:standard8)]

Attachment: Bug 1676692 - Simplify browser_contentSearchUI.js and avoid the need for separate xml search engine definitions. r?daleharvey!

Depends on D96743

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f97e800165b9
Move contentSearchUI and related files to browser/components/search. r=daleharvey
https://hg.mozilla.org/integration/autoland/rev/c1fc1c25afde
Simplify browser_contentSearchUI.js and avoid the need for separate xml search engine definitions. r=daleharvey

Comment 4

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/f97e800165b9
https://hg.mozilla.org/mozilla-central/rev/c1fc1c25afde