diff options
author | Tom Sepez <tsepez@chromium.org> | 2015-04-09 12:35:01 -0700 |
---|---|---|
committer | Tom Sepez <tsepez@chromium.org> | 2015-04-09 12:35:01 -0700 |
commit | 308e05e5f06bd8daabaed12c211f6e072810f8de (patch) | |
tree | 5fa92ed72fb93ff260b22c175f13cb8a775778b5 /testing/tools/pngdiffer.py | |
parent | 1ed2ceb70476b135a3dedbb45549d6b3bc6ecdea (diff) | |
download | pdfium-308e05e5f06bd8daabaed12c211f6e072810f8de.tar.xz |
Consider platform-specific expected .png files.
Rolls DEPS to pull in the first windows-specific .png files, and
unsupresses the corresponding tests.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1072613003
Diffstat (limited to 'testing/tools/pngdiffer.py')
-rwxr-xr-x | testing/tools/pngdiffer.py | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/testing/tools/pngdiffer.py b/testing/tools/pngdiffer.py index dc65b4717f..7a7e8ddbc6 100755 --- a/testing/tools/pngdiffer.py +++ b/testing/tools/pngdiffer.py @@ -10,9 +10,11 @@ import sys class PNGDiffer(): ACTUAL_TEMPLATE = '.pdf.%d.png' EXPECTED_TEMPLATE = '_expected' + ACTUAL_TEMPLATE + PLATFORM_EXPECTED_TEMPLATE = '_expected_%s' + ACTUAL_TEMPLATE def __init__(self, finder): self.pdfium_diff_path = finder.ExecutablePath('pdfium_diff') + self.os_name = finder.os_name def HasDifferences(self, input_filename, source_dir, working_dir): input_root, _ = os.path.splitext(input_filename) @@ -20,12 +22,18 @@ class PNGDiffer(): working_dir, input_root + self.ACTUAL_TEMPLATE) expected_path_template = os.path.join( source_dir, input_root + self.EXPECTED_TEMPLATE) + platform_expected_path_template = os.path.join( + source_dir, input_root + self.PLATFORM_EXPECTED_TEMPLATE) i = 0 try: while True: - actual_path = actual_path_template % i; - expected_path = expected_path_template % i; - if not os.path.exists(expected_path): + actual_path = actual_path_template % i + expected_path = expected_path_template % i + platform_expected_path = ( + platform_expected_path_template % (self.os_name, i)) + if os.path.exists(platform_expected_path): + expected_path = platform_expected_path + elif not os.path.exists(expected_path): if i == 0: print "WARNING: no expected results files for " + input_filename break |