Open
Bug 1846461
Opened 1 year ago
Updated 1 year ago
Move QueryFolderChildren, QueryFolderChildrenAsync, ProcessFolderNodeRow to nsNavHistoryResult
Categories
(Toolkit :: Places, task, P3)
Toolkit
Places
Tracking
()
NEW
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.
Reporter | ||
Comment 1•1 year ago
|
||
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.
Description
•