Backed out changeset c5906d50f330 (bug 1555661) for failures at LocaleService.cpp CLOSED TREE

This commit is contained in:
Mihai Alexandru Michis 2019-05-31 12:54:25 +03:00
parent b7e2ea03ab
commit 8c0942b6e4
9 changed files with 72 additions and 25 deletions

View File

@ -22,4 +22,4 @@
# changes to stick? As of bug 928195, this shouldn't be necessary! Please
# don't change CLOBBER for WebIDL changes any more.
Bug 1555661 - Clobber due to search engine file removals (bug 1552120)
Bug 1551084 - Part 2. Make QCMS transform files use C++. r=miko

View File

@ -0,0 +1,26 @@
{
"extensionName": {
"message": "Google"
},
"extensionDescription": {
"message": "Google Search"
},
"searchUrl": {
"message": "https://www.google.com/search"
},
"searchForm": {
"message": "https://www.google.com/search?q={searchTerms}"
},
"suggestUrl": {
"message": "https://www.google.com/complete/search?client=firefox&q={searchTerms}"
},
"searchUrlGetParams": {
"message": "q={searchTerms}"
},
"param_keyword": {
"message": "firefox-b-1-ab"
},
"param_searchbar": {
"message": "firefox-b-1"
}
}

View File

@ -16,8 +16,5 @@
},
"searchUrlGetParams": {
"message": "client=firefox-b-1-d&q={searchTerms}"
},
"channelPref": {
"message": "google_channel_us"
}
}
}

View File

@ -16,8 +16,5 @@
},
"searchUrlGetParams": {
"message": "client=firefox-b-1-e&q={searchTerms}"
},
"channelPref": {
"message": "google_channel_us"
}
}
}

View File

@ -16,8 +16,5 @@
},
"searchUrlGetParams": {
"message": "client=firefox-b-d&q={searchTerms}"
},
"channelPref": {
"message": "google_channel_row"
}
}
}

View File

@ -16,8 +16,5 @@
},
"searchUrlGetParams": {
"message": "client=firefox-b-e&q={searchTerms}"
},
"channelPref": {
"message": "google_channel_row"
}
}
}

View File

@ -0,0 +1,26 @@
{
"extensionName": {
"message": "Google"
},
"extensionDescription": {
"message": "Google Search"
},
"searchUrl": {
"message": "https://www.google.com/search"
},
"searchForm": {
"message": "https://www.google.com/search?q={searchTerms}"
},
"suggestUrl": {
"message": "https://www.google.com/complete/search?client=firefox&q={searchTerms}"
},
"searchUrlGetParams": {
"message": "q={searchTerms}"
},
"param_keyword": {
"message": "firefox-b-ab"
},
"param_searchbar": {
"message": "firefox-b"
}
}

View File

@ -9,7 +9,7 @@
}
},
"hidden": true,
"default_locale": "b-d",
"default_locale": "en",
"icons": {
"16": "favicon.ico"
},
@ -23,10 +23,22 @@
"search_form": "__MSG_searchForm__",
"suggest_url": "__MSG_suggestUrl__",
"params": [
{
"name": "client",
"condition": "purpose",
"purpose": "keyword",
"value": "__MSG_param_keyword__"
},
{
"name": "client",
"condition": "purpose",
"purpose": "searchbar",
"value": "__MSG_param_searchbar__"
},
{
"name": "channel",
"condition": "pref",
"pref": "__MSG_channel_pref__"
"pref": "google_channel_us"
}
],
"search_url_get_params": "__MSG_searchUrlGetParams__"

View File

@ -403,13 +403,8 @@ class SearchConfigTest {
const code = (typeof rules.codes === "string") ? rules.codes :
rules.codes[purpose];
const submission = engine.getSubmission("test", "text/html", purpose);
const submissionQueryParams = submission.uri.query.split("&");
this.assertOk(submissionQueryParams.includes(code),
this.assertOk(submission.uri.query.split("&").includes(code),
`Expected "${code}" in url "${submission.uri.spec}" from purpose "${purpose}" ${location}`);
const paramName = code.split("=")[0];
this.assertOk(submissionQueryParams.filter(param => param.startsWith(paramName)).length == 1,
`Expected only one "${paramName}" parameter in "${submission.uri.spec}" from purpose "${purpose}" ${location}`);
}
}