Revert "Analyze the analyze logic by logging any failing tests that would've"

This reverts commit 5392ad369c4519c3394ed326d112c52b0640b868.

Reason for revert: Crashing on windows -- no 'echo' program

Original change's description:
> Analyze the analyze logic by logging any failing tests that would've
> been excluded.
>
> Bug: 923016
> Change-Id: I16b1065c7253bff0f1d01898f7f6c9f8b7d0a93c
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/2337756
> Commit-Queue: Gregory Guterman <guterman@google.com>
> Reviewed-by: Garrett Beaty <gbeaty@chromium.org>
> Reviewed-by: Stephen Martinis <martiniss@chromium.org>

TBR=martiniss@chromium.org,gbeaty@chromium.org,guterman@google.com,infra-scoped@luci-project-accounts.iam.gserviceaccount.com

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 923016
Change-Id: I37ba11170e2436cf1df0057e0111296f971af2fe
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/2442147
Reviewed-by: Gregory Guterman <guterman@google.com>
Reviewed-by: Garrett Beaty <gbeaty@chromium.org>
Commit-Queue: Gregory Guterman <guterman@google.com>
11 files changed
tree: c0e6839368236b34958e0b7400bd67925a86c4d3
  1. infra/
  2. recipes/
  3. scripts/
  4. site_config/
  5. third_party/
  6. .gitattributes
  7. .gitignore
  8. .style.yapf
  9. .vpython
  10. .yapfignore
  11. codereview.settings
  12. CROS_OWNERS
  13. DEPS
  14. environment.cfg.py
  15. LICENSE
  16. OWNERS
  17. PRESUBMIT.py
  18. README.md
  19. WATCHLISTS
README.md

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.

Style

The preferred style is PEP8 with two-space indent; that is, the Chromium Python style. Functions use lowercase_with_underscores, with the exception of the special functions RunSteps and RunTests in recipes. Use yapf (git cl format --no-clang-format) to autoformat new code.