169297 - caret becomes invisible when switching windows if it's not in linked content

Status: RESOLVED FIXED

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

Description

[sairuh (rarely reading bugmail)]

found using 2002.09.16.08 comm trunk builds on win2k and linux rh7.2. oddly, i
cannot reproduce this on mac os 10.1.5.

1. have at least two windows opened. for example, i had two browser windows
open, one pointing to (a) http://bugzilla.mozilla.org and another (b)
http://mozilla.org

2. turn on caret mode (hit F7). actually doesn't seem to matter if you do this
before or after step (1).

3. following the example in step (1):

a. place the caret in http://bugzilla.mozilla.org in the middle of the "Query
Page" link --click outside the link (so you don't activate it), move the caret
with arrow key.

b. place the caret in http://mozilla.org in the middle of the "Status Update"
header (which is not linked content).

4. switch btwn the two windows, either by alt-tabbing, or by mouse-clicking
their titlbars.

observe:

when switching back to (a) where the caret is within linked content, the caret
is (as expected) visible.

however, when switching back to (b) where the caret is within *non-linked*
content, the caret has become invisible. it only becomes visible again if you
arrow-key around into some linked content.

Comment 1

[Alexander Grimalovsky]

Confirming this with trunk build 2002122708 in Windows 2000. It can be
reproduced even if there is only one browser window opened no matter if it have
one or several tabs or if there is any other browser windows are available. 

Comment 2

[Aaron Leventhal]

Still a bug in Mozilla 1.8a2
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a2) Gecko/20040603

Comment 3

[Ginn Chen]

Attachment: patch

Caret is set invisible after switching windows. We should reset it to visible
when switch back.

Comment 4

[Ginn Chen]

Comment on attachment 155669 [details] [diff] [review]
patch

review? Aaron

Comment 5

[Kyle Yuan]

Fixed.

Checking in nsEventStateManager.cpp;
/cvsroot/mozilla/content/events/src/nsEventStateManager.cpp,v  <-- 
nsEventStateManager.cpp
new revision: 1.523; previous revision: 1.522
done

Comment 6

[Ginn Chen]

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