ApplicationConnection lifetime management changes.

mandoline/ui/browser/desktop/desktop_ui.cc

Index: mandoline/ui/browser/desktop/desktop_ui.cc
diff --git a/mandoline/ui/browser/desktop/desktop_ui.cc b/mandoline/ui/browser/desktop/desktop_ui.cc
index 889ee6459e578bc077ff6308c0ab4114d1266054..de42e480de5090a1ea91163fd8758d9b4e847a69 100644
--- a/mandoline/ui/browser/desktop/desktop_ui.cc
+++ b/mandoline/ui/browser/desktop/desktop_ui.cc
@@ -151,7 +151,7 @@ void DesktopUI::ButtonPressed(views::Button* sender, const ui::Event& event) {
     DCHECK(!client_binding_.is_bound());
     mojo::URLRequestPtr request(mojo::URLRequest::New());
     request->url = mojo::String::From("mojo:omnibox");
-    mojo::ApplicationConnection* application_connection =
+    scoped_ptr<mojo::ApplicationConnection> application_connection =

[sky, 2015/08/11 18:28:04]

Because you're putting this connection in a scoped_ptr won't the ViewEmbedder
that is registered on the next line get lost? In other words don't you need to
keep this connection around forever?

         application_impl_->ConnectToApplication(request.Pass());
     application_connection->AddService<ViewEmbedder>(browser_);
     application_connection->ConnectToService(&omnibox_);