[IndexedDB] Closing mojo connections when renderer quits

content/browser/indexed_db/database_impl.h

Index: content/browser/indexed_db/database_impl.h
diff --git a/content/browser/indexed_db/database_impl.h b/content/browser/indexed_db/database_impl.h
index bb5e9f0fb7183d1af51906332899f9e3760188be..f48ff4a961a8e4369b8adb426ea4212ce8ff5673 100644
--- a/content/browser/indexed_db/database_impl.h
+++ b/content/browser/indexed_db/database_impl.h
@@ -18,7 +18,7 @@ class DatabaseImpl : public ::indexed_db::mojom::Database {
  public:
   explicit DatabaseImpl(std::unique_ptr<IndexedDBConnection> connection,
                         const url::Origin& origin,
-                        scoped_refptr<IndexedDBDispatcherHost> dispatcher_host);
+                        IndexedDBDispatcherHost* dispatcher_host);
   ~DatabaseImpl() override;
 
   // ::indexed_db::mojom::Database implementation
@@ -60,7 +60,7 @@ class DatabaseImpl : public ::indexed_db::mojom::Database {
       ::indexed_db::mojom::CallbacksAssociatedPtrInfo callbacks) override;
   void Put(int64_t transaction_id,
            int64_t object_store_id,
-           indexed_db::mojom::ValuePtr value,
+           ::indexed_db::mojom::ValuePtr value,
            const IndexedDBKey& key,
            blink::WebIDBPutMode mode,
            const std::vector<IndexedDBIndexKeys>& index_keys,
@@ -118,7 +118,7 @@ class DatabaseImpl : public ::indexed_db::mojom::Database {
   class IDBThreadHelper;
 
   IDBThreadHelper* helper_;
-  scoped_refptr<IndexedDBDispatcherHost> dispatcher_host_;
+  IndexedDBDispatcherHost* dispatcher_host_;

[Reilly Grant (use Gerrit), 2017/03/03 01:59:59]

Add comment:

// This raw pointer is safe because all DatabaseImpl instances are owned by an
IndexedDBDispatcherHost.

[dmurph, 2017/03/04 00:39:41]

Done.

   const url::Origin origin_;
   scoped_refptr<base::SingleThreadTaskRunner> idb_runner_;