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

Unified Diff: tools/test_pictures.py

Issue 153083003: remove tools/test_rendering.py and tools/test_pictures.py (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 11 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | tools/test_rendering.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/test_pictures.py
diff --git a/tools/test_pictures.py b/tools/test_pictures.py
deleted file mode 100644
index 8cf7410136e58e083153c19c115dc525fa85b607..0000000000000000000000000000000000000000
--- a/tools/test_pictures.py
+++ /dev/null
@@ -1,105 +0,0 @@
-'''
-Copyright 2012 Google Inc.
-
-Use of this source code is governed by a BSD-style license that can be
-found in the LICENSE file.
-
-Compares the rendererings of serialized SkPictures to expected images.
-
-Launch with --help to see more information.
-
-TODO(epoger): We believe this tool is no longer used, so we have disabled it
-and will remove it on 1 Feb 2014 if nobody objects.
-See https://code.google.com/p/skia/issues/detail?id=1943#c2
-'''
-# common Python modules
-import os
-import optparse
-import sys
-import shutil
-import tempfile
-
-# modules declared within this same directory
-import test_rendering
-
-USAGE_STRING = 'Usage: %s input... expectedDir'
-HELP_STRING = '''
-
-Takes input SkPicture files and renders them as PNG files, and then compares
-those resulting PNG files against PNG files found in expectedDir.
-
-Each instance of "input" can be either a file (name must end in .skp), or a
-directory (in which case this script will process all .skp files within the
-directory).
-'''
-
-def ModeParse(option, opt_str, value, parser):
- """Parses the --mode option of the commandline.
-
- The --mode option will either take in three parameters (if tile or
- pow2tile) or a single parameter (otherwise).
- """
- result = [value]
- if value == "tile":
- if (len(parser.rargs) < 2):
- raise optparse.OptionValueError(("--mode tile mising width"
- " and/or height parameters"))
- result.extend(parser.rargs[:2])
- del parser.rargs[:2]
- elif value == "pow2tile":
- if (len(parser.rargs) < 2):
- raise optparse.OptionValueError(("--mode pow2tile mising minWidth"
- " and/or height parameters"))
- result.extend(parser.rargs[:2])
- del parser.rargs[:2]
-
- setattr(parser.values, option.dest, result)
-
-
-def Main(args):
- """Allow other scripts to call this script with fake command-line args.
-
- @param The commandline argument list
- """
- print ('We believe this tool is no longer used, so we have disabled it '
- 'and will remove it on 1 Feb 2014 if nobody objects. See '
- 'https://code.google.com/p/skia/issues/detail?id=1943#c2')
- sys.exit(-1)
-
- parser = optparse.OptionParser(USAGE_STRING % '%prog' + HELP_STRING)
- parser.add_option('--render_dir', dest='render_dir',
- help = ("specify the location to output the rendered files."
- " Default is a temp directory."))
- parser.add_option('--diff_dir', dest='diff_dir',
- help = ("specify the location to output the diff files."
- " Default is a temp directory."))
- parser.add_option('--mode', dest='mode', type='string',
- action="callback", callback=ModeParse,
- help = ("specify how rendering is to be done."))
- parser.add_option('--device', dest='device',
- help = ("specify the device to render to."))
-
- options, arguments = parser.parse_args(args)
-
- if (len(arguments) < 3):
- print("Expected at least one input and one ouput folder.")
- parser.print_help()
- sys.exit(-1)
-
- inputs = arguments[1:-1]
- expected_dir = arguments[-1]
-
- extra_args = ''
-
- if (options.mode is not None):
- extra_args += ' --mode %s' % ' '.join(options.mode)
-
- if (options.device is not None):
- extra_args += ' --device %s' % options.device
-
- test_rendering.TestRenderSkps(inputs, expected_dir, options.render_dir,
- options.diff_dir, 'render_pictures',
- extra_args)
-
-if __name__ == '__main__':
- Main(sys.argv)
« no previous file with comments | « no previous file | tools/test_rendering.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698