Detect crashes while running native tests in APK.

build/android/test_result.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 
 import logging
 
 
 # Language values match constants in Sponge protocol buffer (sponge.proto).
 JAVA = 5
@@ skipping 43 matching lines @@
     self.ok = []
     self.failed = []
     self.crashed = []
     self.unknown = []
     self.disabled = []
     self.unexpected_pass = []
     self.timed_out = False
     self.overall_fail = False
 
   @staticmethod
-  def FromOkAndFailed(ok, failed, timed_out, overall_fail):
+  def FromOkFailedAndCrashed(ok, failed, crashed, timed_out, overall_fail):

[John Grabowski, 2012/05/07 19:22:15]

That's a lot of arguments.
Can you make all callers pass by name?  E.g.
ok=[], failed=[], ...
so input is more clear?

[nilesh, 2012/05/08 01:38:10]

Done.

     ret = TestResults()
     ret.ok = ok
     ret.failed = failed
+    ret.crashed = crashed
     ret.timed_out = timed_out
     ret.overall_fail = overall_fail
     return ret
 
   @staticmethod
   def FromTestResults(results):
     """Combines a list of results in a single TestResults object."""
     ret = TestResults()
     for t in results:
       ret.ok += t.ok
@@ skipping 41 matching lines @@
                                     len(self.crashed) + len(self.unknown))
     summary_string += 'PASSED=%d\n' % (len(self.ok))
     summary_string += 'FAILED=%d %s\n' % (len(self.failed),
                                           [t.name for t in self.failed])
     summary_string += 'CRASHED=%d %s\n' % (len(self.crashed),
                                            [t.name for t in self.crashed])
     summary_string += 'UNKNOWN=%d %s\n' % (len(self.unknown),
                                            [t.name for t in self.unknown])
     logging.critical(summary_string)
     return summary_string