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

Unified Diff: tools/test_pictures.py

Issue 135783003: deprecate tools/test_rendering.py and tools/test_pictures.py (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: March -> Feb 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
index 7ccb3b87315d7d34fa71a657373cc7972270a730..8cf7410136e58e083153c19c115dc525fa85b607 100644
--- a/tools/test_pictures.py
+++ b/tools/test_pictures.py
@@ -8,7 +8,8 @@ Compares the rendererings of serialized SkPictures to expected images.
Launch with --help to see more information.
-TODO(epoger): Combine with overlapping tools/tests/render_pictures_test.py .
+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
@@ -60,6 +61,11 @@ def Main(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."
« 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