Enable Dr. Memory to run javascript, pixel, and corpus tests

tools/drmemory/scripts/pdfium_tests.py

 #!/usr/bin/env python
 # 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.
 
 ''' Runs various chrome tests through valgrind_test.py.'''
 
 import glob
 import logging
 import optparse
@@ skipping 251 matching lines @@
     cmd = self._DefaultCommand(tool, None, self._args)
     self.SetupLdPath(False)
     return tool.Run(cmd, None)
 
   def TestPDFiumUnitTests(self):
     return self.SimpleTest("pdfium_unittests", "pdfium_unittests")
 
   def TestPDFiumEmbedderTests(self):
     return self.SimpleTest("pdfium_embeddertests", "pdfium_embeddertests")
 
+  def TestPDFiumTest(self, script_name):
+    # Build the commandline in 'cmd'.

[Lei Zhang, 2015/11/20 00:17:47]

nit: command line

[zhaoqin1, 2015/11/21 04:20:46]

Done.

+    # It's going to be roughly
+    #  python valgrind_test.py ...
+    # but we'll use the --indirect_pdfium_test flag to valgrind_test.py
+    # to avoid valgrinding python.
+
+    # Start by building the valgrind_test.py commandline.
+    tool = valgrind_test.CreateTool(self._options.valgrind_tool)
+    cmd = self._DefaultCommand(tool)
+    cmd.append("--trace_children")
+    cmd.append("--indirect_pdfium_test")
+    cmd.append("--ignore_exit_code")
+    # Now build script_cmd, the run_corpus_tests commandline.
+    script = os.path.join(self._source_dir, "..", "testing", "tools", script_name)

[Lei Zhang, 2015/11/20 00:17:47]

80 chars / line, ditto on line 290.

[Lei Zhang, 2015/11/20 00:17:47]

I think the |self._source_dir| calculation is wrong. You should not need to
compensate for it with another '..' here.

[zhaoqin1, 2015/11/21 04:20:45]

line 57:
self._source_dir = os.path.dirname(os.path.dirname(script_dir))
I am using pdfium/tools/drmemory/scripts/ as script_dir, so need update there
instead.

[zhaoqin1, 2015/11/21 04:20:45]

Done.

+    script_cmd = ["python", script]
+    if self._options.build_dir:
+      script_cmd.extend(["--build-dir", self._options.build_dir])
+    # TODO(zhaoqin): figure out why wrapper does not work with test_one_file_parallel
+    # in run_corpus_tests.py, now it only runs in single process mode.
+    if script_name == "run_corpus_tests.py":
+        script_cmd.extend(["-j", "1"])
+    # Now run script_cmd with the wrapper in cmd
+    cmd.extend(["--"])

[Lei Zhang, 2015/11/20 00:17:47]

Just append("--")

[zhaoqin1, 2015/11/21 04:20:45]

Done.

+    cmd.extend(script_cmd)
+
+    ret = tool.Run(cmd, "layout", min_runtime_in_seconds=0)
+    return ret
+
+  def TestPDFiumJavascript(self):
+    return self.TestPDFiumTest("run_javascript_tests.py")
+
+  def TestPDFiumPixel(self):
+    return self.TestPDFiumTest("run_pixel_tests.py")
+
+  def TestPDFiumCorpus(self):
+    return self.TestPDFiumTest("run_corpus_tests.py")

[Lei Zhang, 2015/11/20 00:17:47]

blank line after this

[zhaoqin1, 2015/11/21 04:20:45]

Done.

   # The known list of tests.
   _test_list = {
     "cmdline" : RunCmdLine,
     "pdfium_unittests": TestPDFiumUnitTests,
     "pdfium_embeddertests": TestPDFiumEmbedderTests,
+    "pdfium_javascript": TestPDFiumJavascript,

[Lei Zhang, 2015/11/20 00:17:47]

Can we sort the list?

[zhaoqin1, 2015/11/21 04:20:45]

Done.

+    "pdfium_pixel": TestPDFiumPixel,
+    "pdfium_corpus": TestPDFiumCorpus,
   }
 
 
 def _main():
   parser = optparse.OptionParser("usage: %prog -b <dir> -t <test> "
                                  "[-t <test> ...]")
 
   parser.add_option("--help-tests", dest="help_tests", action="store_true",
                     default=False, help="List all available tests")
   parser.add_option("-b", "--build-dir",
@@ skipping 57 matching lines @@
 
   for t in options.test:
     tests = ChromeTests(options, args, t)
     ret = tests.Run()
     if ret: return ret
   return 0
 
 
 if __name__ == "__main__":
   sys.exit(_main())