make nsIMicrosummaryService use PRInt64 bookmark IDs



12 years ago
3 years ago


(Reporter: myk, Unassigned)


Firefox Tracking Flags

(Not tracked)


(Whiteboard: [microsummaries-feature-removal])



12 years ago
nsIMicrosummaryService currently specifies the data type of bookmark IDs as nsISupports so that the interface can be the same for both RDF-based bookmarks, where IDs are nsIRDFResources, and Places-based bookmarks, where they are currently nsIURIs.

Once Places for bookmarks is enabled by default, by which time Places-based bookmarks will be using PRInt64 IDs, we should change the data type of bookmark IDs in nsIMicrosummaryService to PRInt64 to make it easier to use the interface with Places-based bookmarks.

This bug should depend on a bug about enabling Places-based bookmarks, but I can't find such a bug at the moment.
My immediate plan is to pass place: uris to the service rather than bookmark IDs. Otherwise we'll need to wrap integers as nsISupports-primitives and have a very different code path for each bookmarks-implementation in the service code.

I will post the patch for that on bug 370215, this bug will be used for making the service take simple identifiers once support for legacy-bookmarks is dropped.
Whiteboard: depends on legacy-bookmarks support removal
Target Milestone: Firefox 3 → ---
Wontfixing all Microsummaries bugs, since the feature has been removed from the core product and previous versions won't get further fixes for it.
If interested in supporting Microsummaries in your add-on, you're free to use our old microsummaries code and to search all previously open bugs by looking for [microsummaries-feature-removal] in the status whiteboard field.
Last Resolved: 8 years ago
Resolution: --- → WONTFIX
Whiteboard: depends on legacy-bookmarks support removal → [microsummaries-feature-removal]
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.