Use ChannelMojo for remote security key channels.

remoting/host/security_key/fake_security_key_ipc_client.cc

 // Copyright 2016 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 "remoting/host/security_key/fake_security_key_ipc_client.h"
 
 #include <memory>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "base/run_loop.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "ipc/ipc_channel.h"
 #include "ipc/ipc_message.h"
 #include "ipc/ipc_message_macros.h"
+#include "mojo/edk/embedder/embedder.h"
+#include "mojo/edk/embedder/named_platform_handle_utils.h"
 #include "remoting/host/chromoting_messages.h"
 
 namespace remoting {
 
 FakeSecurityKeyIpcClient::FakeSecurityKeyIpcClient(
     const base::Closure& channel_event_callback)
     : channel_event_callback_(channel_event_callback), weak_factory_(this) {
   DCHECK(!channel_event_callback_.is_null());
 }
 
@@ skipping 27 matching lines @@
   }
 
   return send_security_request_should_succeed_;
 }
 
 void FakeSecurityKeyIpcClient::CloseIpcConnection() {
   client_channel_.reset();
   channel_event_callback_.Run();
 }
 
-bool FakeSecurityKeyIpcClient::ConnectViaIpc(const std::string& channel_name) {
-  // The retry loop is needed as the IPC Servers we connect to are reset (torn
-  // down and recreated) in some tests and we should be resilient in that case.
-  IPC::ChannelHandle channel_handle(channel_name);
-  for (int i = 0; i < 5; i++) {
-    client_channel_ = IPC::Channel::CreateNamedClient(channel_handle, this);
-    if (client_channel_->Connect()) {
-      return true;
-    }
+bool FakeSecurityKeyIpcClient::ConnectViaIpc(
+    const mojo::edk::NamedPlatformHandle& channel_handle) {
+  mojo::edk::ScopedPlatformHandle handle =
+      mojo::edk::CreateClientHandle(channel_handle);
+  if (!handle.is_valid())
+    return false;
 
-    base::RunLoop run_loop;
-    base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
-        FROM_HERE, run_loop.QuitClosure(),
-        base::TimeDelta::FromMilliseconds(100));
-    run_loop.Run();
-  }
-
-  return false;
+  client_channel_ = IPC::Channel::CreateClient(
+      mojo::edk::ConnectToPeerProcess(std::move(handle)).release(), this);
+  return client_channel_->Connect();
 }
 
 void FakeSecurityKeyIpcClient::SendSecurityKeyRequestViaIpc(
     const std::string& request_payload) {
   client_channel_->Send(
       new ChromotingRemoteSecurityKeyToNetworkMsg_Request(request_payload));
 }
 
 bool FakeSecurityKeyIpcClient::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(FakeSecurityKeyIpcClient, message)
-    IPC_MESSAGE_HANDLER(
-        ChromotingNetworkToRemoteSecurityKeyMsg_ConnectionDetails,
-        OnConnectionDetails)
     IPC_MESSAGE_HANDLER(ChromotingNetworkToRemoteSecurityKeyMsg_Response,
                         OnSecurityKeyResponse)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
 
   CHECK(handled) << "Received unexpected IPC type: " << message.type();
   return handled;
 }
 
 void FakeSecurityKeyIpcClient::OnChannelConnected(int32_t peer_pid) {
   ipc_channel_connected_ = true;
   channel_event_callback_.Run();
 }
 
 void FakeSecurityKeyIpcClient::OnChannelError() {
   ipc_channel_connected_ = false;
   channel_event_callback_.Run();
 }
 
-void FakeSecurityKeyIpcClient::OnConnectionDetails(
-    const std::string& channel_name) {
-  last_message_received_ = channel_name;
-  channel_event_callback_.Run();
-}
-
 void FakeSecurityKeyIpcClient::OnSecurityKeyResponse(
     const std::string& request_data) {
   last_message_received_ = request_data;
   channel_event_callback_.Run();
 }
 
 }  // namespace remoting