Closed Bug 1383621 Opened 2 years ago Closed 2 years ago

Remove use of sync bookmarks API from _ensureMobileQuery

Categories

(Firefox :: Sync, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 56
Tracking Status
firefox56 --- fixed

People

(Reporter: markh, Assigned: lina)

References

()

Details

Attachments

(1 file)

The only non-test use of the sync bookmarks API is in _ensureMobileQuery - we should rewrite that using the async API.
Priority: -- → P1
Assignee: nobody → kit
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!
Attachment #8890068 - Flags: review?(markh) → review+
Pushed by kcambridge@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3b56ac96c918
Use the async bookmarks API to insert the mobile query. r=markh
https://hg.mozilla.org/mozilla-central/rev/3b56ac96c918
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
You need to log in before you can comment on or make changes to this bug.