Bug 1312642 - PlacesSyncUtils should pass SOURCE_SYNC to PlacesUtils.bookmarks.reorder. r?kitcambridge
MozReview-Commit-ID: 4EsnU7LSWDG
--- a/toolkit/components/places/PlacesSyncUtils.jsm
+++ b/toolkit/components/places/PlacesSyncUtils.jsm
@@ -123,17 +123,18 @@ const BookmarkSyncUtils = PlacesSyncUtil
}
let parentGuid = BookmarkSyncUtils.syncIdToGuid(parentSyncId);
if (parentGuid == PlacesUtils.bookmarks.rootGuid) {
// Reordering roots doesn't make sense, but Sync will do this on the
// first sync.
return;
}
let orderedChildrenGuids = childSyncIds.map(BookmarkSyncUtils.syncIdToGuid);
- return PlacesUtils.bookmarks.reorder(parentGuid, orderedChildrenGuids);
+ return PlacesUtils.bookmarks.reorder(parentGuid, orderedChildrenGuids,
+ { source: SOURCE_SYNC });
}),
/**
* Removes an item from the database. Options are passed through to
* PlacesUtils.bookmarks.remove.
*/
remove: Task.async(function* (syncId, options = {}) {
let guid = BookmarkSyncUtils.syncIdToGuid(syncId);