Closed Bug 434428 Opened 16 years ago Closed 16 years ago

assertion "QueryInterface needed" when list accessible is created

Categories

(Core :: Disability Access APIs, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.1a1

People

(Reporter: surkov, Assigned: surkov)

References

Details

(Whiteboard: post1.9[has-patch,has-review])

Attachments

(1 file)

Attached patch patchSplinter Review
nsXULListitemAccessible::GetListAccessible() pass nsIDOMXULSelectControlElement to GetAccessibleInWeakShell without quering to nsIDOMNode, it leads to assertion NS_ASSERTION(query_result.get() == mRawPtr, "QueryInterface needed"); inside nsCOMPtr.h file.

This assertion makes hard to debug some code.
Attachment #321553 - Flags: review?(marco.zehe)
Comment on attachment 321553 [details] [diff] [review]
patch

r=me
Attachment #321553 - Flags: review?(marco.zehe) → review+
could we nominate it for landing? It's obviously plain patch, without any risk and helps with debugging.
Status: NEW → ASSIGNED
Blocks: 191a11y
Whiteboard: post1.9
Whiteboard: post1.9 → post1.9[has-patch,has-review]
Pushed to mozilla-central in changeset:
http://hg.mozilla.org/mozilla-central/index.cgi/rev/071937b753b5
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1a1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: