From a893ba34e2d82ffcf870a0c41cbe7f2f5c263b1d Mon Sep 17 00:00:00 2001
From: "sdwilsh%shawnwilsher.com" <sdwilsh%shawnwilsher.com>
Date: Mon, 2 Jul 2007 17:29:59 +0000
Subject: [PATCH] Bug 385876 - DM should cache the query that
 nsDownload::UpdateDB uses.  Patch by Ryan Jones <sciguyryan@gmail.com>. 
 r=sdwilsh

---
 .../downloads/src/nsDownloadManager.cpp       | 20 +++++++++----------
 .../downloads/src/nsDownloadManager.h         |  2 ++
 2 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/toolkit/components/downloads/src/nsDownloadManager.cpp b/toolkit/components/downloads/src/nsDownloadManager.cpp
index 4d9975a9554..d6deabf2048 100644
--- a/toolkit/components/downloads/src/nsDownloadManager.cpp
+++ b/toolkit/components/downloads/src/nsDownloadManager.cpp
@@ -63,7 +63,6 @@
 #include "nsEmbedCID.h"
 #include "mozStorageCID.h"
 #include "mozIStorageService.h"
-#include "mozIStorageStatement.h"
 #include "mozStorageHelper.h"
 #include "nsIMutableArray.h"
 #include "nsIAlertsService.h"
@@ -436,6 +435,13 @@ nsDownloadManager::Init()
   mObserverService->AddObserver(this, "quit-application-requested", PR_FALSE);
   mObserverService->AddObserver(this, "offline-requested", PR_FALSE);
 
+  rv = mDBConn->CreateStatement(NS_LITERAL_CSTRING(
+    "UPDATE moz_downloads "
+    "SET name = ?1, source = ?2, target = ?3, startTime = ?4, endTime = ?5,"
+    "state = ?6 "
+    "WHERE id = ?7"), getter_AddRefs(mUpdateDownloadStatement));
+  NS_ENSURE_SUCCESS(rv, rv);
+
   return NS_OK;
 }
 
@@ -1581,16 +1587,10 @@ nsDownload::UpdateDB()
   NS_ASSERTION(mID, "Download ID is stored as zero.  This is bad!");
   NS_ASSERTION(mDownloadManager, "Egads!  We have no download manager!");
 
-  nsCOMPtr<mozIStorageStatement> stmt;
-  nsresult rv = mDownloadManager->mDBConn->CreateStatement(NS_LITERAL_CSTRING(
-    "UPDATE moz_downloads "
-    "SET name = ?1, source = ?2, target = ?3, startTime = ?4, endTime = ?5,"
-    "state = ?6 "
-    "WHERE id = ?7"), getter_AddRefs(stmt));
-  NS_ENSURE_SUCCESS(rv, rv);
-  
+  mozIStorageStatement *stmt = mDownloadManager->mUpdateDownloadStatement;
+
   // name
-  rv = stmt->BindStringParameter(0, mDisplayName);
+  nsresult rv = stmt->BindStringParameter(0, mDisplayName);
   NS_ENSURE_SUCCESS(rv, rv);
 
   // source
diff --git a/toolkit/components/downloads/src/nsDownloadManager.h b/toolkit/components/downloads/src/nsDownloadManager.h
index 6b1a48e5e2b..b67bc4eef2f 100644
--- a/toolkit/components/downloads/src/nsDownloadManager.h
+++ b/toolkit/components/downloads/src/nsDownloadManager.h
@@ -60,6 +60,7 @@
 #include "nsIMIMEInfo.h"
 #include "nsITimer.h"
 #include "mozIStorageConnection.h"
+#include "mozIStorageStatement.h"
 #include "nsISupportsArray.h"
 #include "nsCOMArray.h"
 #include "nsArrayEnumerator.h"
@@ -184,6 +185,7 @@ private:
   nsCOMPtr<mozIStorageConnection> mDBConn;
   nsCOMArray<nsDownload> mCurrentDownloads;
   nsCOMPtr<nsIObserverService> mObserverService;
+  nsCOMPtr<mozIStorageStatement> mUpdateDownloadStatement;
 
   friend class nsDownload;
 };