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.
I can confirm the bug on Windows.
In nsNavHistory.cpp there is a function NeedToFilterRes ultSet, 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 nodeIndex] ->IsURI( ))
// plan URI nodes
if (!aSet[
continue;
--->8---
This has impact on bug 385245 too. After removal of grouping, usage of maxResults excludes folders from the output.