Chromium Code Reviews| Index: scripts/slave/recipe_modules/chromium_android/api.py |
| diff --git a/scripts/slave/recipe_modules/chromium_android/api.py b/scripts/slave/recipe_modules/chromium_android/api.py |
| index 9b2008567e1e357194ec3885bcaefd190df0acfa..87baf7e9bcf720f0dec15814748e34636d1aff3e 100644 |
| --- a/scripts/slave/recipe_modules/chromium_android/api.py |
| +++ b/scripts/slave/recipe_modules/chromium_android/api.py |
| @@ -1237,16 +1237,26 @@ class AndroidApi(recipe_api.RecipeApi): |
| 'android_webview', 'tools', 'cts_config') |
| cts_filenames_json = self.m.file.read( |
| 'Fetch CTS filename data', |
| - _CTS_CONFIG_SRC_PATH.join('webview_cts_gcs_path.json'), |
| + _CTS_CONFIG_SRC_PATH.join('webview_cts_gcs_path_new.json'), |
| test_data=''' |
| { |
| "arm_64": { |
| - "L": "cts_arm64_L.zip" |
| + "L": { |
| + "filename": "cts_arm64_L.zip", |
| + "_origin": "aosp-lollipop-mr1-cts-dev@12345" |
| + }, |
| + "N": { |
| + "filename": "cts_arm64_N.zip", |
| + "_origin": "aosp-nougat-cts-release@67890" |
| + } |
| } |
| }''') |
| cts_filenames = self.m.json.loads(cts_filenames_json) |
| + result = self.m.step.active_result |
| + result.presentation.logs['webview_cts_gcs_path.json'] = ( |
| + cts_filenames_json.split('\n')) |
|
mikecase (-- gone --)
2016/10/27 15:54:09
nit: split('\n') --> splitlines()
|
| try: |
| - cts_filename = cts_filenames[arch][android_platform] |
| + cts_filename = cts_filenames[arch][android_platform]["filename"] |
| except KeyError: |
| raise self.m.step.StepFailure( |
| 'No CTS test found to use for arch:%s android:%s' % ( |