Skip to content
This repository has been archived by the owner on Jan 4, 2019. It is now read-only.

Commit

Permalink
Merge pull request #615 from brave/download_pref_fix
Browse files Browse the repository at this point in the history
Remove DIR_DEFAULT_DOWNLOADS_SAFE which is covered by DownloadPrefs
  • Loading branch information
darkdh committed Jun 20, 2018
1 parent 22ecf9e commit bba8c8a
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions atom/browser/api/atom_api_dialog.cc
Original file line number Diff line number Diff line change
Expand Up @@ -156,11 +156,6 @@ void ShowDialog(const file_dialog::DialogSettings& settings,
NOTREACHED();
}
}
// This is only useful on platforms that support
// DIR_DEFAULT_DOWNLOADS_SAFE.
if (!PathService::Get(chrome::DIR_DEFAULT_DOWNLOADS_SAFE, &default_path)) {
NOTREACHED();
}
file_type_info.include_all_files = settings.include_all_files;
file_type_info.extension_description_overrides =
settings.extension_description_overrides;
Expand Down

0 comments on commit bba8c8a

Please sign in to comment.