Coalesce the removeDownload reset with the following refreshList

Moves the ModelResetGuard out of the try-catch so it also wraps the
refreshList() call below. Without this, one reset fires when the guard
destructs at the end of the try block and another fires from
refreshList's own guard, producing two resets where one is sufficient.

Author: Al12rs

src/downloadmanager.cpp

@@ -914,10 +914,11 @@ void DownloadManager::removeDownload(int index, bool deleteFile)
     return;
   }
 
+  // coalesce the removal and the subsequent refresh into one reset
+  ModelResetGuard guard(*this);
+
   try {
     DirWatcherManager::Guard dirWatcherGuard = m_DirWatcher.scopedGuard();
-    // coalesce the removal and the subsequent refresh into one reset
-    ModelResetGuard guard(*this);
 
     if (index < 0) {
       bool removeAll            = (index == -1);