217964 - Error when pressing letter keys in alert box; (in <button.xml>, |fireAccessKeyButton|)

Status: VERIFIED WORKSFORME

(Core :: XUL, defect)

Description

[Christian Schmidt]

Steps to reproduce:
1. Open the Javascript console
2. Display an alert box by entering "alert('Press D')" in the evalution field.
3. When the alert box opens, press D (or any other alphanumeric key)

The JS console displays the following error:

Error: uncaught exception: [Exception... "Component returned failure code:
0x8000ffff (NS_ERROR_UNEXPECTED) [nsIDOMTreeWalker.nextNode]"  nsresult:
"0x8000ffff (NS_ERROR_UNEXPECTED)"  location: "JS frame ::
chrome://global/content/bindings/button.xml :: fireAccessKeyButton :: line 94" 
data: no]

I know that it serves no purpose to press alphanumeric keys in an alert box, but
 at least it shouldn't generate errors.

I am sing Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5a)
Gecko/20030728 Mozilla Firebird/0.6.1

Comment 1

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.6) Gecko/20040113] (W98SE)

Confirmed:
I tried it on the MailNews "Compact folder ?" dialog:
I pressed 'o' while focus was on 'OK' button. ('O' is not an accesskey there.)
I also tried the |alert()| testcase.

Updating:
*(P) FireBird -> Browser
 *(C) General -> Browser-General
 *(V) unspecified -> Trunk
*(S) normal -> major, exception !
*(K) testcase

Comment 2

[Serge Gautherie (:sgautherie)]

Code is
{{
          var iterator = aSubtree.ownerDocument.createTreeWalker(aSubtree, 
                                                                
NodeFilter.SHOW_ELEMENT, 
                                                                
this.filterButtons, false);
          while (iterator.nextNode()) {     <-- 94
}}

Comment 3

[Serge Gautherie (:sgautherie)]

I made a test:

I added
{{
alert( 'iterator = ' + iterator );
}}
just before the |while()|.

Using the testcase:
0. (beginning of testcase)
1a. I get a dialog with "[object TreeWalker]" :-)
1b. I press 'OK'
2a. I get a dialog with "[object TreeWalker]" (again !) :-|
2b. This time I get the exception :-(
2c. I press 'OK'...
3. (back to end of testcase)

I would seem that the code is executed twice !?

Comment 4

[Christian Schmidt]

WORKSFORME using recent trunk builds of Firefox, Thunderbird and Seamonkey.

Comment 5

[Serge Gautherie (:sgautherie)]

(In reply to comment #4)
> WORKSFORME using recent trunk builds of Firefox, Thunderbird and Seamonkey.

I would have prefered "Fixed" since:

it's fixed in
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b) Gecko/20050217] (<-- 1.8b1 !)
(W98SE)
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b2) Gecko/20050329] (nightly) (W98SE)

but still broken in
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7.7) Gecko/20050414] (release) (W98SE)

Brendan:
I guess this bug does not qualify as "security" to go in the MAS v1.7 branch.
ReOpen if it would.