Make --ignore-in-dtor=no the default ThreadSanitizer option

tools/valgrind/valgrind_test.py

 #!/usr/bin/python
 # Copyright (c) 2006-2008 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.
 
 # valgrind_test.py
 
 '''Runs an exe through Valgrind and puts the intermediate files in a
 directory.
 '''
@@ skipping 374 matching lines @@
     return False
 
   def ExtendOptionParser(self, parser):
     ValgrindTool.ExtendOptionParser(self, parser)
     parser.add_option("", "--suppressions", default=[],
                       action="append",
                       help="path to a valgrind suppression file")
     parser.add_option("", "--pure-happens-before", default="yes",
                       dest="pure_happens_before",
                       help="Less false reports, more missed races")
+    parser.add_option("", "--ignore-in-dtor", default="no",
+                      dest="ignore_in_dtor",
+                      help="Ignore data races inside destructors")
     parser.add_option("", "--announce-threads", default="yes",
                       dest="announce_threads",
                       help="Show the the stack traces of thread creation")
 
   def EvalBoolFlag(self, flag_value):
     if (flag_value in ["1", "true", "yes"]):
       return True
     elif (flag_value in ["0", "false", "no"]):
       return False
     raise RuntimeError, "Can't parse flag value (%s)" % flag_value
@@ skipping 12 matching lines @@
           "tools", "valgrind", "tsan", ignore_file)
       if os.path.exists(fullname):
         ret += ["--ignore=%s" % fullname]
 
     # The -v flag is needed for printing the list of used suppressions.
     ret += ["-v"]
 
     ret += ["--file-prefix-to-cut=%s/" % self._source_dir]
 
     if self.EvalBoolFlag(self._options.pure_happens_before):
-      ret += ["--pure-happens-before=yes"];
+      ret += ["--pure-happens-before=yes"] # "no" is the default value for TSAN
+
+    if not self.EvalBoolFlag(self._options.ignore_in_dtor):
+      ret += ["--ignore-in-dtor=no"] # "yes" is the default value for TSAN
 
     if self.EvalBoolFlag(self._options.announce_threads):
       ret += ["--announce-threads"]
 
     # --show-pc flag is needed for parsing the error logs on Darwin.
     if platform_suffix == 'mac':
       ret += ["--show-pc=yes"]
 
     return ret
 
@@ skipping 39 matching lines @@
 
 if __name__ == "__main__":
   if sys.argv.count("-v") > 0 or sys.argv.count("--verbose") > 0:
     google.logging_utils.config_root(logging.DEBUG)
   else:
     google.logging_utils.config_root()
   # TODO(timurrrr): valgrind tools may use -v/--verbose as well
 
   ret = RunTool(sys.argv)
   sys.exit(ret)