Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make suspend threads for on-demand fatal thread recording configurable #14391

Merged
merged 4 commits into from
Feb 18, 2025

Conversation

themiswang
Copy link
Contributor

@themiswang themiswang commented Jan 28, 2025

For on-demand fatal we don't want to do suspend then record the threads info.

eg. If there is an audio thread, user might experiencing audio glitch during crash recording

Also added gating flag to settings, backend change: cl/725699502

@themiswang themiswang changed the title No suspend threads for on-demand fatal Not suspend threads for on-demand fatal thread recording Jan 28, 2025
@themiswang themiswang marked this pull request as ready for review February 11, 2025 23:17
Copy link
Contributor

@samedson samedson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@themiswang
Copy link
Contributor Author

Verified backend change is here, merge the code. https://screenshot.googleplex.com/5DfnhAbfQYQDCNY

@themiswang themiswang merged commit 4ed50a9 into main Feb 18, 2025
53 checks passed
@themiswang themiswang deleted the onDemandNonsuspend branch February 18, 2025 18:26
@themiswang themiswang changed the title Not suspend threads for on-demand fatal thread recording Make suspend threads for on-demand fatal thread recording configurable Feb 18, 2025
themiswang added a commit that referenced this pull request Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants