Closed
Bug 551679
Opened 14 years ago
Closed 14 years ago
focus is not fired for focused document when switching between windows
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
People
(Reporter: surkov, Assigned: surkov)
References
(Blocks 1 open bug)
Details
(Keywords: access)
Attachments
(1 file, 1 obsolete file)
|
11.53 KB,
patch
|
MarcoZ
:
review+
davidb
:
review+
|
Details | Diff | Splinter Review |
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.
|
Assignee | |
Comment 1•14 years ago
|
||
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
|
|
Assignee | |
Comment 2•14 years ago
|
||
simple patch + mochitests what needs some polishing
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 3•14 years ago
|
||
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)
|
||
Updated•14 years ago
|
Attachment #432081 -
Flags: review?(marco.zehe) → review+
|
|
||
Comment 4•14 years ago
|
||
Comment on attachment 432081 [details] [diff] [review] patch r=me. Thanks for cleaning this one up also!
|
||
Comment 5•14 years ago
|
||
Comment on attachment 432081 [details] [diff] [review] patch r=me; great idea!
Attachment #432081 -
Flags: review?(bolterbugz) → review+
|
|
Assignee | |
Comment 6•14 years ago
|
||
landed - http://hg.mozilla.org/mozilla-central/rev/077e765a8e8c
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 7•14 years ago
|
||
landed on 1.9.3 leak fix - http://hg.mozilla.org/mozilla-central/rev/56b8c6333c0c
|
|
Assignee | |
Comment 8•14 years ago
|
||
right leak fix - http://hg.mozilla.org/mozilla-central/rev/70edf4db7676
You need to log in
before you can comment on or make changes to this bug.
Description
•