Open Bug 556437 Opened 11 years ago Updated 5 years ago

nsSearchService.restoreDefaultEngines should also restore the default order


(Firefox :: Search, defect, P5)

3.6 Branch




(Reporter: whimboo, Unassigned)




(Whiteboard: [bug][fxsearch])

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv: Gecko/20100324 Namoroka/3.6.3pre ID:20100324033852

While implementing Mozmill tests for the search component I have noticed that the restoreDefaultEngines method of the SearchService doesn't restore the default order of the engines. There is also no other method available which offers that functionality in an easy way. Only the "Restore Defaults" button inside the Engine Manager resets everything. Means the sorting and hidden status of the engines get reset.

The code live here:

As Gavin told on IRC this logic should better be moved to the SearchService itself.
Points: --- → 2
Flags: qe-verify-
Flags: firefox-backlog+
Summary: nsSearchService.restoreDefaultEngines needs logic from Engine Manager → nsSearchService.restoreDefaultEngines should also restore the default order
defaults get restored and new search UI this should go away.  code in pref panel we believe restore order - but need to restore order.  low/no visible user impact
Rank: 51
Priority: -- → P5
Whiteboard: [bug]
Whiteboard: [bug] → [bug][fxsearch]
You need to log in before you can comment on or make changes to this bug.