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

Side by Side Diff: slave/skia_slave_scripts/run_gm.py

Issue 295753002: upload SKP renderings that did not match expectations (Closed) Base URL: https://skia.googlesource.com/buildbot.git@master
Patch Set: Created 6 years, 7 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
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """ Run the Skia GM executable. """ 6 """ Run the Skia GM executable. """
7 7
8 from build_step import BuildStep 8 from build_step import BuildStep
9 import build_step 9 import build_step
10 import os 10 import os
11 import sys 11 import sys
12 12
13 13
14 JSON_SUMMARY_FILENAME = 'actual-results.json' 14 JSON_SUMMARY_FILENAME = 'actual-results.json'
15 15
16 16
17 class RunGM(BuildStep): 17 class RunGM(BuildStep):
18 def __init__(self, timeout=9600, no_output_timeout=9600, **kwargs): 18 def __init__(self, timeout=9600, no_output_timeout=9600, **kwargs):
19 super(RunGM, self).__init__( 19 super(RunGM, self).__init__(
20 timeout=timeout, no_output_timeout=no_output_timeout, **kwargs) 20 timeout=timeout, no_output_timeout=no_output_timeout, **kwargs)
21 21
22 def _Run(self): 22 def _Run(self):
23 # TODO(epoger): I think some of these os.path.join() calls need to be
epoger 2014/05/19 19:35:40 Am I right about this? If so, I can make the chan
borenet 2014/05/19 21:04:38 Yeah, I think that's true. I think we get lucky b
epoger 2014/05/19 22:07:24 Done.
24 # self._flavor_utils.DevicePathJoin() instead. Seems to work anyway.
23 output_dir = os.path.join(self._device_dirs.GMActualDir(), 25 output_dir = os.path.join(self._device_dirs.GMActualDir(),
24 self._builder_name) 26 self._builder_name)
25 cmd = ['--verbose', 27 cmd = ['--verbose',
26 '--writeChecksumBasedFilenames', 28 '--writeChecksumBasedFilenames',
27 # Don't bother writing out image files that match our expectations-- 29 # Don't bother writing out image files that match our expectations--
28 # we know that previous runs have already uploaded those! 30 # we know that previous runs have already uploaded those!
29 '--mismatchPath', output_dir, 31 '--mismatchPath', output_dir,
30 '--missingExpectationsPath', output_dir, 32 '--missingExpectationsPath', output_dir,
31 '--writeJsonSummaryPath', os.path.join(output_dir, 33 '--writeJsonSummaryPath', os.path.join(output_dir,
32 JSON_SUMMARY_FILENAME), 34 JSON_SUMMARY_FILENAME),
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 '~scaled_tilemodes', 99 '~scaled_tilemodes',
98 '~convexpaths', 100 '~convexpaths',
99 '~clipped-bitmap', 101 '~clipped-bitmap',
100 '~xfermodes3']) 102 '~xfermodes3'])
101 103
102 self._flavor_utils.RunFlavoredCmd('gm', cmd) 104 self._flavor_utils.RunFlavoredCmd('gm', cmd)
103 105
104 106
105 if '__main__' == __name__: 107 if '__main__' == __name__:
106 sys.exit(BuildStep.RunBuildStep(RunGM)) 108 sys.exit(BuildStep.RunBuildStep(RunGM))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698