Closed
Bug 1215413
Opened 9 years ago
Closed 9 years ago
Remove unlisted search plugins from en-US (creativecommons, answers)
Categories
(Firefox :: Search, defect)
Tracking
()
RESOLVED
FIXED
Firefox 44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(1 file)
(deleted),
patch
|
adw
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•9 years ago
|
||
For reference, they were removed from list.txt in bug 603298.
Assignee: nobody → mh+mozilla
Attachment #8674707 -
Flags: review?(dolske)
Comment 2•9 years ago
|
||
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.
Assignee | ||
Comment 3•9 years ago
|
||
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 4•9 years ago
|
||
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)
Updated•9 years ago
|
Attachment #8674707 -
Flags: review?(adw) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
Updated•9 years ago
|
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.
Description
•