[IndexedDB] Closing mojo connections when renderer quits

mojo/public/cpp/bindings/associated_binding.h

Index: mojo/public/cpp/bindings/associated_binding.h
diff --git a/mojo/public/cpp/bindings/associated_binding.h b/mojo/public/cpp/bindings/associated_binding.h
index 59411666f5aca80a6e05953070ccb821ef2040e3..177af29ff590236bda6cea6ff8ed8c1d68823db1 100644
--- a/mojo/public/cpp/bindings/associated_binding.h
+++ b/mojo/public/cpp/bindings/associated_binding.h
@@ -98,7 +98,9 @@ class AssociatedBinding : public AssociatedBindingBase {
   // implementation |impl|. It may be completed with a subsequent call to the
   // |Bind| method. Does not take ownership of |impl|, which must outlive this

[cmumford, 2017/03/07 20:24:23]

Doesn't moving |impl| mean that we're taking ownership of |impl|?

[dmurph, 2017/03/09 20:44:15]

Done.

   // object.
-  explicit AssociatedBinding(ImplPointerType impl) { stub_.set_sink(impl); }
+  explicit AssociatedBinding(ImplPointerType impl) {
+    stub_.set_sink(std::move(impl));
+  }
 
   // Constructs a completed associated binding of |impl|. The output |ptr_info|
   // should be sent by another interface. |impl| must outlive this object.