Closed Bug 671292 Opened 14 years ago Closed 8 years ago

Shift Sync bookmarks.js constants etc. into PlacesUtils

Categories

(Toolkit :: Places, defect)

Product:
Toolkit
Component:
Places
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: rnewman, Unassigned)

Details

(Whiteboard: [prune])

E.g., our const definitions of const SIDEBAR_ANNO = "bookmarkProperties/loadInSidebar"; should really live elsewhere. Some are in PlacesUIUtils.jsm.
Marco, what's a good place to put them and make them available for other code to use?
Component: Firefox Sync: Backend → Places
Product: Mozilla Services → Toolkit
QA Contact: sync-backend → places
PlacesUIUtils.jsm should already have those. but it lives in browser so I'm not sure if that's acceptable for Sync (I guess not if Sync may be part of non-browser apps). adding to PlacesUtils is fine, if you change the PlacesUIUtils one to redirect to the PlacesUtils one.
Priority: P4 → --
it's now in PlacesSyncUtils
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.