1383621 - Remove use of sync bookmarks API from _ensureMobileQuery

Status: RESOLVED FIXED

(Firefox :: Sync, enhancement, P1)

Description

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

The only non-test use of the sync bookmarks API is in _ensureMobileQuery - we should rewrite that using the async API.

Comment 1

[Lina Butler (ex-Mozilla)]

Attachment: Bug 1383621 - Use the async bookmarks API to insert the mobile query.

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

Comment 2

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

Comment on attachment 8890068 [details]
Bug 1383621 - Use the async bookmarks API to insert the mobile query.

https://reviewboard.mozilla.org/r/161134/#review167192

Thanks!

Comment 3

[Lina Butler (ex-Mozilla)]

Comment on attachment 8890068 [details]
Bug 1383621 - Use the async bookmarks API to insert the mobile query.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/161134/diff/1-2/

Comment 4

[Lina Butler (ex-Mozilla)]

Comment on attachment 8890068 [details]
Bug 1383621 - Use the async bookmarks API to insert the mobile query.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/161134/diff/2-3/

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3b56ac96c918
Use the async bookmarks API to insert the mobile query. r=markh

Comment 6

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

https://hg.mozilla.org/mozilla-central/rev/3b56ac96c918