[IndexedDB] Closing mojo connections when renderer quits

content/browser/indexed_db/indexed_db_callbacks.h

Index: content/browser/indexed_db/indexed_db_callbacks.h
diff --git a/content/browser/indexed_db/indexed_db_callbacks.h b/content/browser/indexed_db/indexed_db_callbacks.h
index b72fc605e8cd53ff426a5fc3e80cafc062876907..e42fce092ad30c002913dde417583c4473f486e0 100644
--- a/content/browser/indexed_db/indexed_db_callbacks.h
+++ b/content/browser/indexed_db/indexed_db_callbacks.h
@@ -40,7 +40,7 @@ class CONTENT_EXPORT IndexedDBCallbacks
       IndexedDBValue* value);
 
   IndexedDBCallbacks(
-      scoped_refptr<IndexedDBDispatcherHost> dispatcher_host,
+      IndexedDBDispatcherHost* dispatcher_host,
       const url::Origin& origin,
       ::indexed_db::mojom::CallbacksAssociatedPtrInfo callbacks_info);
 
@@ -107,7 +107,7 @@ class CONTENT_EXPORT IndexedDBCallbacks
   class IOThreadHelper;
 
   // Originally from IndexedDBCallbacks:
-  scoped_refptr<IndexedDBDispatcherHost> dispatcher_host_;
+  IndexedDBDispatcherHost* dispatcher_host_;

[Reilly Grant (use Gerrit), 2017/03/10 00:11:22]

I'm less confident that this is safe unless we can be sure that destruction of
an IndexedDBConnection guarantees that this will be freed.

[dmurph, 2017/03/10 23:40:26]

Removed. All raw pointers are removed except for ones we can prove lifetime
ownership.

 
   // IndexedDBDatabase callbacks ------------------------
   url::Origin origin_;