media: Quit MojoMediaApplication when no ServiceFactory service is bound.

mojo/application/public/cpp/application_impl.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MOJO_APPLICATION_PUBLIC_CPP_APPLICATION_IMPL_H_
 #define MOJO_APPLICATION_PUBLIC_CPP_APPLICATION_IMPL_H_
 
 #include <vector>
 
 #include "base/callback.h"
@@ skipping 62 matching lines @@
 
   const std::string& url() const { return url_; }
 
   AppLifetimeHelper* app_lifetime_helper() { return &app_lifetime_helper_; }
 
   // Requests a new connection to an application. Returns a pointer to the
   // connection if the connection is permitted by this application's delegate,
   // 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);
 
   // Connect to application identified by |request->url| and connect to the
   // service implementation of the interface identified by |Interface|.
   template <typename Interface>
   void ConnectToService(mojo::URLRequestPtr request,
                         InterfacePtr<Interface>* ptr) {
     ApplicationConnection* connection = ConnectToApplication(request.Pass());
@@ skipping 46 matching lines @@
   bool quit_requested_;
   bool in_destructor_;
   base::WeakPtrFactory<ApplicationImpl> weak_factory_;
 
   MOJO_DISALLOW_COPY_AND_ASSIGN(ApplicationImpl);
 };
 
 }  // namespace mojo
 
 #endif  // MOJO_APPLICATION_PUBLIC_CPP_APPLICATION_IMPL_H_