822203 - Simplify the code for searching places views

Status: REOPENED

(Firefox :: Bookmarks & History, defect, P3)

Description

[Asaf Romano (gone)]

There's some mess going on in places.js wrt. search. For resetting the search box, we rely on the fact that the places tree binding doesn't update the "place" value when tree.load is called directly, so we do something like
  tree.place = tree.place;

Apart the fact that it is quite an ugly hack, there are two problems here:
1. Because of this we cannot optimize the tree setter to avoid reloading everything if the "place" value hasn't changed.
2. The new downloads view doesn't use places search for searching downloads,
   so I had to hack its "place" setter to clear the current search term.

I think we should just introduce a "search" method in the pseudo view interface, that takes care of this.

Comment 1

[BMO Automation]

Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.