Bug 696483 - Improve error handling. r=mak. a=Ms2ger.

Add an if to avoid a common case and reportError if something else happens.

--HG--
extra : rebase_source : 7a5952e8ba8e5f14d7b7a667d945d754051f9f05

toolkit/components/satchel/nsFormHistory.js

@@ -894,9 +894,14 @@ FormHistory.prototype = {
 
         this._dbFinalize();
 
-        // Close the connection, ignore 'already closed' error
-        // FIXME (bug 696483): we should reportError in here.
-        try { this.dbConnection.close(); } catch(e) {}
+        if (this.dbConnection !== undefined) {
+            try {
+                this.dbConnection.close();
+            } catch (e) {
+                Components.utils.reportError(e);
+            }
+        }
+
         this.dbFile.remove(false);
     }
 };