[go: up one dir, main page]

Optimize the StopRecording logic in logcat monotor.

The code used to send a stop nonce to logcat, then copied the tmp record
file to the output file.
However when the device is not responding or crashed, the script will
error out at the step of sending the stop nonce due to timeout, and lead
to an empty output file, due to the tmp record file not being copied.

This CL fixes this issue by wrapping the "stop nonce" step under a try
except clause to ensure the file copying is done properly.

Bug: chromium:425720132
Change-Id: Iec1364a1252a28d9c2d4afa096eb728df9348773
Reviewed-on: https://chromium-review.googlesource.com/c/catapult/+/7029919
Reviewed-by: Ben Pastene <bpastene@chromium.org>
Commit-Queue: Haiyang Pan <hypan@google.com>
1 file changed
tree: 7996da76f3af4b2e3082e8bdcc5eb91a237133cc
  1. bin/
  2. catapult_build/
  3. common/
  4. dashboard/
  5. dependency_manager/
  6. devil/
  7. docs/
  8. experimental/
  9. hooks/
  10. infra/
  11. netlog_viewer/
  12. perf_issue_service/
  13. skia_bridge/
  14. systrace/
  15. telemetry/
  16. third_party/
  17. trace_processor/
  18. tracing/
  19. web_page_replay_go/
  20. .eslintignore
  21. .eslintrc
  22. .gcloudignore
  23. .gitignore
  24. .style.yapf
  25. .vpython3
  26. .yapfignore
  27. AUTHORS
  28. BUILD.gn
  29. codereview.settings
  30. CONTRIBUTING.md
  31. DIR_METADATA
  32. generate_telemetry_build.py
  33. LICENSE
  34. navbar.md
  35. OWNERS
  36. PRESUBMIT.py
  37. pylintrc
  38. README.md
  39. 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