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>
14 files changed
tree: 307176e02dac3274d9b89988e0861526c1858d27
  1. infra/
  2. masters/
  3. scripts/
  4. site_config/
  5. slave/
  6. tests/
  7. third_party/
  8. .gitattributes
  9. .gitignore
  10. .vpython
  11. codereview.settings
  12. DEPS
  13. environment.cfg.py
  14. LICENSE
  15. OWNERS
  16. PRESUBMIT.py
  17. README.md
  18. WATCHLISTS
README.md

build

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.

Recipes

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.