1313941 - Forward change sources in the livemarks service

Status: RESOLVED FIXED

(Firefox :: Sync, defect, P1)

Description

[Lina Butler (ex-Mozilla)]

Uncovered by a test for bug 1258127.

Comment 1

[Lina Butler (ex-Mozilla)]

Attachment: Bug 1313941 - Forward change sources in the livemarks service.

Review commit: https://reviewboard.mozilla.org/r/89474/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/89474/

Comment 2

[Mark Hammond [:markh] [:mhammond]]

Comment on attachment 8805851 [details]
Bug 1313941 - Forward change sources in the livemarks service.

https://reviewboard.mozilla.org/r/89474/#review89454

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8eb889dd9271
Forward change sources in the livemarks service. r=markh

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/8eb889dd9271

Comment 5

[Lina Butler (ex-Mozilla)]

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.

Comment 6

[Gerry Chang [:gchang]]

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.

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/8b531cab2170