From a34258b3d79b5a71997100af1920ba9aca0ed909 Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Wed, 27 May 2020 18:06:20 +0200
Subject: [PATCH] ProtocolServer: Forget downloads after they are stopped

Stopping means the client no longer cares about the download, so we
should just forget about it in the server.
---
 Services/ProtocolServer/ClientConnection.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Services/ProtocolServer/ClientConnection.cpp b/Services/ProtocolServer/ClientConnection.cpp
index f688dc58a8..b9f78e2520 100644
--- a/Services/ProtocolServer/ClientConnection.cpp
+++ b/Services/ProtocolServer/ClientConnection.cpp
@@ -26,8 +26,8 @@
 
 #include <AK/Badge.h>
 #include <AK/SharedBuffer.h>
-#include <ProtocolServer/Download.h>
 #include <ProtocolServer/ClientConnection.h>
+#include <ProtocolServer/Download.h>
 #include <ProtocolServer/Protocol.h>
 #include <ProtocolServer/ProtocolClientEndpoint.h>
 
@@ -78,6 +78,7 @@ OwnPtr<Messages::ProtocolServer::StopDownloadResponse> ClientConnection::handle(
     bool success = false;
     if (download) {
         download->stop();
+        m_downloads.remove(message.download_id());
         success = true;
     }
     return make<Messages::ProtocolServer::StopDownloadResponse>(success);