IndexedDB: Have IDBCursor and IDBRequest explicitly break ref cycles

Source/modules/indexeddb/IDBAny.h

Index: Source/modules/indexeddb/IDBAny.h
diff --git a/Source/modules/indexeddb/IDBAny.h b/Source/modules/indexeddb/IDBAny.h
index 21caadcf8786315e850c85cae22524014644e1c2..1d5c5d2f4eab69a21b99270469d9be96294a2a63 100644
--- a/Source/modules/indexeddb/IDBAny.h
+++ b/Source/modules/indexeddb/IDBAny.h
@@ -91,14 +91,14 @@ public:
 
     Type type() const { return m_type; }
     // Use type() to figure out which one of these you're allowed to call.
-    PassRefPtr<DOMStringList> domStringList();
-    PassRefPtr<IDBCursor> idbCursor();
-    PassRefPtr<IDBCursorWithValue> idbCursorWithValue();
-    PassRefPtr<IDBDatabase> idbDatabase();
-    PassRefPtr<IDBFactory> idbFactory();
-    PassRefPtr<IDBIndex> idbIndex();
-    PassRefPtr<IDBObjectStore> idbObjectStore();
-    PassRefPtr<IDBTransaction> idbTransaction();
+    DOMStringList* domStringList();

[jsbell, 2013/09/06 00:27:51]

This may seem unrelated, but without this the request->getResultCursor() call
churns the cursor's reference count, which ends up with recursing in the
cursor's deref().

Using PassRefPtr<> here is not necessary since these accessors never mint a new
RefPtr<>.
 

+    IDBCursor* idbCursor();
+    IDBCursorWithValue* idbCursorWithValue();
+    IDBDatabase* idbDatabase();
+    IDBFactory* idbFactory();
+    IDBIndex* idbIndex();
+    IDBObjectStore* idbObjectStore();
+    IDBTransaction* idbTransaction();
     const ScriptValue& scriptValue();
     int64_t integer();
     const String& string();