Created attachment 618605 [details] [diff] [review] patch this shouldn't be terribly bad bug, since without the context safejscontext is used for JSON message creation.
...but I want to fix the assertion mention in bug 749018
Comment on attachment 618605 [details] [diff] [review] patch This looks fine to me, but I don't feel comfortable reviewing this code.
5 years ago
This fixes my assertion, so I'm happy.
Comment on attachment 618605 [details] [diff] [review] patch - In TabChildGlobal::Init() mMessageManager = new nsFrameMessageManager(false, SendSyncMessageToParent, SendAsyncMessageToParent, nsnull, mTabChild, nsnull, - aTabChild->GetJSContext()); + mTabChild->GetJSContext()); } Should we assert here that this is not called more than once? r=jst either way.
I can add an assert