Print out the ui_test test case in the reports

tools/valgrind/tsan_analyze.py

 #!/usr/bin/python
 # Copyright (c) 2011 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.
 
 # tsan_analyze.py
 
 ''' Given a ThreadSanitizer output file, parses errors and uniques them.'''
 
 import gdb_helper
@@ skipping 103 matching lines @@
       self.ReadLine()
       if re.match('-+ suppression -+', self.line_):
         # We need to calculate the suppression hash and prepend a line like
         # "Suppression (error hash=#0123456789ABCDEF#):" so the buildbot can
         # extract the suppression snippet.
         supp = ""
         while not re.match('-+ end suppression -+', self.line_):
           self.ReadLine()
           supp += self.line_
         self.ReadLine()
+        if self._cur_testcase:
+          result.append("The report came from the `%s` test.\n" % \
+                        self._cur_testcase)
         result.append("Suppression (error hash=#%016X#):\n" % \
                       (int(hashlib.md5(supp).hexdigest()[:16], 16)))
         result.append("  For more info on using suppressions see "
             "http://dev.chromium.org/developers/how-tos/using-valgrind/threadsanitizer#TOC-Suppressing-data-races\n")
         result.append(supp)
     else:
       self.ReadLine()
 
     return result
 
@@ skipping 72 matching lines @@
     reports = []
     self.used_suppressions = {}
     for file in files:
       reports.extend(self.ParseReportFile(file))
     if self._use_gdb:
       TheAddressTable.ResolveAll()
       # Make each line of each report a string.
       reports = map(lambda(x): map(str, x), reports)
     return [''.join(report_lines) for report_lines in reports]
 
-  def Report(self, files, check_sanity=False):
+  def Report(self, files, testcase, check_sanity=False):
     '''Reads in a set of files and prints ThreadSanitizer report.
 
     Args:
       files: A list of filenames.
       check_sanity: if true, search for SANITY_TEST_SUPPRESSIONS
     '''
 
+    self._cur_testcase = testcase
     reports = self.GetReports(files)
+    self._cur_testcase = None

[Alexander Potapenko, 2011/11/11 09:41:31]

Please add a comment describing why you need to reset _cur_testcase

 
     is_sane = False
     print "-----------------------------------------------------"
     print "Suppressions used:"
     print "  count name"
     for item in sorted(self.used_suppressions.items(), key=lambda (k,v): (v,k)):
       print "%7s %s" % (item[1], item[0])
       if item[0].startswith(TsanAnalyzer.SANITY_TEST_SUPPRESSION):
         is_sane = True
     print "-----------------------------------------------------"
@@ skipping 23 matching lines @@
   parser.add_option("", "--source_dir",
                     help="path to top of source tree for this build"
                     "(used to normalize source paths in baseline)")
 
   (options, args) = parser.parse_args()
   if not args:
     parser.error("no filename specified")
   filenames = args
 
   analyzer = TsanAnalyzer(options.source_dir, use_gdb=True)
-  retcode = analyzer.Report(filenames)
+  retcode = analyzer.Report(filenames, None)
 
   sys.exit(retcode)