Bug 333848 - add full-featured user-defined functions and progress handlers to storage. Patch by Lev Serebryakov <blacklion@gmail.com>. r=cbiesinger, sr=cbiesinger

netwerk/cache/src/nsDiskCacheDeviceSQL.cpp

@@ -46,6 +46,7 @@
 #include "nsString.h"
 #include "nsPrintfCString.h"
 #include "nsCRT.h"
+#include "nsIVariant.h"
 
 #include "mozIStorageService.h"
 #include "mozIStorageStatement.h"
@@ -202,9 +203,11 @@ GetCacheDataFile(nsIFile *cacheDir, const char *cid, const char *key,
 }
 
 NS_IMETHODIMP
-nsOfflineCacheEvictionFunction::OnFunctionCall(mozIStorageValueArray *values)
+nsOfflineCacheEvictionFunction::OnFunctionCall(mozIStorageValueArray *values, nsIVariant **_retval)
 {
   LOG(("nsOfflineCacheDevice::EvictionObserver\n"));
+  
+  *_retval = nsnull;
 
   PRUint32 numEntries;
   nsresult rv = values->GetNumEntries(&numEntries);
@@ -762,7 +765,7 @@ nsOfflineCacheDevice::Init()
     new nsOfflineCacheEvictionFunction(this);
   if (!evictionFunction) return NS_ERROR_OUT_OF_MEMORY;
 
-  rv = mDB->CreateFunction("cache_eviction_observer", 3, evictionFunction);
+  rv = mDB->CreateFunction(NS_LITERAL_CSTRING("cache_eviction_observer"), 3, evictionFunction);
   NS_ENSURE_SUCCESS(rv, rv);
 
   // create all (most) of our statements up front