Closed Bug 423698 Opened 13 years ago Closed 13 years ago

Incorrectly used variables in nsAccessibleWrap::FireAccessibleEvent()

Categories

(Core :: Disability Access APIs, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9beta5

People

(Reporter: aaronlev, Assigned: aaronlev)

Details

(Keywords: access)

Attachments

(1 file)

The evolution of FireAccessibleEvent() has left several variables unused and in one case can lead to the wrong HWND being reported during the removal of nodes, because the frame is no longer present.
Target Milestone: --- → mozilla1.9beta5
Attachment #310299 - Flags: review?(surkov.alexander)
Attachment #310299 - Flags: review+
Attachment #310299 - Flags: approval1.9?
Comment on attachment 310299 [details] [diff] [review]
Remove unused variables, and use newAccessible where are supposed to

a1.9=beltzner
Attachment #310299 - Flags: approval1.9? → approval1.9+
Requesting checkin on Aaron's behalf. He'll be offline for a couple of days.
Keywords: checkin-needed
/cvsroot/mozilla/accessible/src/msaa/nsAccessibleWrap.cpp,v  <--  nsAccessibleWrap.cpp
new revision: 1.114; previous revision: 1.113
done

Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.