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

Unified Diff: scripts/slave/recipe_modules/chromium/chromium_perf.py

Issue 873403002: Add support for cc_perftests and other non-telemetry gtest based tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Rebase again + coverage. Created 5 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « scripts/slave/recipe_modules/chromium/api.py ('k') | scripts/slave/recipe_modules/chromium/steps.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/chromium/chromium_perf.py
diff --git a/scripts/slave/recipe_modules/chromium/chromium_perf.py b/scripts/slave/recipe_modules/chromium/chromium_perf.py
index 94a4821f3fcef1c11786a83b44a7bb1aaebbacb6..bb4215ce1d9e835323a5ddf1e085101eae255157 100644
--- a/scripts/slave/recipe_modules/chromium/chromium_perf.py
+++ b/scripts/slave/recipe_modules/chromium/chromium_perf.py
@@ -11,7 +11,7 @@ def _GetTargetName(platform, target_bits):
def _Spec(platform, parent_builder, perf_id, index, num_shards, target_bits):
return {
- 'disable_tests': True,
+ 'disable_tests': False,
'bot_type': 'tester',
'chromium_config_kwargs': {
'BUILD_CONFIG': 'Release',
@@ -22,11 +22,17 @@ def _Spec(platform, parent_builder, perf_id, index, num_shards, target_bits):
'testing': {
'platform': platform,
},
+ 'test_spec_file': 'chromium.perf.json',
+ 'test_generators': [
+ steps.generate_script,
+ ],
'tests': [
steps.DynamicPerfTests(
_GetTargetName(platform, target_bits).lower(),
perf_id, index, num_shards),
],
+ 'perf-id': perf_id,
+ 'results-url': 'https://chromeperf.appspot.com',
}
@@ -249,4 +255,6 @@ for k, v in _AndroidSpecs.iteritems():
'tests': [
steps.AndroidPerfTests(v['perf_id'], v['num_device_shards']),
],
+ 'perf-id': v['perf_id'],
+ 'results-url': 'https://chromeperf.appspot.com',
}
« no previous file with comments | « scripts/slave/recipe_modules/chromium/api.py ('k') | scripts/slave/recipe_modules/chromium/steps.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698