This is basically similar to bug 422308, which was a bug in older Flash 9. To reproduce: 1) Go to http://www.quakelive.com/, set up an account, install plugin 2) Enter game (tutorial game works fine) 3) Select Crash Me Now from Ted's crashme extension -> Windows crash dialog, no breakpad. Battlefield Heroes (another similar in-browser game plugin) might be doing the same thing. It's not clear whether QL is resetting the exception handler state on every plugin entry, or only on initialization.
Is there anything *we* can do about this?
I don't know that we'd want to go there, I'd much rather see the plugins behave and not leave things in a different state than what it was when they changed the state. I certainly wouldn't mind notifying about this one way or another, but I'm not sure what good ways we have to do that which would actually cause plugin developers to get the message. Either way, I really don't think I'd hold the release for this. blocking-
Flags: blocking1.9.1? → blocking1.9.1-
We can't control plugins though, and until we can get them out of process, this leads to a pretty crappy user experience. Wrapping some calls into the plugin with a simple object that'll save/restore some state should have a negligible effect on perf, and will help us collect crash data that we can then go to the plugin vendor with..
I think we *do* want to go there -- we should restore our invariants as aggressively as we can without slaughtering perf.
I disagree. There's several cases where we can not do anything about this (short of sticking fixup and restore code in the guts of our event loop), like if a plugin messes with any of the states in question (and who knows what other state) and goes back to our event loop, or if they create child widgets and do this as a response to events there, we're in the same boat again. I'd much rather see plugins not stomp over stuff that affects others and leave things as such than us trying to fix up after them, and fully knowing that we can only do that in some cases at best.
Now having said that, restoring the crash reporter if plugin initialization wipes it out I think would be ok (assuming that actually is enough to fix this bug), but I don't like the idea of attempting to do that across the board for the whole plugin API.
Component: Plug-ins → Quake Live
Product: Core → Plugins
QA Contact: plugins → quake-live
Closing old bugs in the Plugins component. We aren't going to track issues in 3rd-party plugins in the Mozilla bug tracker. In addition, support for NPAPI plugins will be removed at the end of this year; for more details see the post at https://blog.mozilla.org/futurereleases/2015/10/08/npapi-plugins-in-firefox/ If there is a serious bug in Firefox, it needs to be filed in the "Core" product, "Plug-Ins" component.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INCOMPLETE
Product: Plugins → Plugins Graveyard
You need to log in before you can comment on or make changes to this bug.