commit | a53512f33454a992d9d165ed3005e3cead919b15 | [log] [tgz] |
---|---|---|
author | Oleh Prypin <oprypin@webrtc.org> | Thu Jan 10 14:50:00 2019 |
committer | Commit Bot <commit-bot@chromium.org> | Thu Jan 10 14:50:00 2019 |
tree | 307176e02dac3274d9b89988e0861526c1858d27 | |
parent | 68f199581bd4f37f1500718a943344f9edf671c2 [diff] |
WebRTC: Refactor performance upload code (mainly Android) * Fix badly overridden methods that had chunks of code copied from chromium_tests. * Fail dashboard upload only if the test itself succeeds. Otherwise rely on composite step failure. * Remove 'idempotent' config, specify it by default. * Remove 'add_adb_path' config, specify the command line arg directly. Note: * "validate_task_results" doesn't sound like the right place to *process* the results, but it's the most suitable place to hook into chromium_tests, certainly better than the previous overridden private method. Bug: chromium:755660 Change-Id: I2beaeb135b8624b6c752a6af8fb6c8a2d679aa6e Reviewed-on: https://chromium-review.googlesource.com/c/1377536 Commit-Queue: Oleh Prypin <oprypin@chromium.org> Reviewed-by: John Budorick <jbudorick@chromium.org>
Hi build contributor! If you do any change in scripts/master/
or touching any master's html/
directories, you must restart master.chromium.fyi first and ensure that it still works before restarting other masters.
If you're here to make a change to ‘recipes’ (the code located in scripts/slave/recipes*
), please take a look at the README for more information pertaining to recipes.