assertion "QueryInterface needed" when list accessible is created

RESOLVED FIXED in mozilla1.9.1a1

Status

()

Core
Disability Access APIs
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: surkov, Assigned: surkov)

Tracking

unspecified
mozilla1.9.1a1
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

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

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Created attachment 321553 [details] [diff] [review]
patch

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 1

10 years ago
Comment on attachment 321553 [details] [diff] [review]
patch

r=me
Attachment #321553 - Flags: review?(marco.zehe) → review+
(Assignee)

Comment 2

10 years ago
could we nominate it for landing? It's obviously plain patch, without any risk and helps with debugging.
Status: NEW → ASSIGNED

Updated

10 years ago
Blocks: 391535
(Assignee)

Updated

10 years ago
Whiteboard: post1.9
(Assignee)

Updated

10 years ago
Whiteboard: post1.9 → post1.9[has-patch,has-review]

Comment 3

10 years ago
Pushed to mozilla-central in changeset:
http://hg.mozilla.org/mozilla-central/index.cgi/rev/071937b753b5
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Updated

10 years ago
Target Milestone: --- → mozilla1.9.1a1
You need to log in before you can comment on or make changes to this bug.