188088 - content area is not focused if chimera is in the background when page finishes loading

Status: VERIFIED FIXED

(Camino Graveyard :: Page Layout, defect)

Description

[Brian Ryner (not reading)]

1. Start loading a page
2. Switch to another app
3. Wait for the page to finish loading
4. Switch back to Chimera
5. hit down-arrow

Expected result: the page scrolls down
Actual result: it beeps at you, because the page isn't focused.

Comment 1

[Brian Ryner (not reading)]

Attachment: patch

chimera isn't setting gecko to be the first responder if the load finishes
while the window is unfocused, so when it does get focused, Activate isn't
called on the nsWebBrowser.  I think if we do it this way, there's no need for
the ugly focus controller hack.

Comment 2

[Mike Pinkerton (not reading bugmail)]

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

if you have two windows open, load a page into one and switch windows while
it's loading, when the page finishes, it will pop to the front.

bad :(

i think we have to leave it as is.

Comment 3

[Brian Ryner (not reading)]

Attachment: new patch

don't make focusChangedFrom:To: activate the web browser if the window isn't
focused

Comment 4

[Mike Pinkerton (not reading bugmail)]

landed 2nd patch, with a cleanup of the patch to not set the focus if the user
has typed in the urlbar.

Comment 5

[John Anderson]

Verified with latest builds, this appears to be fixed.

Comment 6

[sairuh (rarely reading bugmail)]

vrfy'd fixed using 2003.02.06.15 on 10.2.3. tested both mouse and keyboard
shortcuts to switch btwn another app and chimera.