sciancio/searchfirefoxbookmarks — pull request #3 by mathematicalcoffee (2012-11-13 22:26:22+00:00)
Title: added 3.6 and 3.2 support

Hi,

I modified the code to get it working on 3.2, 3.4, and 3.6 with the one version.

I also made some other changes (which you may or may not like) like

* running through jshint
* make the bookmarkFileMonitor variable local to the class (since it's not really a public variable)
* modified the search algorithm because before if a bookmark matched (say) 2 of the search terms, it would turn up twice in the results. Now it only turns up once.

If you don't like these, the bare minimum specific changes to get it working with 3.6 are:

* in getInitialResultSet and getSubsearchResultSet for 3.6 instead of returning the resutls you do

this.searchSystem.pushResults(results);
* in getResultMetas there's an additional parameter `callback` and you're meant to apply it to the search results.