From bba8c8a21a2f989caf17d26bee8df98ba88ab13e Mon Sep 17 00:00:00 2001 From: Anthony Tseng Date: Wed, 20 Jun 2018 11:30:39 -0700 Subject: [PATCH] Merge pull request #615 from brave/download_pref_fix Remove DIR_DEFAULT_DOWNLOADS_SAFE which is covered by DownloadPrefs --- atom/browser/api/atom_api_dialog.cc | 5 ----- 1 file changed, 5 deletions(-) diff --git a/atom/browser/api/atom_api_dialog.cc b/atom/browser/api/atom_api_dialog.cc index 8ea4fca139..777fffdd86 100644 --- a/atom/browser/api/atom_api_dialog.cc +++ b/atom/browser/api/atom_api_dialog.cc @@ -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;