bug 514397 - xpcshell-tests: test_crashreporter.js crashes intermittently. disable parts of the test that set/unset exception handler. upstream breakpad issue filed, this is not code that gets run in the browser, so it doesn't seem critical to have this test hit this bug and cause orange.

--HG--
extra : rebase_source : 5f9381e387aa69d049139605615d97b4667bc817

toolkit/crashreporter/test/unit/test_crashreporter.js

@@ -12,7 +12,10 @@ function run_test()
 
   /**********
    * Check behavior when disabled.
-   **********/
+   * XXX: don't run these tests right now, there's a race condition in
+   * Breakpad with the handler thread. See:
+   * http://code.google.com/p/google-breakpad/issues/detail?id=334
+   **********
 
   // Crash reporting is enabled by default in </testing/xpcshell/head.js>.
   dump("INFO | test_crashreporter.js | Disable crashreporter.\n");
@@ -53,11 +56,12 @@ function run_test()
 
   /**********
    * Check behavior when enabled.
-   **********/
+   **********
 
   dump("INFO | test_crashreporter.js | Re-enable crashreporter (in default state).\n");
   // check that we can enable the crashreporter
   cr.enabled = true;
+*/
   do_check_true(cr.enabled);
   // ensure that double-enabling doesn't error
   cr.enabled = true;
@@ -136,12 +140,15 @@ function run_test()
   // Add more data.
   cr.appendAppNotesToCrashReport("additional testData4");
 
+  /*
+   * These tests are also disabled, see comment above.
   // check that we can disable the crashreporter
   cr.enabled = false;
   do_check_false(cr.enabled);
   // ensure that double-disabling doesn't error
   cr.enabled = false;
   do_check_false(cr.enabled);
+*/
 
   /**********
    * Reset to initial state.