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

Side by Side Diff: gm/rebaseline_server/compare_rendered_pictures_test.py

Issue 343813006: Drop --clone from skia (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 5 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 unified diff | Download patch
« no previous file with comments | « no previous file | tools/tests/render_pictures_test.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 2
3 """ 3 """
4 Copyright 2014 Google Inc. 4 Copyright 2014 Google Inc.
5 5
6 Use of this source code is governed by a BSD-style license that can be 6 Use of this source code is governed by a BSD-style license that can be
7 found in the LICENSE file. 7 found in the LICENSE file.
8 8
9 Test compare_rendered_pictures.py 9 Test compare_rendered_pictures.py
10 10
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 for skpname, redvalue in skpdict.iteritems(): 71 for skpname, redvalue in skpdict.iteritems():
72 self._run_skpmaker( 72 self._run_skpmaker(
73 output_path=os.path.join(out_path, skpname), red=redvalue) 73 output_path=os.path.join(out_path, skpname), red=redvalue)
74 74
75 # TODO(epoger): Add --mode tile 256 256 --writeWholeImage to the unittest, 75 # TODO(epoger): Add --mode tile 256 256 --writeWholeImage to the unittest,
76 # and fix its result! (imageURLs within whole-image entries are wrong when 76 # and fix its result! (imageURLs within whole-image entries are wrong when
77 # I tried adding that) 77 # I tried adding that)
78 binary = self.find_path_to_program('render_pictures') 78 binary = self.find_path_to_program('render_pictures')
79 return subprocess.check_output([ 79 return subprocess.check_output([
80 binary, 80 binary,
81 '--clone', '1',
82 '--config', '8888', 81 '--config', '8888',
83 '-r', out_path, 82 '-r', out_path,
84 '--writeChecksumBasedFilenames', 83 '--writeChecksumBasedFilenames',
85 '--writeJsonSummaryPath', os.path.join(out_path, 'summary.json'), 84 '--writeJsonSummaryPath', os.path.join(out_path, 'summary.json'),
86 '--writePath', out_path]) 85 '--writePath', out_path])
87 86
88 def _run_skpmaker(self, output_path, red=0, green=0, blue=0, 87 def _run_skpmaker(self, output_path, red=0, green=0, blue=0,
89 width=640, height=400): 88 width=640, height=400):
90 """Runs the skpmaker binary to generate SKP with known characteristics. 89 """Runs the skpmaker binary to generate SKP with known characteristics.
91 90
(...skipping 19 matching lines...) Expand all
111 """Mock version of BaseComparisons.get_timestamp() for testing.""" 110 """Mock version of BaseComparisons.get_timestamp() for testing."""
112 return 12345678 111 return 12345678
113 112
114 113
115 def main(): 114 def main():
116 base_unittest.main(CompareRenderedPicturesTest) 115 base_unittest.main(CompareRenderedPicturesTest)
117 116
118 117
119 if __name__ == '__main__': 118 if __name__ == '__main__':
120 main() 119 main()
OLDNEW
« no previous file with comments | « no previous file | tools/tests/render_pictures_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698