Closed Bug 1626968 Opened 4 years ago Closed 4 years ago

Search engine list for Ja misses engines with Search Modernization on

Categories

(Firefox :: Search, defect, P2)

76 Branch
defect
Points:
2

Tracking

()

VERIFIED FIXED
Iteration:
77.1 - Apr 6 - Apr 19
Tracking Status
firefox-esr68 --- unaffected
firefox74 --- unaffected
firefox75 --- disabled
firefox76 --- verified

People

(Reporter: aflorinescu, Assigned: standard8)

References

Details

Attachments

(2 files)

[Environment:]

tested on:

  • Windows 10

with:

  • 76.0a1 20200402095145
[Steps to reproduce:]
  1. Download and Install a Ja build.
  2. Create a new profile and before starting it, set the region to Us by adding an user.js in the profile folder with the following line: user_pref("browser.search.region", "US");
  3. Start the new profile.
  4. Check the available search engines.
  5. Switch browser.search.modernConfig to false
  6. Restart Browser, check the available search engines.
[Actual result:]
  1. Search displays 8 search engines with browser.search.modernConfig true.
  2. Search displays 10 search engines with browser.search.modernConfig false.
[Expected result:]

Both with browser.search.modernConfig to true or false the number of expected engines, at least at the current date should be 10.

[Note:]
  1. :standard8 hinted this is a typo in the configuration file: "The missing two engines have “jp” rather than “ja” (for the locale they’re deployed to)"
  2. Switching on/off browser.search.modernConfig will keep the engines as described above: 8 for on and 10 for off.

This looks like it's a search bug and so belongs in Firefox::Search, but please correct me if I'm wrong.

Component: Address Bar → Search
Assignee: nobody → standard8
Status: NEW → ASSIGNED
Iteration: --- → 77.1 - Apr 6 - Apr 19
Points: --- → 2
Priority: -- → P2

I have just fixed the typos (they were jp instead of ja) for rakuten and oshiete-goo search engines up for approval on stage/production, so they are now on the preview channels.

I've also run our automated tests against production preview and they pass

Adrian, your team should be able to test these now when you're ready.

Flags: needinfo?(aflorinescu)

Just to note, that the testing should be against production-preview. We're currently use stage for other development.

Production Preview Search engine updates for the missing engines on ja look good for ja-jp and en.Us -jp using Ubuntu 18.04 and Windows 10.
We're gonna follow-up tomorrow with the Mac verification, but it makes no sense to me to block the publish on that alone: redirecting the NI for akw.

Flags: needinfo?(aflorinescu) → needinfo?(standard8)

Thanks, handing off to Mike to get it approved.

Flags: needinfo?(standard8) → needinfo?(mconnor)

Mike has now approved it, the update looks ok on searchengine-devtools.

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Flags: needinfo?(mconnor)
Resolution: --- → FIXED

We can confirm the fix; we verified this issue on Ubuntu 18.04, macOS 10.13 and Windows 10 x64.
Marking the tracking flags accordingly.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: