Closed Bug 1215413 Opened 9 years ago Closed 9 years ago

Remove unlisted search plugins from en-US (creativecommons, answers)

Categories

(Firefox :: Search, defect)

38 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 44
Tracking Status
firefox44 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file)

The list of search plugins actually installed for the en-US locale is in browser/locales/en-US/searchplugins/list.txt. It doesn't contain some of the files that can be found in browser/locales/en-US/searchplugins, and as a result, those files are installed when using `mach build faster` because it doesn't rely on that list.txt file. We might as well remove those unlisted search plugins. They can be found in the repository history anyways, if necessary at some point in the future.
Attached patch Remove unlisted search plugins (deleted) — Splinter Review
For reference, they were removed from list.txt in bug 603298.
Assignee: nobody → mh+mozilla
Attachment #8674707 - Flags: review?(dolske)
A few notes: * Searchplugins in browser/locales/en-US/searchplugins can be used by other locales. * ddg is not in list.txt (bug 1105092) but it's shipping in all locales I think you're actually looking at answers.xml and creativecommons.xml, in which case this bug is a dupe of bug 1068456. If someone can confirm bug 1068456 comment 6 from the build side, I think we can remove them.
The code from bug 1073212 is still there and will emit an info message, not an error, for a listed search plugin that doesn't exist anywhere.
Comment on attachment 8674707 [details] [diff] [review] Remove unlisted search plugins Seems fine, but I don't know this code very well (especially wrt comment 2). Drew, maybe?
Attachment #8674707 - Flags: review?(dolske) → review?(adw)
Attachment #8674707 - Flags: review?(adw) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
Summary: Remove unlisted search plugins → Remove unlisted search plugins from en-US (creativecommons, answers)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: