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
fix: crash when showing item in folder on DevTools #33024
Conversation
0344393
to
5adf590
Compare
5adf590
to
202569d
Compare
Release Notes Persisted
|
I was unable to backport this PR to "16-x-y" cleanly; |
I have automatically backported this PR to "17-x-y", please check out #33196 |
I have automatically backported this PR to "18-x-y", please check out #33197 |
@codebytere has manually backported this PR to "16-x-y", please check out #33313 |
Description of Change
Closes #33023.
Fixes an issue where clicking "Open in Containing Folder" in the Sources tab in Devtools caused a crash.
We were incorrectly passing an empty callback instead of a callback which calls
ShowItemInFolder
. Chromium's call also opens the directory containing the file in their call toplatform_util::OpenItem
, so we correct that on our end (since we don't pass an option to differentiate) by passing the file's parent directory toOpenPath
Checklist
npm test
passesRelease Notes
Notes: Fixed an issue where clicking "Open in Containing Folder" in the Sources tab in Devtools caused a crash.