Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(299)

Side by Side Diff: tools/valgrind/valgrind_test.py

Issue 4713003: Revert a workaround & exclude filters for TSan/Win failures... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « tools/valgrind/gtest_exclude/net_unittests.gtest-tsan_win32.txt ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 # Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 # valgrind_test.py 6 # valgrind_test.py
7 7
8 """Runs an exe through Valgrind and puts the intermediate files in a 8 """Runs an exe through Valgrind and puts the intermediate files in a
9 directory. 9 directory.
10 """ 10 """
(...skipping 689 matching lines...) Expand 10 before | Expand all | Expand 10 after
700 return proc 700 return proc
701 701
702 def Analyze(self, check_sanity=False): 702 def Analyze(self, check_sanity=False):
703 filenames = glob.glob(self.temp_dir + "/tsan.*") 703 filenames = glob.glob(self.temp_dir + "/tsan.*")
704 analyzer = tsan_analyze.TsanAnalyzer(self._source_dir) 704 analyzer = tsan_analyze.TsanAnalyzer(self._source_dir)
705 ret = analyzer.Report(filenames, check_sanity) 705 ret = analyzer.Report(filenames, check_sanity)
706 if ret != 0: 706 if ret != 0:
707 logging.info(self.INFO_MESSAGE) 707 logging.info(self.INFO_MESSAGE)
708 return ret 708 return ret
709 709
710 # TODO(timurrrr): temporary workaround for http://crbug.com/62512
711 def Execute(self):
712 super(ThreadSanitizerWindows, self).Execute()
713 return 0
714
715 710
716 class DrMemory(BaseTool): 711 class DrMemory(BaseTool):
717 """Dr.Memory 712 """Dr.Memory
718 Dynamic memory error detector for Windows. 713 Dynamic memory error detector for Windows.
719 714
720 http://dynamorio.org/drmemory.html 715 http://dynamorio.org/drmemory.html
721 It is not very mature at the moment, some things might not work properly. 716 It is not very mature at the moment, some things might not work properly.
722 """ 717 """
723 718
724 def __init__(self): 719 def __init__(self):
(...skipping 200 matching lines...) Expand 10 before | Expand all | Expand 10 after
925 platform_name = sys.platform + "(Unknown)" 920 platform_name = sys.platform + "(Unknown)"
926 raise RuntimeError, "Unknown tool (tool=%s, platform=%s)" % (tool_name, 921 raise RuntimeError, "Unknown tool (tool=%s, platform=%s)" % (tool_name,
927 platform_name) 922 platform_name)
928 923
929 def CreateTool(tool): 924 def CreateTool(tool):
930 return ToolFactory().Create(tool) 925 return ToolFactory().Create(tool)
931 926
932 if __name__ == '__main__': 927 if __name__ == '__main__':
933 logging.error(sys.argv[0] + " can not be run from command line") 928 logging.error(sys.argv[0] + " can not be run from command line")
934 sys.exit(1) 929 sys.exit(1)
OLDNEW
« no previous file with comments | « tools/valgrind/gtest_exclude/net_unittests.gtest-tsan_win32.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698