Open Bug 1846461 Opened 1 year ago Updated 1 year ago

Move QueryFolderChildren, QueryFolderChildrenAsync, ProcessFolderNodeRow to nsNavHistoryResult

Categories

(Toolkit :: Places, task, P3)

task

Tracking

()

People

(Reporter: mak, Unassigned)

Details

These methods are only used by nsNavHistoryResult so it doesn't make sense to have them in the Bookmarks service.
Long term (after tags are converted) the Bookmarks service should be merged into renamed nsNavHistory anyway.

Methods that need a db connection can be moved into nsNavHistory, others into nsNavHistoryResult... we can eval.

Code renames/cleanups are welcome while doing this.

Moving ResultNodeForContainer to nsNavHistory would also be fine, along with the GUID version of FetchItemInfo() that is only used there.

You need to log in before you can comment on or make changes to this bug.