Created separate shards for suspicious testcases.

build/android/pylib/gtest/gtest_test_instance.py

 # Copyright 2014 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
 import os
 import re
 import tempfile
 
 from devil.android import apk_helper
@@ skipping 325 matching lines @@
         host_paths.DIR_SOURCE_ROOT, 'build', 'android', 'pylib', 'gtest',
         'filter', '%s_disabled' % self._suite)
     if disabled_tests_file_path and os.path.exists(disabled_tests_file_path):
       with open(disabled_tests_file_path) as disabled_tests_file:
         disabled_filter_items += [
             '%s' % l for l in (line.strip() for line in disabled_tests_file)
             if l and not l.startswith('#')]
 
     return '*-%s' % ':'.join(disabled_filter_items)
 
+  def GetCrashedTestCase(self, output):
+    crash_happened = re.search(r'FAILURES!!! Tests run: \d+, Failures: \d+,' \

[jbudorick, 2016/02/27 00:31:37]

all of the regexes in here should compiled into module-private constants, e.g.
this should be

_TEST_ERROR_RE = re.compile(r'FAILURES!!! ...')

  ...

  def GetCrashedTestCase(self, output):
    crash_happened = _TEST_ERROR_RE.search(output[-1])

+                               ' Errors: 1', output[-1])
+    if not crash_happened:
+      return
+
+    #Crashes that will cause output like DCHECK(false).

[jbudorick, 2016/02/27 00:31:37]

nit: # Crashes ...

+    has_crash = re.search(r'\[ERROR:.*?\] Currently running: ', output[-3])

[jbudorick, 2016/02/27 00:31:37]

magical -3

+    if has_crash:
+      return output[-3][len(has_crash.group(0)):]
+
+    # Crashes that will cause output like null pointer dereference.
+    index = output.index('[ CRASHED      ]')
+    if not index == -1:
+      run_info = re.search(r'(?<=\[ RUN      \] ).*', output[index-1])

[jbudorick, 2016/02/27 00:31:37]

Why is this doing a lookbehind if you're matching on .*? (I had to look up the
regex syntax here.) What exactly do you intend to match?

+      if run_info:
+        return run_info.group(0)
+
   # pylint: disable=no-self-use
   def ParseGTestOutput(self, output):
     """Parses raw gtest output and returns a list of results.
 
     Args:
       output: A list of output lines.
     Returns:
       A list of base_test_result.BaseTestResults.
     """
     log = []
@@ skipping 23 matching lines @@
         result_type = None
 
     return results
 
   #override
   def TearDown(self):
     """Clear the mappings created by SetUp."""
     if self._isolate_delegate:
       self._isolate_delegate.Clear()