1261926 - Better inclusion of bookmarks in awesomebar search results

Status: RESOLVED FIXED

Description

[Richard Newman [:rnewman]]

The first stab at this landed in Bug 1185038.

This adds 0-5 matching bookmarks at the front of the matching history results.

This is missing a few things:

* You can get duplicate bookmarks if they're present multiple times in the DB (e.g., in your Toolbar and in Menu, or bookmarked locally and remotely).
* You can get duplicates between bookmarks and history -- e.g., visit Reddit, bookmark it, search for "reddit", and you'll see two rows.
* History items aren't marked with the star, even if they're not present in the list at the top.

See Bug 1185038 for discussion of these limitations and possibilities for improving them.

Comment 1

[Richard Newman [:rnewman]]

Nomming for prioritization.

Comment 3

[Emily Toop (:fluffyemily)]

Created attachment 8759647 [details] [review]
Pull request

:rnewman - I added you in case you were interested, if you're too busy don't worry.

Comment 4

[Stephan Leroux [:sleroux]]

Comment on attachment 8759647 [details] [review]
Pull request

I _think_ this looks pretty good. Just left a question about the alias. The feature works pretty well on my bookmarks and from what I can tell from the SQL changes to UNION -> UNION ALL and the added JOIN it looks good but my SQL-foo isn't the greatest so I'll defer to Richard if he has any additional comments.

Comment 5

[Emily Toop (:fluffyemily)]

merged behind feature flag https://github.com/mozilla/firefox-ios/commit/72d82db92109676d1b41fd62db32387348be3ef4

Comment 6

[Stephan Leroux [:sleroux]]

v5.x 05006be3a52ad7c2cc4a8ba929f0e942591845f2