Some more minor cleanup to ApplicationManager.

mojo/runner/context.cc

 // Copyright 2013 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/runner/context.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 191 matching lines @@
   else
     runner_factory.reset(new InProcessNativeRunnerFactory(this));
   application_manager_->set_blocking_pool(task_runners_->blocking_pool());
   application_manager_->set_native_runner_factory(runner_factory.Pass());
 
   ServiceProviderPtr service_provider_ptr;
   ServiceProviderPtr tracing_service_provider_ptr;
   new TracingServiceProvider(GetProxy(&tracing_service_provider_ptr));
   mojo::URLRequestPtr request(mojo::URLRequest::New());
   request->url = mojo::String::From("mojo:tracing");
-  application_manager_->ConnectToApplication(
-      nullptr, request.Pass(), std::string(), GetProxy(&service_provider_ptr),
-      tracing_service_provider_ptr.Pass(),
-      shell::GetPermissiveCapabilityFilter(), base::Closure(),
-      shell::EmptyConnectCallback());
+
+  scoped_ptr<shell::ConnectToApplicationParams> params(
+      new shell::ConnectToApplicationParams);
+  params->SetURLInfo(request.Pass());
+  params->set_services(GetProxy(&service_provider_ptr));
+  params->set_exposed_services(tracing_service_provider_ptr.Pass());
+  params->set_filter(shell::GetPermissiveCapabilityFilter());
+  application_manager_->ConnectToApplication(params.Pass());
 
   // Record the shell startup metrics used for performance testing.
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(
           tracing::kEnableStatsCollectionBindings)) {
     tracing::StartupPerformanceDataCollectorPtr collector;
     service_provider_ptr->ConnectToService(
         tracing::StartupPerformanceDataCollector::Name_,
         GetProxy(&collector).PassMessagePipe());
 #if defined(OS_MACOSX) || defined(OS_WIN) || defined(OS_LINUX)
     // CurrentProcessInfo::CreationTime is only defined on some platforms.
@@ skipping 29 matching lines @@
 }
 
 void Context::Run(const GURL& url) {
   DCHECK(app_complete_callback_.is_null());
   ServiceProviderPtr services;
   ServiceProviderPtr exposed_services;
 
   app_urls_.insert(url);
   mojo::URLRequestPtr request(mojo::URLRequest::New());
   request->url = mojo::String::From(url.spec());
-  application_manager_->ConnectToApplication(
-      nullptr, request.Pass(), std::string(), GetProxy(&services),
-      exposed_services.Pass(), shell::GetPermissiveCapabilityFilter(),
-      base::Bind(&Context::OnApplicationEnd, base::Unretained(this), url),
-      shell::EmptyConnectCallback());
+
+  scoped_ptr<shell::ConnectToApplicationParams> params(
+      new shell::ConnectToApplicationParams);
+  params->SetURLInfo(request.Pass());
+  params->set_services(GetProxy(&services));
+  params->set_exposed_services(exposed_services.Pass());
+  params->set_filter(shell::GetPermissiveCapabilityFilter());
+  params->set_on_application_end(
+      base::Bind(&Context::OnApplicationEnd, base::Unretained(this), url));
+  application_manager_->ConnectToApplication(params.Pass());
 }
 
 void Context::RunCommandLineApplication(const base::Closure& callback) {
   DCHECK(app_urls_.empty());
   DCHECK(app_complete_callback_.is_null());
   base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
   base::CommandLine::StringVector args = command_line->GetArgs();
   for (size_t i = 0; i < args.size(); ++i) {
     GURL possible_app(args[i]);
     if (possible_app.SchemeIs("mojo")) {
@@ skipping 14 matching lines @@
         base::MessageLoop::current()->Quit();
       } else {
         app_complete_callback_.Run();
       }
     }
   }
 }
 
 }  // namespace runner
 }  // namespace mojo