Closed
Bug 493531
Opened 16 years ago
Closed 14 years ago
MSAA fires hide event for document's HWND
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
People
(Reporter: surkov, Unassigned)
References
(Blocks 1 open bug)
Details
The problem is here - http://mxr.mozilla.org/mozilla-central/source/accessible/src/msaa/nsAccessibleWrap.cpp#1675
We trying to get parent accessible for invalidated accessible (in the case of DOM node removing), therefore we get document accessible here.
Reporter | ||
Comment 1•16 years ago
|
||
for information, this code goes from bug 290352, from times where we hadn't delayed events. So obviously we should reconsider the code. I think it's worth to try Marco's suggestion, search closest HWND for visible area.
Reporter | ||
Comment 2•16 years ago
|
||
So, that's what actually happen:
1) process removing of text node and pend hide event
2) process adding of text node, invalidate parent and null mParent of removed text node
3) process event for removed text node, try to get cached parent (fails), get parent DOM node (fails), get document accessible.
4) fire hide event for HWND of document accessible
So there is no way to get closest visible HWND for removed text node. We should reconsider our logic or cache some data (like accessible parent or its HWND), I can't say I like second approach.
Reporter | ||
Comment 3•16 years ago
|
||
I didn't find any details how AccessibleObjectFromEvent works but it sends WM_GETOBJECT to accessible window where we return document accessible (or root accessible) always, then it looks like AccessibleObjectFromEvent uses childId passed into NotifyWinEvent and obtained document accessible to get event target. I don't understand how childId can be used but this scenario looks true. So I think the problem might be not in the fact we return HWND for document accessible, it might be because of accessible was invalidated already and there is no way to get it from childId.
Reporter | ||
Comment 4•16 years ago
|
||
I filed bug 494109 to keep accessibles alive unti we fired a11y event. I think that bug should fix this one.
Depends on: 494109
Comment 5•15 years ago
|
||
Note we are going to have less and less HWNDs: http://weblogs.mozillazine.org/roc/archives/2009/06/native_widgets.html
Reporter | ||
Comment 6•14 years ago
|
||
This bug should have been fixed by bug 570275. Need to check.
Reporter | ||
Comment 7•14 years ago
|
||
(In reply to comment #6)
> This bug should have been fixed by bug 570275. Need to check.
confirm fixed.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•