304784 - Nested menupopups use top-level oncommand instead of its own.

Status: RESOLVED WORKSFORME

(Firefox :: General, defect)

Description

[Matthew Waymost]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20050816 Firefox/1.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20050816 Firefox/1.0+

When dynamically creating menus and submenus using JS, the oncommand attribute
of the lower-level menupopups are not being run to construct the menu. Instead,
the parent menupopup's oncommand attribute is being used. This results in an
endless recursion of the same menu being popped up from the menu.

Reproducible: Always

Steps to Reproduce:
1. Dynamically create nested menupopups that all use oncommand attribute.
2. Attempt to access lower level menus in nested submenu interface.
3.

Actual Results:  
The top-level menu is recursively shown.

Expected Results:  
The correct menu should appear.

Comment 1

[Matthew Waymost]

Attachment: ForwardFork (extension which shows the problem)

Comment 2

[Matthew Waymost]

Attachment: Image showing actual results.

Comment 3

[Matthew Waymost]

(In reply to comment #0)

> Steps to Reproduce:
> 1. Dynamically create nested menupopups that all use oncommand attribute.
> 2. Attempt to access lower level menus in nested submenu interface.
> 3.
> 

I looked back and saw how vague those steps are. Here's a more specific
reproduction method given you have the extension attached installed.

Steps to Reproduce:
1. Open a new tab.
2. Surf to three or four websites. (I usually just open up a forum like at
MozillaZine)
3. Once you have a history of a few sites, go back two or three, and then click
ahead to a different site.
4. With the extension installed, click the forward menu popup and you will see
the recursion.

Comment 4

[Steve England [:stevee]]

Reporter, do you still see this problem with the latest Firefox 2? If not, can you please close this bug as WORKSFORME. Thanks!

Comment 5

[Matthew Waymost]

I haven't been able to reproduce this in 2.0.0.4. Closing as WFM.