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

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

Issue 2306853003: Add resource_sizes to chromium_perf steps for Android (Closed)
Patch Set: fix typo Created 4 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
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 import datetime 5 import datetime
6 import re 6 import re
7 import string 7 import string
8 8
9 9
10 class Test(object): 10 class Test(object):
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 'archive build', 92 'archive build',
93 self.gs_bucket, 93 self.gs_bucket,
94 gs_acl=self.gs_acl, 94 gs_acl=self.gs_acl,
95 ) 95 )
96 96
97 @staticmethod 97 @staticmethod
98 def compile_targets(_): 98 def compile_targets(_):
99 return [] 99 return []
100 100
101 101
102 class ResourceSizesStep(Test):
ghost stip (do not use) 2016/09/07 21:30:50 aw man, another step type? do we really need this?
jbudorick 2016/09/07 21:35:43 The last time a new step vs an existing one (i.e.,
103 def __init__(self, compile_target, apk_name):
104 self.compile_target = compile_target
105 self.apk_name = apk_name
106
107 def run(self, api, suffix, test_filter=None):
108 apk_path = api.chromium_android.apk_path(self.apk_name)
109 return api.chromium_android.resource_sizes(apk_path, chartjson_file=True)
110
111 def compile_targets(self, api):
112 return [self.compile_target]
113
114 @property
115 def name(self):
116 return 'resource_sizes' # pragma: no cover
117
118
102 class SizesStep(Test): 119 class SizesStep(Test):
103 def __init__(self, results_url, perf_id): 120 def __init__(self, results_url, perf_id):
104 self.results_url = results_url 121 self.results_url = results_url
105 self.perf_id = perf_id 122 self.perf_id = perf_id
106 123
107 def run(self, api, suffix, test_filter=None): 124 def run(self, api, suffix, test_filter=None):
108 return api.chromium.sizes(self.results_url, self.perf_id) 125 return api.chromium.sizes(self.results_url, self.perf_id)
109 126
110 @staticmethod 127 @staticmethod
111 def compile_targets(_): 128 def compile_targets(_):
(...skipping 1633 matching lines...) Expand 10 before | Expand all | Expand 10 after
1745 args=args) 1762 args=args)
1746 api.gsutil.upload( 1763 api.gsutil.upload(
1747 temp_output_dir.join( 1764 temp_output_dir.join(
1748 '%s-android-chrome.json' % timestamp_string), 1765 '%s-android-chrome.json' % timestamp_string),
1749 'chromium-annotated-tests', 'android') 1766 'chromium-annotated-tests', 'android')
1750 1767
1751 GOMA_TESTS = [ 1768 GOMA_TESTS = [
1752 GTestTest('base_unittests'), 1769 GTestTest('base_unittests'),
1753 GTestTest('content_unittests'), 1770 GTestTest('content_unittests'),
1754 ] 1771 ]
OLDNEW
« no previous file with comments | « scripts/slave/recipe_modules/chromium_tests/chromium_perf.py ('k') | scripts/slave/recipes/android/builder.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698