Closed Bug 1704127 Opened 4 years ago Closed 4 years ago

When opening a native context menu in the bookmarks sidebar, add-on items are missing

Categories

(Core :: Widget: Cocoa, defect)

All
macOS
defect

Tracking

()

VERIFIED FIXED
89 Branch
Tracking Status
firefox89 --- verified

People

(Reporter: mstange, Assigned: mstange)

References

Details

(Whiteboard: [proton-context-menus][mac:mr1])

Attachments

(4 files)

Steps to reproduce:

  1. Enable widget.macos.native-context-menus.
  2. Install the Side View add-on from https://addons.mozilla.org/en-US/firefox/addon/side-view/ .
  3. Open the bookmarks sidebar.
  4. Right-click a bookmark.

Expected results:
The menu should contain an "Open in sidebar" item.

Actual results:
The item is not present. It is only present if widget.macos.native-context-menus is false.

This bug was caught by the test browser/components/extensions/test/browser/browser_ext_contextMenus.js.

The problem here is that native context menus do not set the right x/y values on the popupshowing event. onBookmarksContextMenu uses those values to find the right cell in the tree.

This makes sure that all the correct properties are set, such as the event
position and input mode.

Depends on D111501

Pushed by mstange@themasta.com: https://hg.mozilla.org/integration/autoland/rev/68ecb4413768 Use NS_NewRunnableFunction instead of a custom nsXULPopupShowingEvent class. r=tnikkel https://hg.mozilla.org/integration/autoland/rev/60238814a4bd Rename FirePopupShowingEvent to BeginShowingPopup. r=tnikkel https://hg.mozilla.org/integration/autoland/rev/3a08030ba3a3 Factor out part of BeginShowingPopup into a new FirePopupShowingEvent method. r=tnikkel https://hg.mozilla.org/integration/autoland/rev/b23d2354ba02 Fire the initial popupshowing event for native context menus from nsXULPopupManager, not from nsMenuX. r=tnikkel

Reproduced the issue mentioned in comment 0 using Firefox 89.0a1 (build from 2021-04-09).

This issue is verified fixed using Firefox 89.0a1 (BuildId:20210414160838) on macOS 10.15 with native menus enabled.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: