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
chore: add a TRACE call for crash_reporter::Start()
#34268
chore: add a TRACE call for crash_reporter::Start()
#34268
Conversation
Initializing the crashReporter takes around 620 milliseconds on Intel macOS. I have sent a CL to crashpad to partially fix the performance issue in https://chromium-review.googlesource.com/c/crashpad/crashpad/+/3641386. It would be beneficial to log the performance impact of this function in the traces in case this slows down in the future. Signed-off-by: Darshan Sen <raisinten@gmail.com>
I think this is semver-minor, so shouldn't this be backported to all the supported release lines? |
@RaisinTen this doesn't affect the API surfaced to end users, so we'd classify this as |
Release Notes Persisted
|
I have automatically backported this PR to "17-x-y", please check out #34324 |
I have automatically backported this PR to "18-x-y", please check out #34325 |
I have automatically backported this PR to "19-x-y", please check out #34326 |
chore: add a TRACE call for crash_reporter::Start() Initializing the crashReporter takes around 620 milliseconds on Intel macOS. I have sent a CL to crashpad to partially fix the performance issue in https://chromium-review.googlesource.com/c/crashpad/crashpad/+/3641386. It would be beneficial to log the performance impact of this function in the traces in case this slows down in the future. Signed-off-by: Darshan Sen <raisinten@gmail.com>
Description of Change
Initializing the crashReporter takes around 620 milliseconds on Intel
macOS. I have sent a CL to crashpad to partially fix the performance
issue in https://chromium-review.googlesource.com/c/crashpad/crashpad/+/3641386.
It would be beneficial to log the performance impact of this function in
the traces in case this slows down in the future.
Signed-off-by: Darshan Sen raisinten@gmail.com
Checklist
npm test
passesRelease Notes
Notes: Added a TRACE call named
crash_reporter::Start
under theelectron
category forcrash_reporter::Start()
.