Age | Commit message (Collapse) | Author |
|
Change the defaults for the directory arguments to be lists, since
that is what argparse gives when the user provides a value. Also
changed type of os.mkdirs to os.makedirs.
Bug:
Change-Id: I04c15323efedc9483327c01bab18d45ffffdc21b
Reviewed-on: https://pdfium-review.googlesource.com/9590
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
|
|
It appears that some of my final changes to coverage_report.py didn't
make it into the patch that I submitted. This means that the current
version of the script in the repo does not work.
Bug:
Change-Id: I8822840f8e3d9b6f75224bb811209860c8191026
Reviewed-on: https://pdfium-review.googlesource.com/9511
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
|
|
Adds script coverage_report.py for generating code coverage
reports. This script depends on the llvm-gcov wrapper script and the
lcovrc config file that are checked in with it.
BUG=pdfium:797
Change-Id: I0416349186835c0e4dd7f4907d2b472a4406e027
Reviewed-on: https://pdfium-review.googlesource.com/7810
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
|
|
The DrMemory bots have been removed, remove the config and infrastructure.
BUG=chromium:655521
Change-Id: I065d717b11d615a6dc981b79a8caefd8783b105b
Reviewed-on: https://pdfium-review.googlesource.com/2494
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>
|
|
Add all the necessary files used for MSan build and run.
BUG=pdfium:1
Review-Url: https://codereview.chromium.org/2384353002
|
|
We have cleared all the leaks in our code. There is no need for
those suppressions. Plus, most of them are obsolete due to code
refactoring.
Still keep this file around so that suppressions can be added if needed
in the future.
BUG=pdfium:242,pdfium:260
Review-Url: https://codereview.chromium.org/2360743002
|
|
The Dr Memory tests currently run -j1 which makes corpus tests take a long time.
Previously, multip-process corpus would override stdout which caused conflict
when using Dr. Memory. That has been resolved so enable more parallelism.
Review-Url: https://codereview.chromium.org/1981153003
|
|
This version contains fixes that are required for running binaries
compiled with Visual Studio 2015.
R=zhaoqin@chromium.org
Review URL: https://codereview.chromium.org/1933493002 .
|
|
R=dsinclair@chromium.org,zhaoqin@chromium.org
Review URL: https://codereview.chromium.org/1890593002
|
|
Also defines ADDRESS_SANITIZER and LEAK_SANITIZER for ASan builds.
TBR=thestig@chromium.org
Original Review URL: https://codereview.chromium.org/1540433004 .
(cherry picked from commit 38cecc343561395cad8a33889f91c1a254ca1f73)
Review URL: https://codereview.chromium.org/1540433005 .
|
|
corpus tests with Dr. Memory
TBR=thestig@chromium.org
BUG=pdfium:238
Review URL: https://codereview.chromium.org/1486413005 .
(cherry picked from commit be3ad4b56b38221b40e2a1f0ee56e371738a752a)
Review URL: https://codereview.chromium.org/1495443005 .
|
|
R=tsepez@chromium.org
BUG=PDFium:238
Review URL: https://codereview.chromium.org/1471073011 .
(cherry picked from commit 7e413d168beba325078aa4b484729ad22919a2c3)
Review URL: https://codereview.chromium.org/1482953003 .
|
|
Visual Studio sometimes uses new instead of new[] for array allocation,
which causes Dr. Memory report invalid-heap-arg errors.
Suppress such errors since it is not developers' problems.
TBR=tsepez@chromium.org
BUG=pdfium:287
Review URL: https://codereview.chromium.org/1473913002 .
(cherry picked from commit 1fb4bc47e216c9caebdf7bc8479d45285ee7693b)
Review URL: https://codereview.chromium.org/1475023003 .
|
|
buildbots
- update pdfium_tests.py to setup Dr. Memory so it can be used directly without
pdfium_tests.bat on buildbots
R=thestig@chromium.org
BUG=PDFium:238
Review URL: https://codereview.chromium.org/1478453002 .
(cherry picked from commit abea9d857e8e197d0095940f882b0db6a785d825)
Review URL: https://codereview.chromium.org/1467183005 .
|
|
- widening suppression for DrM-i#471
TBR=thestig@chromium.org
BUG=pdfium:238
Review URL: https://codereview.chromium.org/1467373003 .
(cherry picked from commit 3b1bd5cbbcc1a606275fe98e813a37531572f374)
Review URL: https://codereview.chromium.org/1465373002 .
|
|
- add DrMemoryWrapper in common.py to adjust Dr. Memory wrapper
- add --wrapper option to run_*_tests.py for Dr. Mempry wrapper
- update run_*_tests.py to handle Dr. Memory wrapper
- add TestPDFiumTest in pdfium_tests.py to support run_*_tests.py
- remove ValgrindTool in valgrind_tests.py
R=thestig@chromium.org
BUG=pdfium:238
Review URL: https://codereview.chromium.org/1464453003 .
(cherry picked from commit 36476923ae5eb8e9283e605f3c85ee8811c86014)
Review URL: https://codereview.chromium.org/1465333002 .
|
|
- add suppressions.txt in tools/drmemory/
- update pdfium_tests.py to read suppression file
R=thestig@chromium.org
BUG=pdfium:238
Review URL: https://codereview.chromium.org/1454193002 .
(cherry picked from commit 76b0104270b0a7f3ce7d40ddde2b6d6c8cb944db)
Review URL: https://codereview.chromium.org/1455293003 .
|
|
This CL adds DrMemory-Windows-sfx.exe into tools/drmemory.
It also adds a set of scripts from chromium/src/tools/valgrind for
running Dr. Memory. Now we can run pdfium tests with Dr. Memory:
$ .\tools\drmemory\scripts\pdfium_tests.bat -b out\Debug -t pdfium_unittests
More update on scripts is required, but this should be a good start.
Changes from chromium/src/tools/valgrind
- rename chrome_tests.py to pdfium_tests.py
- update pdfium_tests.bat
- remove chrome tests in pdfium_tests.py
- remove memcheck code in valgrind_test.py
R=thestig@chromium.org
BUG=pdfium:238
Review URL: https://codereview.chromium.org/1452293002 .
(cherry picked from commit 4dcf313c46c176e5afa1328f5f07fce4e614366d)
Review URL: https://codereview.chromium.org/1452373004 .
|