Closed Bug 1596860 Opened 5 years ago Closed 5 years ago

If the region and/or locale is unknown, ensure the modern search config falls back to a reasonable default.

Categories

(Firefox :: Search, task, P2)

task
Points:
5

Tracking

()

RESOLVED FIXED
Firefox 74
Iteration:
74.1 - Jan 6 - Jan 19
Tracking Status
firefox74 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(2 files)

I'm not sure if there's an explicit test for this, but we should make sure that if we've not got a region, we're falling back to a sensible default.

Blocks: 1542269

It appears we already have handling for "default" for region - though we're not testing for it in the search config tests, so I'll add that.

The other bit that is missing is making sure that if we don't have a locale, then we fall back to a sensible set of engines. The good thing with the new configuration is that it is more region based so even if there's a locale we don't recognise, then we will have a minimal set that's reasonable. The minor downside is that it is not so explicit as to what is default in that case.

Hence I'll be adding a test for the locale case as well, and possibly some other fixes.

Assignee: nobody → standard8
Status: NEW → ASSIGNED
Iteration: --- → 73.1 - Dec 2 - Dec 15
Points: --- → 3
Summary: If the region is unknown, ensure the modern search config falls back to a reasonable default. → If the region and/or locale is unknown, ensure the modern search config falls back to a reasonable default.
Iteration: 73.1 - Dec 2 - Dec 15 → 74.1 - Jan 6 - Jan 19
Points: 3 → 5

Also adds tests for checking the search engine lists when falling back to a default region.

Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/18b48dfaae65 Remove default region setting for various search engines in the modern config as it is different to the legacy config. r=daleharvey https://hg.mozilla.org/integration/autoland/rev/a77944eb37ca Add search engine configuration tests for unknown locales. r=daleharvey
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 74
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: