nsXMLHttpRequest::GetResponse duplicates quickstub code

RESOLVED FIXED in mozilla11

Status

()

defect
RESOLVED FIXED
8 years ago
4 months ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

Trunk
mozilla11
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

Posted patch Patch v1Splinter Review
Except it doesn't handle the case where XPCStringConvert::ReadableToJSVal returns null.
Attachment #574266 - Flags: review?(bzbarsky)
Is it correct including XPCQuickStubs.h outside the quickstub?
xpc_qsStringToJsval is used elsewhere already, and I don't see any reasons why not use it in XHR.
Although xpcprivate.h is already used outside the xpconnect, there exists bug 691463.
Yep, and that bug is the reason I was checking if XHR needed to use XPCConvert.
Comment on attachment 574266 [details] [diff] [review]
Patch v1

r=me.  Yay code removal.
Attachment #574266 - Flags: review?(bzbarsky) → review+
https://hg.mozilla.org/mozilla-central/rev/178a3e2dc3f0
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
Whiteboard: [qa-]
Component: DOM: Mozilla Extensions → DOM
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.