Created attachment 574266 [details] [diff] [review]
Except it doesn't handle the case where XPCStringConvert::ReadableToJSVal returns null.
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]
r=me. Yay code removal.