Closed Bug 1425555 Opened 6 years ago Closed 6 years ago
Avoid a fetch after showing the new bookmark dialog
59 bytes, text/x-review-board-request
In controller.js -> newItem we are currently doing a fetch just after the call to PlacesUIUtils.showBookmarkDialog. Once we remove the sync places transactions (bug 1131491), I think we should look at changing PlacesUIUtils.showBookmarkDialog so that it returns the new/affected guids. newItem can then simply select the new guid. This might not be possible due to how async PlacesTransactions work, but it feels like we should be able to somehow get the new guid to return somehow.
Assignee: nobody → standard8
Priority: P2 → P1
Comment on attachment 8960833 [details] Bug 1425555 - Avoid a database fetch after showing the new bookmark dialog. https://reviewboard.mozilla.org/r/229576/#review235402 it looks good and cheap :)
Attachment #8960833 - Flags: review?(mak77) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/ed517bd510e7 Avoid a database fetch after showing the new bookmark dialog. r=mak
You need to log in before you can comment on or make changes to this bug.