Closed
Bug 348621
Opened 18 years ago
Closed 17 years ago
Contents of <iframe> not firing valid accessibility events
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
References
(Blocks 1 open bug, )
Details
(Keywords: access, regression)
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
text/html
|
Details | |
(deleted),
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
Steps: Load a document with an iframe, such as http://www.mozilla.org/access/samples/js-nsIAccessible.htm 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 | ||
Updated•18 years ago
|
Assignee: aaronleventhal → nian.liu
Comment 1•18 years ago
|
||
Comment 2•18 years ago
|
||
Comment 3•18 years ago
|
||
aaron, which at event tool you use?
Assignee | ||
Comment 4•18 years ago
|
||
(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.
Assignee | ||
Updated•18 years ago
|
Assignee | ||
Comment 5•17 years ago
|
||
1-25 works 1-26 broken
Assignee | ||
Comment 6•17 years ago
|
||
Sorry, forgot to say the regression was in 2006. 1-25-2006 works 1-26-2006 broken
Assignee | ||
Comment 7•17 years ago
|
||
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
Assignee | ||
Comment 8•17 years ago
|
||
I think it's bug 317375, which is probably changing how NS_ACCESSIBLE_EVENT is being targeted. That's the most likely thing from the checkins in the regression range: http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2006-01-25+01%3A00%3A00&maxdate=2006-01-26+14%3A00%3A00&cvsroot=%2Fcvsroot
Assignee | ||
Comment 9•17 years ago
|
||
Robert, this was a regression from your patch to bug 317375.
Attachment #261718 -
Flags: superreview?(roc)
Attachment #261718 -
Flags: review?(roc)
Assignee | ||
Comment 10•17 years ago
|
||
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)
Assignee | ||
Updated•17 years ago
|
Keywords: regression
OS: All → Windows XP
Attachment #261718 -
Flags: superreview?(dbaron)
Attachment #261718 -
Flags: superreview+
Attachment #261718 -
Flags: review?(roc)
Attachment #261718 -
Flags: review+
Assignee | ||
Updated•17 years ago
|
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•17 years ago
|
Flags: in-testsuite?
Comment 11•5 years ago
|
||
Keywords: sec508
You need to log in
before you can comment on or make changes to this bug.
Description
•