Closed
Bug 332614
Opened 18 years ago
Closed 18 years ago
Expose livemark annotation names for use in JavaScript
Categories
(Firefox :: Bookmarks & History, enhancement)
Tracking
()
RESOLVED
WONTFIX
Firefox 2 alpha2
People
(Reporter: mozilla, Assigned: mozilla)
Details
Attachments
(1 file)
3.78 KB,
patch
|
annie.sullivan
:
review-
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•18 years ago
|
||
Attachment #217076 -
Flags: review?(annie.sullivan)
Assignee | ||
Updated•18 years ago
|
Target Milestone: --- → Firefox 2 alpha2
Assignee | ||
Updated•18 years ago
|
Status: NEW → ASSIGNED
Comment 2•18 years ago
|
||
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 3•18 years ago
|
||
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-
Assignee | ||
Comment 4•18 years ago
|
||
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
Comment 5•15 years ago
|
||
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.
Description
•