Closed Bug 1808666 Opened 2 years ago Closed 2 years ago

Remove placesRoot, toolbarFolder, bookmarksMenuFolder from nsINavBookmarks

Categories

(Toolkit :: Places, task, P3)

task

Tracking

()

RESOLVED FIXED
110 Branch
Tracking Status
firefox110 --- fixed

People

(Reporter: mak, Assigned: mak)

References

(Blocks 1 open bug)

Details

Attachments

(4 files)

These are only used by tests, and in all the cases there should be alternatives using GUIDs.
PlacesUtils.tagsFolderId and tagsFolder are out of scope due to bug 424160.
The scope is reducing the likelihood of introducing more use of itemIds while we're supposed to use GUIDs.

Component: Bookmarks & History → Places
Product: Firefox → Toolkit
Assignee: nobody → mak
Status: NEW → ASSIGNED

Depends on D166274

Depends on D166275

Depends on D166276

Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/33ca921e49f3 Remove placesRoot from nsINavBookmarks. r=Standard8 https://hg.mozilla.org/integration/autoland/rev/783a7e88b6c6 Remove toolbarFolder from nsINavBookmarks. r=Standard8 https://hg.mozilla.org/integration/autoland/rev/6ad1770f9c57 Remove bookmarksMenuFolder from nsINavBookmarks. r=Standard8,markh https://hg.mozilla.org/integration/autoland/rev/daec9223e3c6 Remove no longer necessary folderId accessors. r=Standard8
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: