Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(598)

Unified Diff: mojo/application/public/cpp/application_impl.h

Issue 1200323003: media: Quit MojoMediaApplication when no service is bound. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 7036d33dbc6b72beff66800258d101ca20a093db..c76488f6aad9e089c7d6c7b30282fb86b0435194 100644
--- a/mojo/application/public/cpp/application_impl.h
+++ b/mojo/application/public/cpp/application_impl.h
@@ -80,8 +80,11 @@ class ApplicationImpl : public Application,
// 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, or until the
- // ApplicationImpl is destroyed, whichever occurs first.
+ // ApplicationImpl is destroyed, whichever occurs first. If |error_cb| is
+ // provided, it will be fired when the application is terminated.
ApplicationConnection* ConnectToApplication(mojo::URLRequestPtr request);
+ ApplicationConnection* ConnectToApplication(mojo::URLRequestPtr request,
+ const base::Closure& error_cb);
sandersd (OOO until July 31) 2015/06/30 21:32:47 It's true that this is caused by a connection erro
xhwang 2015/07/07 20:29:24 Done.
// Connect to application identified by |request->url| and connect to the
// service implementation of the interface identified by |Interface|.

Powered by Google App Engine
This is Rietveld 408576698