IndexedDB: Have IDBCursor and IDBRequest explicitly break ref cycles

Source/modules/indexeddb/IDBTransaction.h

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 61 matching lines @@
     bool isReadOnly() const { return m_mode == IndexedDB::TransactionReadOnly; }
     bool isVersionChange() const { return m_mode == IndexedDB::TransactionVersionChange; }
 
     // Implement the IDBTransaction IDL
     const String& mode() const;
     IDBDatabase* db() const { return m_database.get(); }
     PassRefPtr<DOMError> error() const { return m_error; }
     PassRefPtr<IDBObjectStore> objectStore(const String& name, ExceptionState&);
     void abort(ExceptionState&);
 
-    class OpenCursorNotifier {
-    public:
-        OpenCursorNotifier(PassRefPtr<IDBTransaction>, IDBCursor*);
-        ~OpenCursorNotifier();
-        void cursorFinished();
-    private:
-        RefPtr<IDBTransaction> m_transaction;
-        IDBCursor* m_cursor;
-    };
-
     void registerRequest(IDBRequest*);
     void unregisterRequest(IDBRequest*);
     void objectStoreCreated(const String&, PassRefPtr<IDBObjectStore>);
     void objectStoreDeleted(const String&);
     void setActive(bool);
     void setError(PassRefPtr<DOMError>);
 
     DEFINE_ATTRIBUTE_EVENT_LISTENER(abort);
     DEFINE_ATTRIBUTE_EVENT_LISTENER(complete);
     DEFINE_ATTRIBUTE_EVENT_LISTENER(error);
@@ skipping 13 matching lines @@
     virtual bool canSuspend() const OVERRIDE;
     virtual void stop() OVERRIDE;
 
     using RefCounted<IDBTransaction>::ref;
     using RefCounted<IDBTransaction>::deref;
 
 private:
     IDBTransaction(ScriptExecutionContext*, int64_t, const Vector<String>&, IndexedDB::TransactionMode, IDBDatabase*, IDBOpenDBRequest*, const IDBDatabaseMetadata&);
 
     void enqueueEvent(PassRefPtr<Event>);
-    void closeOpenCursors();
-
-    void registerOpenCursor(IDBCursor*);
-    void unregisterOpenCursor(IDBCursor*);
 
     // EventTarget
     virtual void refEventTarget() { ref(); }
     virtual void derefEventTarget() { deref(); }
     virtual EventTargetData* eventTargetData();
     virtual EventTargetData* ensureEventTargetData();
 
     enum State {
         Inactive, // Created or started, but not in an event callback
         Active, // Created or started, in creation scope or an event callback
@@ skipping 16 matching lines @@
     typedef HashMap<String, RefPtr<IDBObjectStore> > IDBObjectStoreMap;
     IDBObjectStoreMap m_objectStoreMap;
 
     typedef HashSet<RefPtr<IDBObjectStore> > IDBObjectStoreSet;
     IDBObjectStoreSet m_deletedObjectStores;
 
     typedef HashMap<RefPtr<IDBObjectStore>, IDBObjectStoreMetadata> IDBObjectStoreMetadataMap;
     IDBObjectStoreMetadataMap m_objectStoreCleanupMap;
     IDBDatabaseMetadata m_previousMetadata;
 
-    HashSet<IDBCursor*> m_openCursors;
-
     EventTargetData m_eventTargetData;
 };
 
 } // namespace WebCore
 
 #endif // IDBTransaction_h