http://searchfox.org/mozilla-central/rev/c2a60adfc7b16761cbbfcefa2093fa402ba1aa69/docshell/base/LoadContext.cpp#16 does some crazy JS stuff, when it should just use mOriginAttributes when nsILoadContext is C++ implemented.

er, not perhaps mOriginAttributes, but whatever native method there is to access OA.

Yoshi, do you think you could take a look at this? Somehow make the method fast in the common case when nsILoadContext is C++ implemented.

I started to write a patch for this in bug 1316683 before I saw this. I think this was originally done to avoid having C++ callers go through JSAPI, so maybe that was the thinking behind the implementation, but the implementation certainly doesn't make sense in retrospect. :-) I think we can make this a dupe?

Now this function won't even exist!