ApplicationConnection lifetime management changes.

mojo/common/tracing_impl.cc

 // 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.
 
 #include "mojo/common/tracing_impl.h"
 
 #include "base/trace_event/trace_event.h"
 #include "mojo/application/public/cpp/application_connection.h"
 #include "mojo/application/public/cpp/application_impl.h"
 #include "mojo/common/trace_controller_impl.h"
 
 namespace mojo {
 
 TracingImpl::TracingImpl() {
 }
 
 TracingImpl::~TracingImpl() {
 }
 
 void TracingImpl::Initialize(ApplicationImpl* app) {
   mojo::URLRequestPtr request(mojo::URLRequest::New());
   request->url = mojo::String::From("mojo:tracing");
-  ApplicationConnection* connection = app->ConnectToApplication(request.Pass());
+  scoped_ptr<ApplicationConnection> connection =

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

Don't you need to persist this connection, else the AddService() doesn't stay
around?

+      app->ConnectToApplication(request.Pass());
   connection->AddService(this);
 }
 
 void TracingImpl::Create(ApplicationConnection* connection,
                          InterfaceRequest<tracing::TraceController> request) {
   new TraceControllerImpl(request.Pass());
 }
 
 }  // namespace mojo