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: honor user-defined Downloads directory #29938
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test? 😇
@nornagon what would you envision for a test here? Our session downloader shows a dialog if a user doesn't explicitly set the save path, and all calls to |
58f69e4
to
6f6aa90
Compare
Ah, okay, I wasn't sure if there was a way to interrogate this value without user interaction. There's |
Unrelated failure. |
Release Notes Persisted
|
I have automatically backported this PR to "12-x-y", please check out #29966 |
I have automatically backported this PR to "14-x-y", please check out #29967 |
I have automatically backported this PR to "13-x-y", please check out #29968 |
Description of Change
Closes #29909.
Makes it such that
ElectronBrowserClient::GetDefaultDownloadDirectory()
honors user-defined Downloads directories by calling out to platform-specific download logic defined in ChromiumChecklist
npm test
passesRelease Notes
Notes: Fixes an issue where Electron would sometimes not honor the user-defined Downloads directory.