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