Comment 1 for bug 237941

Revision history for this message
In , Ondrej-allpeers (ondrej-allpeers) wrote :

I can confirm the bug on Windows.

In nsNavHistory.cpp there is a function NeedToFilterResultSet, this returns TRUE for all bookmarks queries and if search term is defined or a folder is specified.

Afterwards all the bookmarks are read from the database (performance problem) and filtered later in FilterResultSet. Inside of this functions is following code:

---8<---

    // exclude-queries is implicit when searching, we're only looking at
    // plan URI nodes
    if (!aSet[nodeIndex]->IsURI())
      continue;

--->8---

This has impact on bug 385245 too. After removal of grouping, usage of maxResults excludes folders from the output.