Closed
Bug 1601894
Opened 5 years ago
Closed 5 years ago
Compare list.json + ABSearch with modern config and figure out if the differences need to be addressed
Categories
(Firefox :: Search, task, P2)
Firefox
Search
Tracking
()
Tracking | Status | |
---|---|---|
firefox73 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
Currently, the modern configuration is based entirely off list.json. At some stage, we should compare the differences when ABSearch has been applied to list.json, and check if there's anything we should address before we ship the modern config.
For example, with the modern config eBay is currently available to most en-US locales, however, ABSearch turns it off for RO and others.
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → standard8
Assignee | ||
Comment 1•5 years ago
|
||
Depends on D57682
Assignee | ||
Comment 2•5 years ago
|
||
Some more notes here:
- I downloaded the ABSearch data from the live server for each locale/region combination that FF currently supports.
- I then did a xpcshell-test that used some of the current
searchconfig
test functions to load and reload the engines for different locale/regions using the engine selector. - This then compared the identifiers of the engines with the list from ABSearch.
- Generally, most things matched:
be
locale has some differences, due to ABSearch having to support older engines which we don't now.uk
locale has some differences, but matches the current list.json which is the best thing to do.zh-TW
has the extrareadmoo
extension.en-US
has some differences in the shipped eBay domain (.com instead of the region specific one) - especially on some of the overrides. The patch here fixes those.
Generally though, we're good with matching, so once these have landed, we should be good now.
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f3b42c0737c7
Ensure the search engine configuration tests check the details for each locale/region pair, and fix eBay setup in the modern configuration. r=mikedeboer
Comment 4•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox73:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 73
Assignee | ||
Updated•5 years ago
|
Iteration: --- → 73.2 - Dec 16 - Jan 5
You need to log in
before you can comment on or make changes to this bug.
Description
•