227741 - Unable the focus/edit a textfield after focusing popup with JS

Status: NEW

(Core :: DOM: UI Events & Focus Handling, defect)

Description

[Mats Palmgren (inactive)]

This bug is spawned off from bug 227587
Unable the focus/edit a textfield after focusing popup with JS

STEPS TO REPRODUCE:
1. Open the testcase
2. Click on the background in the main window
3. Close the popup window
4. Try to focus/edit the textfields in the main window (does not work).
(focus/edit the textarea works though)

Manually focusing an unrelated application window cures the problem
(at least on Linux).

ACTUAL RESULTS:
It is impossible to focus either of the two textfields in the testcase
using mouse or keyboard.

EXPECTED RESULTS:
focus/edit should work as normal.

PLATFORMS and BUILDS TESTED:
Bug occurs in 2003-11-10-05 nightly trunk build on Linux
Bug occurs in Mozilla 1.5 on Linux
Bug occurs in Mozilla 1.4 on Linux
Bug occurs in Mozilla 1.3.1 on Linux
Bug does NOT occur in Mozilla 1.0.2 on Linux

Comment 1

[Mats Palmgren (inactive)]

Attachment: Testcase

Comment 2

[Boris Zbarsky [:bzbarsky]]

This seems to have broken between 2002-06-11-08 and 2002-06-13-21.  The most
likely culprits are bug 131651 and bug 141900 

Comment 3

[Mats Palmgren (inactive)]

Yes, backing out bug 131651 fixes this bug.  Interestingly enough it does not
seem to regress 131651 either.

Comment 4

[Mats Palmgren (inactive)]

Attachment: Testcase #2

Slightly better testcase for logging - it opens about:mozilla in the popup
instead of about:blank, since about:blank is always loaded anyway...

Comment 5

[Mats Palmgren (inactive)]

Attachment: Trace output (with Tescase #2)

FWIW,  trace data from EventStateManager & FocusController.
The most interesting thing (for me at least ;) was that it shows that separate
threads seem to run through the ESM code in a rather randomly interleaved
fashion (line 65 & 67 etc).  Considering there is bunch of global variables
(gLast*) there just have to be race condition problems in this code...

Comment 6

[Mats Palmgren (inactive)]

Speaking of race conditions... the URL from bug 131651
(http://www.parker.gr/engjotter.html) crashes "viewer" (both with and
without the patch from bug 131651)  -  it does not crash "mozilla" though.

Comment 7

[Boris Zbarsky [:bzbarsky]]

bryner, this is all you...