Created attachment 8471117 [details] [diff] [review] leaky test patch Bug 1009765 landed without a test that makes sure crashed GMP plugins are reported to the crash service. This patch, copied over from the other bug, adds adds one and makes related necessary changes, but it leaks a CondVar, GeckoChildProcessHost, and Mutex. Filing this in Core::WebRTC because that's where the test in this patch lives.
jesup, do you have any ideas?
In bug 1035056 I encountered this leak when plugins crashed while shutting down. I fixed it by ensuring that GeckoMediaPluginService::ReAddOnGMPThread() does not re-add a GMP clone if we're already shutting down. This is a different case (the GMP is not crashing while shutting down here, right?), but the cause could be something similar.