Change signature of OnConnect

services/shell/public/cpp/lib/connection_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 "services/shell/public/cpp/lib/connection_impl.h"
 
 #include <stdint.h>
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "services/shell/public/cpp/connection.h"
 #include "services/shell/public/cpp/interface_binder.h"
 
 namespace shell {
 namespace internal {
 
 ////////////////////////////////////////////////////////////////////////////////
 // ConnectionImpl, public:
 
 ConnectionImpl::ConnectionImpl()
-    : allow_all_interfaces_(true),
-      weak_factory_(this) {}
+    : weak_factory_(this) {}
 
-ConnectionImpl::ConnectionImpl(
-    const Identity& remote,
-    const CapabilityRequest& capability_request,
-    State initial_state)
+ConnectionImpl::ConnectionImpl(const Identity& remote, State initial_state)
     : remote_(remote),
       state_(initial_state),
-      capability_request_(capability_request),
-      allow_all_interfaces_(capability_request.interfaces.size() == 1 &&
-                            capability_request.interfaces.count("*") == 1),
       weak_factory_(this) {
 }
 
 ConnectionImpl::~ConnectionImpl() {}
 
-void ConnectionImpl::SetExposedInterfaces(
-    std::unique_ptr<InterfaceRegistry> exposed_interfaces) {
-  exposed_interfaces_owner_ = std::move(exposed_interfaces);
-  set_exposed_interfaces(exposed_interfaces_owner_.get());
-}
-
 void ConnectionImpl::SetRemoteInterfaces(
     std::unique_ptr<InterfaceProvider> remote_interfaces) {
   remote_interfaces_owner_ = std::move(remote_interfaces);
   set_remote_interfaces(remote_interfaces_owner_.get());
 }
 
 shell::mojom::Connector::ConnectCallback ConnectionImpl::GetConnectCallback() {
   return base::Bind(&ConnectionImpl::OnConnectionCompleted,
                     weak_factory_.GetWeakPtr());
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // ConnectionImpl, Connection implementation:
 
-bool ConnectionImpl::HasCapabilityClass(const std::string& class_name) const {
-  return capability_request_.classes.count(class_name) > 0;
-}
-
 const Identity& ConnectionImpl::GetRemoteIdentity() const {
   return remote_;
 }
 
 void ConnectionImpl::SetConnectionLostClosure(const base::Closure& handler) {
-  if (remote_interfaces_)
-    remote_interfaces_->SetConnectionLostClosure(handler);
-  else
-    exposed_interfaces_->SetConnectionLostClosure(handler);
+  remote_interfaces_->SetConnectionLostClosure(handler);
 }
 
 shell::mojom::ConnectResult ConnectionImpl::GetResult() const {
   return result_;
 }
 
 bool ConnectionImpl::IsPending() const {
   return state_ == State::PENDING;
 }
 
 void ConnectionImpl::AddConnectionCompletedClosure(
     const base::Closure& callback) {
   if (IsPending())
     connection_completed_callbacks_.push_back(callback);
   else
     callback.Run();
 }
 
-bool ConnectionImpl::AllowsInterface(const std::string& interface_name) const {
-  return allow_all_interfaces_ ||
-         capability_request_.interfaces.count(interface_name);
-}
-
-InterfaceRegistry* ConnectionImpl::GetInterfaceRegistry() {
-  return exposed_interfaces_;
-}
-
 InterfaceProvider* ConnectionImpl::GetRemoteInterfaces() {
   return remote_interfaces_;
 }
 
 base::WeakPtr<Connection> ConnectionImpl::GetWeakPtr() {
   return weak_factory_.GetWeakPtr();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // ConnectionImpl, private:
 
 void ConnectionImpl::OnConnectionCompleted(shell::mojom::ConnectResult result,
                                            mojo::String target_user_id) {
   DCHECK(State::PENDING == state_);
 
   result_ = result;
   state_ = result_ == shell::mojom::ConnectResult::SUCCEEDED ?
       State::CONNECTED : State::DISCONNECTED;
   remote_.set_user_id(target_user_id);
   std::vector<base::Closure> callbacks;
   callbacks.swap(connection_completed_callbacks_);
   for (auto callback : callbacks)
     callback.Run();
 }
 
 }  // namespace internal
 }  // namespace shell