Open Bug 1424079 Opened 7 years ago Updated 2 years ago

Convert _maybeToggleBookmarkToolbarVisibility to use async Places APIs

Categories

(Firefox :: Migration, enhancement, P3)

enhancement

Tracking

()

People

(Reporter: MattN, Unassigned)

References

()

Details

In bug 1415692, code was brought back to show the bookmarks toolbar for new/reset profiles if there are a certain number of bookmarks imported/restored to the user's bookmark toolbar. :mak would like the Places API usage to be converted to use async APIs. That wasn't done in bug 1415692 since the code needs to run before the first window shows (to get the toolbar state correct) and fixing that would add more risk for uplift. Two possible approaches: a) make the migrateUI method async and have it wait for the Places API usage. ** This won't cause any UI flicker as the toolbar will be toggled before the window opens still. b) Let the calculation of bookmark toolbar items be async but broadcast to all windows to show the bookmark toolbar if required in open windows. See the "autoshow-bookmarks-toolbar" (I could have sworn there was another that also showed the toolbar but I could be mistaken). Relevant code: https://hg.mozilla.org/mozilla-central/diff/49358c06a122/browser/components/nsBrowserGlue.js
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.