prefs-always-prompt-for-download-directory-by-defaul.patch 1.7 KB

12345678910111213141516171819202122232425262728293031323334353637383940
  1. From 93010fd16c1c9f01a06eab18055bcab54b028cc8 Mon Sep 17 00:00:00 2001
  2. From: Jan Engelhardt <jengelh@inai.de>
  3. Date: Fri, 13 Feb 2015 01:04:21 +0100
  4. Subject: [PATCH 28/66] prefs: always prompt for download directory by default
  5. If the user opens a link, one of two things might happen. The URI
  6. resolves to a "web page" and will be displayed, then everything is
  7. good. Or it has a MIME type like application/* or so that causes the
  8. browser to consider it a file download instead.
  9. In that case, one of two things might happen. The browser may display
  10. a dialog box asking explicitly for an action, then everything is
  11. good. Or it does not and instead readily stores the file on disk.
  12. (Modern browsers also download the file in the background when waiting
  13. for the dialog box confirmation, but that is a separate tuning knob.)
  14. When the file is chosen to be immediately stored on disk without user
  15. interaction, the visual feedback for this may be rather miniscule.
  16. And then, you have to open a terminal or file manager again just to
  17. move the file to the location you wanted it to be in in the first
  18. place.
  19. TLDR: Do not let files sneak onto the disk too easily.
  20. ---
  21. chrome/browser/download/download_prefs.cc | 2 +-
  22. 1 file changed, 1 insertion(+), 1 deletion(-)
  23. --- a/chrome/browser/download/download_prefs.cc
  24. +++ b/chrome/browser/download/download_prefs.cc
  25. @@ -235,7 +235,7 @@ void DownloadPrefs::RegisterProfilePrefs
  26. user_prefs::PrefRegistrySyncable* registry) {
  27. registry->RegisterBooleanPref(
  28. prefs::kPromptForDownload,
  29. - false,
  30. + true,
  31. user_prefs::PrefRegistrySyncable::SYNCABLE_PREF);
  32. registry->RegisterStringPref(prefs::kDownloadExtensionsToOpen, std::string());
  33. registry->RegisterBooleanPref(prefs::kDownloadDirUpgraded, false);