Closed Bug 1312642 Opened 3 years ago Closed 3 years ago

PlacesSyncUtils should pass SOURCE_SYNC to PlacesUtils.bookmarks.reorder

Categories

(Firefox :: Sync, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 52
Tracking Status
firefox50 --- unaffected
firefox51 --- fixed
firefox52 --- fixed

People

(Reporter: markh, Assigned: markh)

References

Details

Attachments

(2 files)

Otherwise our onItemMove listener doesn't ignore the move.
Assignee: nobody → markh
Status: NEW → ASSIGNED
Comment on attachment 8804121 [details]
Bug 1312642 - PlacesSyncUtils should pass SOURCE_SYNC to PlacesUtils.bookmarks.reorder.

https://reviewboard.mozilla.org/r/88246/#review87322
Attachment #8804121 - Flags: review?(kcambridge) → review+
Blocks: 1299338
See Also: → 1312857
Pushed by mhammond@skippinet.com.au:
https://hg.mozilla.org/integration/autoland/rev/49a1469f39a3
PlacesSyncUtils should pass SOURCE_SYNC to PlacesUtils.bookmarks.reorder. r=kitcambridge
Comment on attachment 8804121 [details]
Bug 1312642 - PlacesSyncUtils should pass SOURCE_SYNC to PlacesUtils.bookmarks.reorder.

Approval Request Comment
[Feature/regressing bug #]: Bug 1299338.
[User impact if declined]: Unnecessary bookmark syncs when syncing folders from another device.
[Describe test coverage new/current, TreeHerder]: Test in bug 1312857.
[Risks and why]: Low risk. One-line change that adds an optional argument.
[String/UUID change made/needed]: None.
Attachment #8804121 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/49a1469f39a3
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Comment on attachment 8804121 [details]
Bug 1312642 - PlacesSyncUtils should pass SOURCE_SYNC to PlacesUtils.bookmarks.reorder.

Fix bookmark syncing issue. Take it in 51 aurora.
Attachment #8804121 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
has problems uplifting to aurora

grafting 371544:49a1469f39a3 "Bug 1312642 - PlacesSyncUtils should pass SOURCE_SYNC to PlacesUtils.bookmarks.reorder. r=kitcambridge"
merging toolkit/components/places/PlacesSyncUtils.jsm
warning: conflicts while merging toolkit/components/places/PlacesSyncUtils.jsm! (edit, then use 'hg resolve --mark')
abort: unresolved conflicts, can't continue
(use 'hg resolve' and 'hg graft --continue')
Tomcats-MacBook-Pro-2:mozilla-central Tomcat$
Flags: needinfo?(markh)
Flags: needinfo?(markh)
You need to log in before you can comment on or make changes to this bug.