Move Shell connection to content

mojo/shell/application_manager.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/shell/application_manager.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/macros.h"
@@ skipping 104 matching lines @@
   const auto& it = identity_to_instance_.find(identity);
   return it != identity_to_instance_.end() ? it->second : nullptr;
 }
 
 void ApplicationManager::CreateInstanceForHandle(ScopedHandle channel,
                                                  const GURL& url,
                                                  const std::string& qualifier) {
   // Instances created by others are considered unique, and thus have no
   // identity. As such they cannot be connected to by anyone else, and so we
   // never call ConnectToClient().
+  // TODO(beng): GetPermissiveCapabilityFilter() here obviously cannot make it
+  //             to production. See note in application_manager.mojom.
+  //             http://crbug.com/555392
   Identity target_id(url, qualifier, GetPermissiveCapabilityFilter());
   InterfaceRequest<Application> application_request =
       CreateInstance(target_id, base::Closure(), nullptr);
   NativeRunner* runner =
       native_runner_factory_->Create(base::FilePath()).release();
   native_runners_.push_back(runner);
   runner->InitHost(channel.Pass(), application_request.Pass());
 }
 
 InterfaceRequest<Application> ApplicationManager::CreateAndConnectToInstance(
@@ skipping 151 matching lines @@
   native_runners_.erase(
       std::find(native_runners_.begin(), native_runners_.end(), runner));
 }
 
 Shell::ConnectToApplicationCallback EmptyConnectCallback() {
   return base::Bind(&OnEmptyOnConnectCallback);
 }
 
 }  // namespace shell
 }  // namespace mojo