Bug 1410165 - Make the test harness properly listen to crash events when running a test more than once; r=mconley

MozReview-Commit-ID: C3PDtOfaz0v

--HG--
extra : rebase_source : 568387a2a53a55b19a03c6cc793125a01ebd3760

testing/mochitest/tests/SimpleTest/TestRunner.js

@@ -458,8 +458,6 @@ TestRunner.runNextTest = function() {
             indicator.style.backgroundColor = "red";
         }
 
-        SpecialPowers.unregisterProcessCrashObservers();
-
         let e10sMode = SpecialPowers.isMainProcess() ? "non-e10s" : "e10s";
 
         TestRunner.structuredLogger.info("TEST-START | Shutdown");
@@ -469,8 +467,10 @@ TestRunner.runNextTest = function() {
         TestRunner.structuredLogger.info("Mode:    " + e10sMode);
         TestRunner.structuredLogger.info("Slowest: " + TestRunner.slowestTestTime + 'ms - ' + TestRunner.slowestTestURL);
 
-        // If we are looping, don't send this cause it closes the log file
+        // If we are looping, don't send this cause it closes the log file,
+        // also don't unregister the crash observers until we're done.
         if (TestRunner.repeat === 0) {
+          SpecialPowers.unregisterProcessCrashObservers();
           TestRunner.structuredLogger.info("SimpleTest FINISHED");
         }