Move new google-no code in list.txt

RESOLVED FIXED in Firefox 48

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mkaply, Unassigned)

Tracking

Trunk
Firefox 48
Points:
---

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

3 years ago
iOS depends on the order of the engines in list.txt, so before the next iOS uplift, google-nocodes should be moved after google:hidden.

(Probably should have done that in the first place).

Not needed for release because Android does not appear to depend on the order in the file.
Does this need to be moco-confidential?
Reporter

Updated

3 years ago
Group: mozilla-employee-confidential
Reporter

Comment 2

3 years ago
Sorry for the weekend review request. Realized I wanted to try to get this in before the merge (since iOS gets search engines from aurora).
Attachment #8744592 - Flags: review?(mark.finkle)
Attachment #8744592 - Flags: review?(margaret.leibovic)
Comment on attachment 8744592 [details] [diff] [review]
Change order of file

Review of attachment 8744592 [details] [diff] [review]:
-----------------------------------------------------------------

Out of curiosity, how exactly does iOS depend on the order here? This current list doesn't reflect our default engine order, nor is it alphabetical, so I'm wondering how this makes a difference.
Attachment #8744592 - Flags: review?(margaret.leibovic) → review+
Reporter

Comment 4

3 years ago
> Out of curiosity, how exactly does iOS depend on the order here? This current list doesn't reflect our default engine order, nor is it alphabetical, so I'm wondering how this makes a difference.

On iOS, the list at the bottom of the app is based directly on the order of list.txt:

https://github.com/mozilla/firefox-ios/blob/01a471f60424224bc8db8f29f01c00cebb99f1ef/Client/Frontend/Browser/SearchEngines.swift#L208

So if I add google-nocodes at the end, it will be on the far right on iOS.

Comment 6

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3cfe3ba1ed21
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Attachment #8744592 - Flags: review?(mark.finkle)
You need to log in before you can comment on or make changes to this bug.