Some more minor cleanup to ApplicationManager.

mojo/shell/connect_util.cc

Index: mojo/shell/connect_util.cc
diff --git a/mojo/shell/connect_util.cc b/mojo/shell/connect_util.cc
new file mode 100644
index 0000000000000000000000000000000000000000..170aa9791f4b4c2435acdcff33df57c31e89f6dd
--- /dev/null
+++ b/mojo/shell/connect_util.cc
@@ -0,0 +1,28 @@
+// Copyright 2015 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/shell/connect_util.h"
+
+#include "mojo/shell/application_manager.h"
+
+namespace mojo {
+namespace shell {
+
+ScopedMessagePipeHandle ConnectToServiceByName(
+    ApplicationManager* application_manager,
+    const GURL& application_url,
+    const std::string& interface_name) {
+  ServiceProviderPtr services;
+  scoped_ptr<ConnectToApplicationParams> params(new ConnectToApplicationParams);

[yzshen1, 2015/09/17 20:49:11]

Please consider adding includes.

+  params->SetURLInfo(application_url);
+  params->set_services(GetProxy(&services));
+  params->set_filter(GetPermissiveCapabilityFilter());
+  application_manager->ConnectToApplication(params.Pass());
+  MessagePipe pipe;
+  services->ConnectToService(interface_name, pipe.handle1.Pass());
+  return pipe.handle0.Pass();
+}
+
+}  // namespace shell
+}  // namespace mojo