Every existing caller of this function ends up QIing from nsISupports anyways, and I think the current setup requires the caller knowing what nsIXPConnectWrappedJS is, which is only available in some places. This was giving me trouble when I was trying to call it in the observer service. I'm also going to rename the function.
Assignee: nobody → continuation
Comment on attachment 610260 [details] [diff] [review] minor cleanup Try run looked fine (same one as the other patch).
Attachment #610260 - Flags: review?(bugs)
Thanks for the reviews. https://hg.mozilla.org/integration/mozilla-inbound/rev/ec1874ee6dc3
Target Milestone: --- → mozilla14
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.