From f92c82ead14152e7ac5c91a3b8febf6ceb430290 Mon Sep 17 00:00:00 2001
From: Daniel Schadt <kingdread@gmx.de>
Date: Tue, 20 Dec 2016 14:52:39 +0100
Subject: [PATCH 1/3] downloads: don't remember dir for temp downloads

Fixes #2173
---
 qutebrowser/browser/downloads.py                    | 8 ++++++--
 qutebrowser/browser/webengine/webenginedownloads.py | 3 ++-
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/qutebrowser/browser/downloads.py b/qutebrowser/browser/downloads.py
index 12e766ecb..74fb12edb 100644
--- a/qutebrowser/browser/downloads.py
+++ b/qutebrowser/browser/downloads.py
@@ -564,13 +564,16 @@ class AbstractDownloadItem(QObject):
         """Set a temporary file when opening the download."""
         raise NotImplementedError
 
-    def _set_filename(self, filename, *, force_overwrite=False):
+    def _set_filename(self, filename, *, force_overwrite=False,
+                      remember_directory=True):
         """Set the filename to save the download to.
 
         Args:
             filename: The full filename to save the download to.
                       None: special value to stop the download.
             force_overwrite: Force overwriting existing files.
+            remember_directory: If True, remember the directory for future
+                                downloads.
         """
         global last_used_directory
         filename = os.path.expanduser(filename)
@@ -600,7 +603,8 @@ class AbstractDownloadItem(QObject):
                                                   os.path.expanduser('~'))
 
         self.basename = os.path.basename(self._filename)
-        last_used_directory = os.path.dirname(self._filename)
+        if remember_directory:
+            last_used_directory = os.path.dirname(self._filename)
 
         log.downloads.debug("Setting filename to {}".format(filename))
         if force_overwrite:
diff --git a/qutebrowser/browser/webengine/webenginedownloads.py b/qutebrowser/browser/webengine/webenginedownloads.py
index e78e939e1..44b326d36 100644
--- a/qutebrowser/browser/webengine/webenginedownloads.py
+++ b/qutebrowser/browser/webengine/webenginedownloads.py
@@ -96,7 +96,8 @@ class DownloadItem(downloads.AbstractDownloadItem):
         raise downloads.UnsupportedOperationError
 
     def _set_tempfile(self, fileobj):
-        self._set_filename(fileobj.name, force_overwrite=True)
+        self._set_filename(fileobj.name, force_overwrite=True,
+                           remember_directory=False)
 
     def _ensure_can_set_filename(self, filename):
         state = self._qt_item.state()

From a24bcd424330c68de2cd5fde97352d8f18046613 Mon Sep 17 00:00:00 2001
From: Daniel Schadt <kingdread@gmx.de>
Date: Tue, 20 Dec 2016 15:57:00 +0100
Subject: [PATCH 2/3] add a test for remembering the temp download dir

---
 tests/end2end/features/downloads.feature | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/tests/end2end/features/downloads.feature b/tests/end2end/features/downloads.feature
index b6a38da29..85917864e 100644
--- a/tests/end2end/features/downloads.feature
+++ b/tests/end2end/features/downloads.feature
@@ -424,6 +424,20 @@ Feature: Downloading things from a website.
         And I open data/downloads/download2.bin without waiting
         Then the download prompt should be shown with "(tmpdir)/download2.bin"
 
+    # https://github.com/The-Compiler/qutebrowser/issues/2173
+
+    Scenario: Remembering the temporary download directory (issue 2173)
+        When I set storage -> prompt-download-directory to true
+        And I set storage -> remember-download-directory to true
+        And I open data/downloads/download.bin without waiting
+        And I wait for the download prompt for "*"
+        And I run :prompt-accept (tmpdir)
+        And I open data/downloads/download.bin without waiting
+        And I wait for the download prompt for "*"
+        And I directly open the download
+        And I open data/downloads/download.bin without waiting
+        Then the download prompt should be shown with "(tmpdir)/download.bin"
+
     # Overwriting files
 
     Scenario: Not overwriting an existing file

From 4b7818589d00ded482ee084dfd9722beb56d73a9 Mon Sep 17 00:00:00 2001
From: Daniel Schadt <kingdread@gmx.de>
Date: Thu, 22 Dec 2016 14:22:37 +0100
Subject: [PATCH 3/3] make test independent of previous tests

---
 tests/end2end/features/downloads.feature | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/end2end/features/downloads.feature b/tests/end2end/features/downloads.feature
index 85917864e..2c04cb5bf 100644
--- a/tests/end2end/features/downloads.feature
+++ b/tests/end2end/features/downloads.feature
@@ -428,6 +428,7 @@ Feature: Downloading things from a website.
 
     Scenario: Remembering the temporary download directory (issue 2173)
         When I set storage -> prompt-download-directory to true
+        And I set completion -> download-path-suggestion to both
         And I set storage -> remember-download-directory to true
         And I open data/downloads/download.bin without waiting
         And I wait for the download prompt for "*"