bug 792850 - add an xpcshell test for our SetUnhandledExceptionFilter hook. r=ehsan

--HG--
extra : rebase_source : f289d7d8214bcfdd90baeae8c47e2747de806426

toolkit/crashreporter/test/CrashTestUtils.jsm

@@ -42,6 +42,13 @@ CrashTestUtils.lockDir = lib.declare("LockDir",
                                      ctypes.voidptr_t);  // nsISupports*
 
 
+try {
+  CrashTestUtils.TryOverrideExceptionHandler = lib.declare("TryOverrideExceptionHandler",
+                                                           ctypes.default_abi,
+                                                           ctypes.void_t);
+}
+catch(ex) {}
+
 CrashTestUtils.dumpHasStream = lib.declare("DumpHasStream",
                                            ctypes.default_abi,
                                            ctypes.bool,

toolkit/crashreporter/test/nsTestCrasher.cpp

@@ -104,3 +104,17 @@ uint64_t SaveAppMemory()
 
   return (int64_t)testData;
 }
+
+#ifdef XP_WIN32
+static LONG WINAPI HandleException(EXCEPTION_POINTERS* exinfo)
+{
+  TerminateProcess(GetCurrentProcess(), 0);
+  return 0;
+}
+
+extern "C" NS_EXPORT
+void TryOverrideExceptionHandler()
+{
+  SetUnhandledExceptionFilter(HandleException);
+}
+#endif

toolkit/crashreporter/test/unit/test_override_exception_handler.js

@@ -0,0 +1,12 @@
+function run_test()
+{
+  // Ensure that attempting to override the exception handler doesn't cause
+  // us to lose our exception handler.
+  do_crash(
+    function() {
+        CrashTestUtils.TryOverrideExceptionHandler();
+    },
+    function(mdump, extra) {
+    },
+    true);
+}

toolkit/crashreporter/test/unit/xpcshell.ini

@@ -11,6 +11,8 @@ skip-if = os == 'win' && debug
 [test_crashreporter.js]
 [test_crashreporter_crash.js]
 [test_crashreporter_crash_profile_lock.js]
+[test_override_exception_handler.js]
+run-if = os == 'win'
 
 [test_crashreporter_appmem.js]
 run-if = os == 'win' || os == 'linux'