Active Accessibility: focus events not working on documents

RESOLVED FIXED

Status

()

Core
Disability Access APIs
RESOLVED FIXED
16 years ago
16 years ago

People

(Reporter: Aaron Leventhal, Assigned: Aaron Leventhal)

Tracking

Trunk
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
When a document is focused, GetFocusedElement() returns null, and thus
GetAccFocused() cannot create an accessible.

I have a patch forthcoming that changed GetFocusedElement() to GetFocusedNode(),
which will return the document node when the document is focused.

This fixes the situations.
(Assignee)

Comment 1

16 years ago
Created attachment 89490 [details] [diff] [review]
Changes  GetFocusedElement() to GetFocusedNode(), which can now see document dom node focus. Also changes Accessible.cpp so that it fires focus events again.

seeking r=

Comment 2

16 years ago
Comment on attachment 89490 [details] [diff] [review]
Changes  GetFocusedElement() to GetFocusedNode(), which can now see document dom node focus. Also changes Accessible.cpp so that it fires focus events again.

r=kyle

Yes, GetFocusedNode is more reasonable.
Attachment #89490 - Flags: review+
(Assignee)

Updated

16 years ago
Blocks: 154704
Comment on attachment 89490 [details] [diff] [review]
Changes  GetFocusedElement() to GetFocusedNode(), which can now see document dom node focus. Also changes Accessible.cpp so that it fires focus events again.

+  /*
+  // Remove this until we can figure out which focus events are coming at
+  // the same time as native window focus events, although
+  // perhaps 2 duplicate focus events on the window isn't really a problem
...
+  */

Please use #if 0 ... #endif to take out blocks of code, using comments is error
prone...

sr=jst
Attachment #89490 - Flags: superreview+
(Assignee)

Comment 4

16 years ago
checked in a while back.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.