39384 - ALT+F beeps as well as drops menu iff no click in content area first

Status: VERIFIED WORKSFORME

(Core :: XUL, defect, P3)

Description

[Sean Richardson]

Overview:
A small regression from the FIX for the mega-DUP-attractor, bug 9701. 
Iff there has been no click to set focus in the content area, the first ALT+F 
beeps as well as dropping down the File menu (of course, the same applies for 
the other menu accelerators).

Steps to Reproduce:
1. Restart Mozilla or invoke a New Navigator Window.
2. Press ALT+F.
3. Click anywhere in the content area (even on a link).
4. Press ALT+F

Actual Results:
In step 2, the computer beeps and the file menu drops down.
In step 4, the file menu drops down.

Expected Results:
In both steps 2 and 4, the file menu drops down, nothing more.

Does not work correctly with: 
the 2000-05-15-08-M16 nightly binary on WinNT.

Works correctly with: 
the M15 release on WinNT.

Additional Information:
No beep if the keying sequence is ALT,F instead (press-and-release ALT before
pressing F).

Trying the steps on a machine with a sound card might help identify the 
type of beep (on Win32, Control Panel->Sounds).

The following comment from bug 9701 might be relevant:
> ------- Additional Comments From Christopher Pratt 1999-08-20 12:58 -------
>
> It's no longer Mac-only. Using the 1999082008 build under NT, try this:
>
> - Launch apprunner
> - Press Alt-F
> 
> Result: The computer beeps.
>
> Expected result: The File menu should drop down.
>
> If you click in the window first to set focus, it does work correctly. 
> Changing platform and OS to All and All.

Comment 1

[Sean Richardson]

Oops. Actually, the beep happens *every* time a menu accelerator keycombo is
used until a click puts focus in the content area -- it's just that the
normal way to dismiss a menu, clicking elsewhere, will normally do just that 
(use ESC or click on the titlebar to avoid it).

Comment 2

[Sean Richardson]

Testing with the 20000622...-M17 nightly binary on WinNT, there are no 
longer any beeps when using a menu accellerator.

Comment 3

[Peter Trudelle]

resolving as wfm

Comment 4

[Sean Richardson]

Yeah, this one is done for dinner (testing with 2000-09-01-04-M17 on WinNT),
but I suppose given the earlier history in the quote from bug 9701, this ought 
to be verified on Mac and Linux as well. Over to you, John.

Comment 5

[Keyser Sose]

Works for me:
Platform: PC
OS: Windows 98
Mozilla Build: 2000101214 M18 Trunk Build

Marking as Verified.