[PGO] Dump all profiles prior to Close()

Devtool client is closed prior to pgo profile dumping, causing a null
reference to the client. This swaps the order such that the client
remains active while dumping, before we close it off.

Bug: chromium:1242138
Change-Id: I7ee7ec3e6a26f9602b16d237dd7c44670eb72dc6
Reviewed-on: https://chromium-review.googlesource.com/c/catapult/+/3307236
Reviewed-by: John Chen <johnchen@chromium.org>
Reviewed-by: Sébastien Marchand <sebmarchand@chromium.org>
Commit-Queue: Jeff Yoon <jeffyoon@google.com>
1 file changed
tree: 6bfdb932126af268bdcede2e3c2ee44afd783976
  1. bin/
  2. catapult_build/
  3. common/
  4. dashboard/
  5. dependency_manager/
  6. devil/
  7. docs/
  8. experimental/
  9. firefighter/
  10. hooks/
  11. infra/
  12. netlog_viewer/
  13. systrace/
  14. telemetry/
  15. third_party/
  16. trace_processor/
  17. tracing/
  18. web_page_replay_go/
  19. .eslintignore
  20. .eslintrc
  21. .gcloudignore
  22. .gitignore
  23. .vpython
  24. .vpython3
  25. AUTHORS
  26. BUILD.gn
  27. codereview.settings
  28. CONTRIBUTING.md
  29. DIR_METADATA
  30. generate_telemetry_build.py
  31. LICENSE
  32. navbar.md
  33. OWNERS
  34. PRESUBMIT.py
  35. pylintrc
  36. README.md
  37. WATCHLISTS
README.md

Catapult

Catapult is the home for several performance tools that span from gathering, displaying and analyzing performance data. This includes:

These tools were created by Chromium developers for performance analysis, testing, and monitoring of Chrome, but they can also be used for analyzing and monitoring websites, and eventually Android apps.

Contributing

Please see our contributor's guide