Crash ppapi processes on Windows when calling exit(), _exit(), abort() and ExitProcess()

content/ppapi_plugin/ppapi_thread.cc

Index: content/ppapi_plugin/ppapi_thread.cc
diff --git a/content/ppapi_plugin/ppapi_thread.cc b/content/ppapi_plugin/ppapi_thread.cc
index 2e6e255b1500f636505949357e5bc53a92135e0a..d5d9c21713fb39abc38a1ff393bd4303a1db9e82 100644
--- a/content/ppapi_plugin/ppapi_thread.cc
+++ b/content/ppapi_plugin/ppapi_thread.cc
@@ -35,6 +35,7 @@
 #include "webkit/plugins/plugin_switches.h"
 
 #if defined(OS_WIN)
+#include "base/win/win_util.h"
 #include "sandbox/win/src/sandbox.h"
 #elif defined(OS_MACOSX)
 #include "content/common/sandbox_init_mac.h"
@@ -78,6 +79,11 @@ PpapiThread::~PpapiThread() {
   if (plugin_entry_points_.shutdown_module)
     plugin_entry_points_.shutdown_module();
   WebKit::shutdown();
+
+#if defined(OS_WIN)
+  if (permissions_.HasPermission(ppapi::PERMISSION_FLASH))
+    base::win::SetShouldCrashOnProcessDetach(false);
+#endif
 }
 
 bool PpapiThread::Send(IPC::Message* msg) {
@@ -271,6 +277,15 @@ void PpapiThread::OnMsgLoadPlugin(const FilePath& path,
   }
 
 #if defined(OS_WIN)
+  // If code subsequently tries to exit using abort(), force a crash (since
+  // otherwise these would be silent terminations and fly under the radar).
+  base::win::SetAbortBehaviorForCrashReporting();
+  if (permissions.HasPermission(ppapi::PERMISSION_FLASH)) {
+    // Force a crash for exit(), _exit(), or ExitProcess(), but only do that for
+    // Pepper Flash.
+    base::win::SetShouldCrashOnProcessDetach(true);
+  }
+
   // Once we lower the token the sandbox is locked down and no new modules
   // can be loaded. TODO(cpu): consider changing to the loading style of
   // regular plugins.