Forward change sources in the livemarks service

RESOLVED FIXED in Firefox 51

Status

()

defect
P1
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: lina, Assigned: lina)

Tracking

unspecified
Firefox 52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 unaffected, firefox51 fixed, firefox52 fixed)

Details

Attachments

(1 attachment)

Uncovered by a test for bug 1258127.
Priority: -- → P1
Comment on attachment 8805851 [details]
Bug 1313941 - Forward change sources in the livemarks service.

https://reviewboard.mozilla.org/r/89474/#review89454
Attachment #8805851 - Flags: review?(markh) → review+
Pushed by kcambridge@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8eb889dd9271
Forward change sources in the livemarks service. r=markh
https://hg.mozilla.org/mozilla-central/rev/8eb889dd9271
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
Comment on attachment 8805851 [details]
Bug 1313941 - Forward change sources in the livemarks service.

Approval Request Comment
[Feature/regressing bug #]: Bug 1299338.
[User impact if declined]: Unnecessary syncs when applying livemarks from another device.
[Describe test coverage new/current, TreeHerder]: Covered by existing Places and Sync tests.
[Risks and why]: Low risk; adds an extra change source argument to the livemarks service.
[String/UUID change made/needed]: None.
Attachment #8805851 - Flags: approval-mozilla-aurora?
Comment on attachment 8805851 [details]
Bug 1313941 - Forward change sources in the livemarks service.

Fix an issue related to sync. Take it in 51 aurora.
Attachment #8805851 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.