Currently CrashReporterParent assumes that crashes are notified on the main thread, which is true for content/plugins, but is not true of GMP plugins where the entire IPDL setup is on the GMP thread. The patch here will just proxy the crash-manager bits to the main thread.

Comment on attachment 8456970 [details] [diff] [review] bug1039579-CrashReporterParent Review of attachment 8456970 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/ipc/CrashReporterParent.cpp @@ +120,1 @@ > type = NS_LITERAL_CSTRING("plugin"); Do we want any additional annotation here or do we think it'll be obvious from the other context in the crash report? @@ +152,5 @@ > + } > + private: > + CrashReporterParent* mCR; > + }; > + SyncRunnable::DispatchToThread(mainThread, new NotifyOnMainThread(this)); Is this just an expensive no-op if this is already being called on the main thread? (Shouldn't matter, this isn't perf-sensitive code.)