Closed Bug 332614 Opened 18 years ago Closed 18 years ago

Expose livemark annotation names for use in JavaScript

Categories

(Firefox :: Bookmarks & History, enhancement)

2.0 Branch
enhancement
Not set
normal

Tracking

()

RESOLVED WONTFIX
Firefox 2 alpha2

People

(Reporter: mozilla, Assigned: mozilla)

Details

Attachments

(1 file)

Currently, the annotation strings used by the live bookmarks service are defined in constants in nsLivemarkService.h.  However, any JS users that want to refer to those annotations would need to redefine those strings to use them.  If we expose those annotation strings through nsILivemarkService, we can avoid having those strings defined in multiple places.
Attachment #217076 - Flags: review?(annie.sullivan)
Target Milestone: --- → Firefox 2 alpha2
Status: NEW → ASSIGNED
I do not think we should do this. I think that the fact that the livemark service uses annotations is an implementation detail that should be hidden. Any information needed by people dealing with livemark features should be exposed through the livemark service API so we can change the annotations in the future. This is bug 332049.
Comment on attachment 217076 [details] [diff] [review]
Patch that adds read-only attributes for the annotation strings


> NS_IMETHODIMP
>+nsLivemarkService::GetFeedURIAnno(nsAString &aAnno)
>+{
>+  aAnno = NS_LITERAL_STRING(LMANNO_FEEDURI);
>+  return NS_OK;
>+}

Use aAnno.AssignLiteral(LMANNO_FEEDURI);
Same for GetSiteURIAnno(), GetFeedExpirationAnno(), and GetItemSourceFeedURIAnno().
Attachment #217076 - Flags: review?(annie.sullivan) → review-
I agree with Brett's comment, so I'm going to WONTFIX this case and add methods to the annotation as described in bug 332617.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → WONTFIX
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: