Closed Bug 1250806 Opened 5 years ago Closed 5 years ago

5 most recently added bookmarks are not available in the Menu bar - Bookmarks on Mac OS X


(Firefox :: Bookmarks & History, defect)

47 Branch
Not set



Firefox 47
Tracking Status
firefox44 --- unaffected
firefox45 --- unaffected
firefox46 --- unaffected
firefox47 --- verified
firefox48 --- verified


(Reporter: mboldan, Assigned: dao)




(1 file)

[Affected versions]:
- Firefox 47.0a1 (2016-02-23)

[Affected platforms]:
- Mac OS X 10.9.5, Mac OS X 10.11

[Steps to reproduce]:
1. Launch Firefox.
2. Bookmark a few pages.
3. Go to Menu Bar -> Bookmarks and observe the Recently Bookmarked section.

[Expected result]:
- The last 5 bookmarked pages are displayed in the Recently Bookmarked section.

[Actual result]:
- Nothing is displayed in the Recently Bookmarked section.

[Regression range]:
- I will verify if this is a regression ASAP.

[Additional notes]:
- This issue is not reproducible on Windows or on Ubuntu platforms.
Is this with a new profile? If this is indeed plain broken, I'm a little surprised that nobody else had reported this yet.
The tests were performed with a clean profile. Only the Recently Bookmark title is displayed in the '5 most recently added bookmarks' section.
This is not a regression since the issue is reproducible on Firefox 47.0a1 (2016-02-14), build where 'Recently Bookmarked' section was implemented.
Marco, any idea what's going on here? We're adding the menuitems in the popupshowing event handler. AFAIK this should just work™ on OS X.
Flags: needinfo?(mak77)
Oh wait, we're actually adding the menuitems asynchronously using asyncExecuteLegacyQueries. I guess this ends up being too late. I thought the native menubar would support live updates, but maybe not. How do we usually deal with this in places menus?
no the native menubar only updates content in onpopupshowing, unfortunately.
We deal with it by populating before it shows... That doesn't work very well with async stuff, indeed we still have an open bug (that atm I cannot find) to add live update support to the native menus...
Flags: needinfo?(mak77)
I think we only update the menu contents here
and when there is new content, we set a flag that tell to rebuild the menu at the next opening.
(In reply to Marco Bonardo [::mak] from comment #6)
> no the native menubar only updates content in onpopupshowing, unfortunately.
> We deal with it by populating before it shows...

How exactly? Should I use something other than asyncExecuteLegacyQueries or listen to some other popup event?
Flags: needinfo?(mak77)
The queries populating the menubar ATM are synchronous (just PlacesUtils.history.executeQuery), that is not very nice for jank, but probably the only solution for the native menubar, unless someone fixes live updating.
Flags: needinfo?(mak77)
Attached patch patchSplinter Review
Assignee: nobody → dao
Attachment #8725620 - Flags: review?(mak77)
Comment on attachment 8725620 [details] [diff] [review]

Review of attachment 8725620 [details] [diff] [review]:

I didn't check on Mac, but it should work off-hand.
Attachment #8725620 - Flags: review?(mak77) → review+
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 47
Flags: qe-verify+
Depends on: 1253930
The 5 most recently added bookmarks are correctly displayed on Firefox 47.0a1 (2016-03-07)and on Firefox 48.0a1 (2016-03-08).
The tests were performed on Mac OS X 10.9.5 and on Mac OS x 10.11.1.
I am marking this issue Verified-Fixed.
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.