Closed Bug 551679 Opened 10 years ago Closed 10 years ago

focus is not fired for focused document when switching between windows

Categories

(Core :: Disability Access APIs, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: surkov, Assigned: surkov)

References

(Blocks 1 open bug)

Details

(Keywords: access)

Attachments

(1 file, 1 obsolete file)

David fixed some portion of the problem in bug 512059. However since we don't get two events when editable document if focused (that was an idea of bug 512059 fix if I get right) then we don't update gLastFocusedNode properly and we ignore DOM focus event (since gLastFocusedNode hasn't been changed).

Also it might be related with bug 526313 and bug 352220.
Actually focus isn't fired for any focused document when user switches between windows. Morphing the bug summary.
Summary: focus is not fired for focused editable document when switching between windows → focus is not fired for focused document when switching between windows
Attached patch wip (obsolete) — Splinter Review
simple patch + mochitests what needs some polishing
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Attached patch patchSplinter Review
presented mochitests don't test the bug actually. The good way to reproduce the bug is open and close print dialog (native window) while editable document is focused. I can't find a way to close the print dialog from mochitests so I add some generic tests.
Attachment #431909 - Attachment is obsolete: true
Attachment #432081 - Flags: review?(marco.zehe)
Attachment #432081 - Flags: review?(bolterbugz)
Attachment #432081 - Flags: review?(marco.zehe) → review+
Comment on attachment 432081 [details] [diff] [review]
patch

r=me. Thanks for cleaning this one up also!
Comment on attachment 432081 [details] [diff] [review]
patch

r=me; great idea!
Attachment #432081 - Flags: review?(bolterbugz) → review+
landed - http://hg.mozilla.org/mozilla-central/rev/077e765a8e8c
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.