Closed Bug 1566117 Opened 5 years ago Closed 5 years ago

Document various SearchService properties, and switch SearchService._engines to be a Map

Categories

(Firefox :: Search, task, P1)

task
Points:
3

Tracking

()

RESOLVED FIXED
Firefox 70
Iteration:
70.1 - Jul 8 - 21
Tracking Status
firefox70 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(2 files)

SearchService has some undocumented properties that I've been struggling to understand what they are actually for. Now I've understood this, I want to get that documented.

Additionally, I noticed that _engines is really a Map rather than an object, and I think it makes more sense to access it as a Map, as well as making one or two things cleaner. Hence I'm working on a patch for that as well.

Assuming Shane's patch in bug 1556789 re-lands reasonably quickly, I want to land these after that. Hence posting as WIP at the moment, but will update once that lands.

Shane tells me he's taking a break on that patch, so unblocking from that.

No longer depends on: 1556789
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e2118b9a9c2d Change ServiceService._engines to be a map rather than an object. r=daleharvey https://hg.mozilla.org/integration/autoland/rev/693494d80274 Document some of the magic properties of the SearchService. r=daleharvey
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: