Bug 1596766 - Followup, fix a missed instance in the SearchService of switching an array to a string for the locales change. r=daleharvey
authorMark Banner <standard8@mozilla.com>
Fri, 03 Jan 2020 15:36:00 +0000
changeset 508771 d61646919205abae168cab482603ecb4ddbec2ab
parent 508770 dc16892ca7526c768cf3716288f3fa232a92fc64
child 508772 e0b049ff115f5d4a72c005127c26424c6753b277
push id104191
push usermbanner@mozilla.com
push dateFri, 03 Jan 2020 15:38:51 +0000
treeherderautoland@d61646919205 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdaleharvey
bugs1596766
milestone73.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1596766 - Followup, fix a missed instance in the SearchService of switching an array to a string for the locales change. r=daleharvey Differential Revision: https://phabricator.services.mozilla.com/D58528
toolkit/components/search/SearchService.jsm
--- a/toolkit/components/search/SearchService.jsm
+++ b/toolkit/components/search/SearchService.jsm
@@ -2603,17 +2603,17 @@ SearchService.prototype = {
 
     if ("telemetryId" in config) {
       params.telemetryId = config.telemetryId;
     }
 
     let locale =
       "locale" in config.webExtension
         ? config.webExtension.locale
-        : [SearchUtils.DEFAULT_TAG];
+        : SearchUtils.DEFAULT_TAG;
 
     let manifest = policy.extension.manifest;
     if (locale != "default") {
       manifest = await policy.extension.getLocalizedManifest(locale);
     }
 
     let engineParams = await Services.search.getEngineParams(
       policy.extension,