Modify autotest to support preserving minidumps generated during tests.

client/bin/test.py

Index: client/bin/test.py
diff --git a/client/bin/test.py b/client/bin/test.py
index 33f194ca39e955d030789b08ae8f89bda8f45cab..b172155a75b44444a48e497cfe7ac285f445d7ed 100644
--- a/client/bin/test.py
+++ b/client/bin/test.py
@@ -107,6 +107,24 @@ class test(common_test.base_test):
                     logging.warning('Please verify %s for more info', dir)
 
 
+    def _call_run_once(self, constraints, profile_only,

[kmixter1, 2011/02/23 02:00:39]

Eric should comment on this since it's common code and he's edited it more than
I have.

+                       postprocess_profiled_run, args, dargs):
+        """
+        The base class does not execute the after_iteration_hooks if a test
+        raises an exception.  We want to override that behavior here since our
+        after_iteration_hooks collect important logs and crash information.
+        """
+        try:
+            common_test.base_test._call_run_once(self, constraints,
+                                                 profile_only,
+                                                 postprocess_profiled_run,
+                                                 args, dargs)
+        except:
+            for hook in self.after_iteration_hooks:

[ericli, 2011/02/23 04:49:34]

common_test.base_test._call_run_once() will also call
self.after_iteration_hooks.  I would suggest you to directly patch into
ommon_test.base_test._call_run_once()  and try to upstream it.

I dont know if they will accept it or not but we at least should try.

[thieule, 2011/03/03 01:27:41]

Done.

+                hook(self)
+            raise
+
+
 def runtest(job, url, tag, args, dargs):
     common_test.runtest(job, url, tag, args, dargs, locals(), globals(),
                         job.sysinfo.log_before_each_test,