Mike, do you have information on what this can be expected to improve performance wise? It's not clear to me what the perf impact of this can be...

All the bookmarks UI is currently using the old synchronous transactions for every change. Basically, adding/removing/editing/moving one or more bookmarks go through this. All of those operations cause main-thread I/O. The new async transactions are expected to solve that problem (once the dependencies here are fixed), moving I/O off the main thread through the new Bookmarks API.

See comment 2. :)

Once the remaining deps are landed, I'll trigger a try build with this patch as well, and just confirm everything is still green. Then we'll land and see how it goes :-)

Comment on attachment 8886116 [details] Bug 1071513 - Enable async PlacesTransactions for nightly builds. https://reviewboard.mozilla.org/r/156918/#review162468 r=me once we have a green Try (may be worth retriggering all mochitest-browser tests at least once). We must coordinate with QA at the next team meeting to create a test plan for letting this go to release in 57.

Try build with all the latest changes is here (based on autoland as that's where they all are): https://treeherder.mozilla.org/#/jobs?repo=try&revision=92d0675c90d26b0856c4cded8093db36e8c4b427

Fresh try push with bug 1381027 included: https://treeherder.mozilla.org/#/jobs?repo=try&revision=0c837a938564f86772abc9b3a8ba54403902d12b

Looks like we need to resolve bug 1336806 or a variant of it before we can push this.

Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0a4690dfd7b3 Enable async PlacesTransactions for nightly builds. r=mak