media: Quit MojoMediaApplication when no ServiceFactory service is bound.

mojo/application/public/cpp/application_impl.h

Index: mojo/application/public/cpp/application_impl.h
diff --git a/mojo/application/public/cpp/application_impl.h b/mojo/application/public/cpp/application_impl.h
index d3fb716b2118dc4a427c518ec13a50bc944bb30f..acfe27432e79d99c2c664e3045fa998ab561af46 100644
--- a/mojo/application/public/cpp/application_impl.h
+++ b/mojo/application/public/cpp/application_impl.h
@@ -80,8 +80,12 @@ class ApplicationImpl : public Application {
   // or nullptr otherwise. Caller does not take ownership. The pointer remains
   // valid until an error occurs on the connection with the Shell, until the
   // ApplicationImpl is destroyed, or until the connection is closed through a
-  // call to ApplicationConnection::CloseConnection.
+  // call to ApplicationConnection::CloseConnection. If |terminate_cb| is

[sky, 2015/07/22 23:56:33]

It's not when the app is terminated, but rather when there is a connection error
on the service provider. That said, I don't think this is the right place for
setting the error handler. I think it makes more sense on ApplicationConnection.
And confusingly we shouldn't use base types here. Use the one defined in
third_party/mojo/src/mojo/public/cpp/bindings/callback.h.

[xhwang, 2015/07/23 21:57:15]

Done.
Done.
Done.

+  // provided, it will be fired when the application is terminated.
   ApplicationConnection* ConnectToApplication(mojo::URLRequestPtr request);
+  ApplicationConnection* ConnectToApplication(
+      mojo::URLRequestPtr request,
+      const base::Closure& terminate_cb);
 
   // Closes the |connection|.
   void CloseConnection(ApplicationConnection* connection);