Closed Bug 84026 Opened 23 years ago Closed 23 years ago

hit alt key and crash

Categories

(Core :: XUL, defect)

x86
Windows 2000
defect
Not set
critical

Tracking

()

VERIFIED DUPLICATE of bug 83974

People

(Reporter: danm.moz, Assigned: hyatt)

Details

With a browser window open, hit the "alt" key, as if you were about to open a 
menu using a keyboard shortcut. Boom (when you let go of the key, or hit another 
key, like alt-f).

Most likely this was introduced in rev 1.74 of nsMenuBarFrame.cpp, which contains 
amusing code like this:

nsMenuBarFrame::FindMenuWithShortcut(PRUint32 aLetter) {
 ...
 GetInsertionPoint(shell, this, nsnull, &immediateParent);
 ...
}
nsMenuBarFrame::GetInsertionPoint(nsIPresShell* aShell, nsIFrame* aFrame,
                nsIFrame* aChild, nsIFrame** aResult) {
 ...
 aChild->GetContent(getter_AddRefs(child));
 ...
}

you see the problem. The call to aChild was added in rev 1.74 to fix bugs 80512 
and bug 83707. The previous rev was simply missing the call. A null check would 
seem to fix this crash, but regress those other bugs.


*** This bug has been marked as a duplicate of 83974 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
vrfy
Status: RESOLVED → VERIFIED
Component: XP Toolkit/Widgets: Menus → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.