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

Side by Side Diff: scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py

Issue 2226753003: Remove the android_isolate_path option from LocalGTestTest. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: thakis/sergiyb comment Created 4 years, 4 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 | scripts/slave/recipe_modules/chromium_tests/chromium_recipe.md » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 from . import steps 5 from . import steps
6 6
7 RESULTS_URL = 'https://chromeperf.appspot.com' 7 RESULTS_URL = 'https://chromeperf.appspot.com'
8 8
9 9
10 KITCHEN_TEST_SPEC = { 10 KITCHEN_TEST_SPEC = {
(...skipping 1585 matching lines...) Expand 10 before | Expand all | Expand 10 after
1596 'testing': { 'platform': 'linux', }, 1596 'testing': { 'platform': 'linux', },
1597 # Workaround so that recipes doesn't add random build targets to our 1597 # Workaround so that recipes doesn't add random build targets to our
1598 # compile line. We want to build everything. 1598 # compile line. We want to build everything.
1599 'add_tests_as_compile_targets': False, 1599 'add_tests_as_compile_targets': False,
1600 'root_devices': True, 1600 'root_devices': True,
1601 'test_generators': [ 1601 'test_generators': [
1602 steps.generate_gtest, 1602 steps.generate_gtest,
1603 steps.generate_junit_test, 1603 steps.generate_junit_test,
1604 ], 1604 ],
1605 'tests': [ 1605 'tests': [
1606 steps.GTestTest( 1606 steps.GTestTest('components_browsertests', android_tool='asan'),
1607 'components_browsertests',
1608 android_isolate_path='components/components_browsertests.isolate',
1609 android_tool='asan'),
1610 steps.GTestTest('gfx_unittests', android_tool='asan'), 1607 steps.GTestTest('gfx_unittests', android_tool='asan'),
Dirk Pranke 2016/08/09 23:52:16 why are these even still listed here, rather than
jbudorick 2016/08/09 23:55:28 AFAICT yes, because the gtest generator doesn't ta
1611 ], 1608 ],
1612 }, 1609 },
1613 'ClangToTMac': { 1610 'ClangToTMac': {
1614 'chromium_config': 'clang_tot_mac', 1611 'chromium_config': 'clang_tot_mac',
1615 'gclient_config': 'chromium', 1612 'gclient_config': 'chromium',
1616 'chromium_apply_config': ['mb'], 1613 'chromium_apply_config': ['mb'],
1617 'chromium_config_kwargs': { 1614 'chromium_config_kwargs': {
1618 'BUILD_CONFIG': 'Release', 1615 'BUILD_CONFIG': 'Release',
1619 'TARGET_BITS': 64, 1616 'TARGET_BITS': 64,
1620 }, 1617 },
(...skipping 657 matching lines...) Expand 10 before | Expand all | Expand 10 after
2278 steps.generate_gtest, 2275 steps.generate_gtest,
2279 steps.generate_script, 2276 steps.generate_script,
2280 steps.generate_isolated_script, 2277 steps.generate_isolated_script,
2281 ], 2278 ],
2282 'testing': { 'platform': 'win', }, 2279 'testing': { 'platform': 'win', },
2283 'use_isolate': True, 2280 'use_isolate': True,
2284 'enable_swarming': True, 2281 'enable_swarming': True,
2285 }, 2282 },
2286 }, 2283 },
2287 } 2284 }
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/chromium_tests/chromium_recipe.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698