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

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

Issue 24076020: [android] Remove unnecessary install of AndroidWebViewTest.apk on bots. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 3 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 | Annotate | Revision Log
« 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 multiprocessing 9 import multiprocessing
10 import os 10 import os
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 def InstallApk(options, test, print_step=False): 155 def InstallApk(options, test, print_step=False):
156 """Install an apk to all phones. 156 """Install an apk to all phones.
157 157
158 Args: 158 Args:
159 options: options object 159 options: options object
160 test: An I_TEST namedtuple 160 test: An I_TEST namedtuple
161 print_step: Print a buildbot step 161 print_step: Print a buildbot step
162 """ 162 """
163 if print_step: 163 if print_step:
164 bb_annotations.PrintNamedStep('install_%s' % test.name.lower()) 164 bb_annotations.PrintNamedStep('install_%s' % test.name.lower())
165 # TODO(gkanwar): Quick hack to make sure AndroidWebViewTest.apk is replaced
166 # before AndroidWebView.apk is. This can be removed once the bots cycle.
167 args = ['--apk', '%s.apk' % test.test_apk]
168 if options.target == 'Release':
169 args.append('--release')
170
171 RunCmd(['build/android/adb_install_apk.py'] + args, halt_on_failure=True)
172 165
173 args = ['--apk', test.apk, '--apk_package', test.apk_package] 166 args = ['--apk', test.apk, '--apk_package', test.apk_package]
174 if options.target == 'Release': 167 if options.target == 'Release':
175 args.append('--release') 168 args.append('--release')
176 169
177 RunCmd(['build/android/adb_install_apk.py'] + args, halt_on_failure=True) 170 RunCmd(['build/android/adb_install_apk.py'] + args, halt_on_failure=True)
178 171
179 172
180 def RunInstrumentationSuite(options, test, flunk_on_failure=True, 173 def RunInstrumentationSuite(options, test, flunk_on_failure=True,
181 python_only=False, official_build=False): 174 python_only=False, official_build=False):
(...skipping 316 matching lines...) Expand 10 before | Expand all | Expand 10 after
498 setattr(options, 'target', options.factory_properties.get('target', 'Debug')) 491 setattr(options, 'target', options.factory_properties.get('target', 'Debug'))
499 if options.coverage_bucket: 492 if options.coverage_bucket:
500 setattr(options, 'coverage_dir', 493 setattr(options, 'coverage_dir',
501 os.path.join(CHROME_OUT_DIR, options.target, 'coverage')) 494 os.path.join(CHROME_OUT_DIR, options.target, 'coverage'))
502 495
503 MainTestWrapper(options) 496 MainTestWrapper(options)
504 497
505 498
506 if __name__ == '__main__': 499 if __name__ == '__main__':
507 sys.exit(main(sys.argv)) 500 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