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

Side by Side Diff: build/android/buildbot/bb_device_steps.py

Issue 607803003: Fix the WebKit Android bot failures. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 | no next file » | 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/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 import collections 6 import collections
7 import glob 7 import glob
8 import hashlib 8 import hashlib
9 import json 9 import json
10 import os 10 import os
(...skipping 266 matching lines...) Expand 10 before | Expand all | Expand 10 after
277 args.append('-p') 277 args.append('-p')
278 if official_build: 278 if official_build:
279 # The option needs to be assigned 'True' as it does not have an action 279 # The option needs to be assigned 'True' as it does not have an action
280 # associated with it. 280 # associated with it.
281 args.append('--official-build') 281 args.append('--official-build')
282 282
283 RunCmd(['build/android/test_runner.py', 'instrumentation'] + args, 283 RunCmd(['build/android/test_runner.py', 'instrumentation'] + args,
284 flunk_on_failure=flunk_on_failure) 284 flunk_on_failure=flunk_on_failure)
285 285
286 286
287 def RunWebkitLint(target): 287 def RunWebkitLint():
288 """Lint WebKit's TestExpectation files.""" 288 """Lint WebKit's TestExpectation files."""
289 bb_annotations.PrintNamedStep('webkit_lint') 289 bb_annotations.PrintNamedStep('webkit_lint')
290 RunCmd([SrcPath(os.path.join(BK_SCRIPTS_DIR, 'lint-test-expectations')), 290 RunCmd([SrcPath(os.path.join(BK_SCRIPTS_DIR, 'lint-test-expectations')),
291 '--target', target]) 291 '--android'])
Dirk Pranke 2014/09/26 20:36:17 We shouldn't actually need either ['--target', tar
292 292
293 293
294 def RunWebkitLayoutTests(options): 294 def RunWebkitLayoutTests(options):
295 """Run layout tests on an actual device.""" 295 """Run layout tests on an actual device."""
296 bb_annotations.PrintNamedStep('webkit_tests') 296 bb_annotations.PrintNamedStep('webkit_tests')
297 cmd_args = [ 297 cmd_args = [
298 '--no-show-results', 298 '--no-show-results',
299 '--no-new-test-results', 299 '--no-new-test-results',
300 '--full-results-html', 300 '--full-results-html',
301 '--clobber-old-results', 301 '--clobber-old-results',
(...skipping 185 matching lines...) Expand 10 before | Expand all | Expand 10 after
487 RunTestSuites(options, suites) 487 RunTestSuites(options, suites)
488 488
489 489
490 def RunInstrumentationTests(options): 490 def RunInstrumentationTests(options):
491 for test in INSTRUMENTATION_TESTS.itervalues(): 491 for test in INSTRUMENTATION_TESTS.itervalues():
492 RunInstrumentationSuite(options, test) 492 RunInstrumentationSuite(options, test)
493 493
494 494
495 def RunWebkitTests(options): 495 def RunWebkitTests(options):
496 RunTestSuites(options, ['webkit_unit_tests', 'blink_heap_unittests']) 496 RunTestSuites(options, ['webkit_unit_tests', 'blink_heap_unittests'])
497 RunWebkitLint(options.target) 497 RunWebkitLint()
498 498
499 499
500 def RunGPUTests(options): 500 def RunGPUTests(options):
501 revision = _GetRevision(options) 501 revision = _GetRevision(options)
502 builder_name = options.build_properties.get('buildername', 'noname') 502 builder_name = options.build_properties.get('buildername', 'noname')
503 503
504 bb_annotations.PrintNamedStep('pixel_tests') 504 bb_annotations.PrintNamedStep('pixel_tests')
505 RunCmd(['content/test/gpu/run_gpu_test.py', 505 RunCmd(['content/test/gpu/run_gpu_test.py',
506 'pixel', 506 'pixel',
507 '--browser', 507 '--browser',
(...skipping 225 matching lines...) Expand 10 before | Expand all | Expand 10 after
733 733
734 if options.coverage_bucket: 734 if options.coverage_bucket:
735 setattr(options, 'coverage_dir', 735 setattr(options, 'coverage_dir',
736 os.path.join(CHROME_OUT_DIR, options.target, 'coverage')) 736 os.path.join(CHROME_OUT_DIR, options.target, 'coverage'))
737 737
738 MainTestWrapper(options) 738 MainTestWrapper(options)
739 739
740 740
741 if __name__ == '__main__': 741 if __name__ == '__main__':
742 sys.exit(main(sys.argv)) 742 sys.exit(main(sys.argv))
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698