349530 - Cannot attach popup and context menu to same XUL element

Status: RESOLVED FIXED

(Core :: XUL, defect)

Description

[Neil Deakin]

The change below is nsXULElement.cpp added in bug 286619 means that both a popup and context menu cannot be applied to the same element. See URL for testcase.

> nsresult
> nsXULElement::AddPopupListener(nsIAtom* aName)
> {
>+    nsCOMPtr<nsIXULPopupListener> popupListener =
>+        NS_STATIC_CAST(nsIXULPopupListener*,
>+                       GetProperty(nsXULAtoms::popuplistener));
>+    if (popupListener) {
>+        // Popup listener is already installed.
>+        return NS_OK;
>+    }

Comment 1

[[e-mail]]

mailWidgets.xml also contains
<content popup="emailAddressPopup" context="emailAddressPopup">

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Attachment: proposed patch.

Sorry, my mistake.
I didn't want to change nsIXULPopupListener interface to use nsIAtoms as popup
types, so just using nsIAtoms as nsINode properties.