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

Side by Side Diff: chrome/telemetry_unittests.isolate

Issue 1471673003: [Telemetry] Set the number of processes to use for telemetry_unittests on win platform to be 1 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Group all the common files to a same condition Created 5 years, 1 month 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 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 'includes': [ 5 'includes': [
6 '../tools/perf/chrome_telemetry_build/telemetry_chrome_test.isolate', 6 '../tools/perf/chrome_telemetry_build/telemetry_chrome_test.isolate',
7 ], 7 ],
8 'conditions': [ 8 'conditions': [
9 ['OS=="mac" or OS=="win"', { 9 ['OS=="mac" or OS=="win"', {
10 'variables': { 10 'variables': {
11 'files': [ 11 'files': [
12 # Other dependencies of the tests and their harness. 12 # Other dependencies of the tests and their harness.
13 '../testing/test_env.py', 13 '../testing/test_env.py',
14 '../testing/xvfb.py', 14 '../testing/xvfb.py',
15 '../testing/scripts/common.py', 15 '../testing/scripts/common.py',
16 '../testing/scripts/run_telemetry_as_googletest.py', 16 '../testing/scripts/run_telemetry_as_googletest.py',
17 ], 17 ],
18 }
19 }],
20 ['OS=="mac"', {
21 'variables': {
18 'command': [ 22 'command': [
19 '../testing/scripts/run_telemetry_as_googletest.py', 23 '../testing/scripts/run_telemetry_as_googletest.py',
20 '../tools/telemetry/run_tests', 24 '../tools/telemetry/run_tests',
21 '-v', 25 '-v',
22 '--chrome-root', 26 '--chrome-root',
23 '..' 27 '..'
M-A Ruel 2015/11/23 16:28:40 add comma here, so this error won't be repeated.
nednguyen 2015/11/23 16:29:51 Done.
24 ], 28 ],
25 }, 29 },
26 }], 30 }],
31 ['OS=="win"', {
32 'variables': {
33 'command': [
34 '../testing/scripts/run_telemetry_as_googletest.py',
35 '../tools/telemetry/run_tests',
36 '-v',
37 '--chrome-root',
38 '..'
M-A Ruel 2015/11/23 16:28:40 missing comma
nednguyen 2015/11/23 16:29:51 Done.
39 # crbug.com/559154
40 # TODO(nednguyen): Remove --jobs=1 once the bug is fixed to speed up
41 # the test run.
42 '--job=1',
43 ],
44 },
45 }],
27 ] 46 ]
28 } 47 }
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