192454 - Window is focused again if onClick='window.open' & href='#foo'(anchor)

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[HARUNAGA Hirotoshi]

<a href="#foo" onClick="window.open();"> causes opening a new window
and focusing the old window that opened the new window.

<a href="http://foo" onClick="window.open();"> causes opening a new
window and focusing the new window that was opened, like as before,
like as IE/NN4.

Steps:
1. open the URL.
2. click <a href="#anchor" onClick="window.open('http://www.mozilla.org/');">.

Actual:
Old window is focused again.

Expected:
New window (http://www.mozilla.org/) should be focused, like as before,
like as IE/NN4.

Confirming with 2003020323/FreeBSD.
2003010701/FreeBSD doesn't have this problem, 2003011500/FreeBSD and
2003012508/WinNT have.
2003013003-MachO/MacOSX has no problem
This problem began to occur between Jan 6 and Jan 15 on PC.
Regression.

Comment 1

[Boris Zbarsky [:bzbarsky]]

This works in build 2003-01-06-21 on Linux.  It's broken in 2003-01-07-21...

Nothing in there looks suspicious; he closest thing is bryner's guievent checkin....

Comment 2

[John Keiser (jkeiser)]

aaronl's keydead patch (bug 110718) might have some bearing on this bug; it was
checked in a couple of hours before the build date, I don't know how accurate
those build dates are.

Comment 3

[Brian Ryner (not reading)]

This is a result of the change aaronl made to
nsEventStateManager::FocusElementButNotDocument() for bug 110718.

-> aaronl

Comment 4

[Aaron Leventhal]

Attachment: This fixes the bug and still contains the necessary zombie doc fix

Comment 5

[Brian Ryner (not reading)]

Comment on attachment 115962 [details] [diff] [review]
This fixes the bug and still contains the necessary zombie doc fix 

makes sense to me.

Comment 6

[Johnny Stenback (:jst)]

Comment on attachment 115962 [details] [diff] [review]
This fixes the bug and still contains the necessary zombie doc fix 

sr=jst

Comment 7

[Aaron Leventhal]

checked in

Comment 8

[HARUNAGA Hirotoshi]

*** Bug 197856 has been marked as a duplicate of this bug. ***