Closed Bug 348621 Opened 17 years ago Closed 16 years ago

Contents of <iframe> not firing valid accessibility events


(Core :: Disability Access APIs, defect)

Windows XP
Not set





(Reporter: aaronlev, Assigned: aaronlev)


(Blocks 1 open bug, )


(Keywords: access, regression)


(3 files)

Load a document with an iframe, such as
With an accessibility testing tool open, tab until you get to the iframe contents.

Accessibility events are fired, but the role, state and other info are not available for the event targets.
Assignee: aaronleventhal → nian.liu
Attached file test case
Attached file foo.html
aaron, which at event tool you use?

(In reply to comment #3)
> aaron, which at event tool you use?

MSAA Inspect will work, or a combination of that and MSAA event watcher. But, I think the same problem would show up if watching the events on Linux using at-poke.
Blocks: htmla11y
No longer blocks: keya11y
1-25 works
1-26 broken
Sorry, forgot to say the regression was in 2006.

1-25-2006 works
1-26-2006 broken
The domNode in PresShell::HandleEventInternal() is for the root html doc instead of the iframe doc. Not sure why yet, because the HWND we use with NotifyWinEvent() is for the correct window.
Assignee: nian.liu → aaronleventhal
Robert, this was a regression from your patch to bug 317375.
Attachment #261718 - Flags: superreview?(roc)
Attachment #261718 - Flags: review?(roc)
Comment on attachment 261718 [details] [diff] [review]
One line patch for regression. Don't treat NS_ACCESSIBLE_EVENT like a mouse event.

Probably better to get 2 eyes on this given how long ago bug 317375 was checked in, and how complex it was.

I wonder if we need to change any other similar places.
Attachment #261718 - Flags: superreview?(roc) → superreview?(dbaron)
Keywords: regression
OS: All → Windows XP
Blocks: 317375
Attachment #261718 - Flags: superreview?(dbaron)
Attachment #261718 - Flags: superreview+
Attachment #261718 - Flags: review?(roc)
Attachment #261718 - Flags: review+
Closed: 16 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.