Fixing SecurityKeySocket unit test issues and cleanup

remoting/host/security_key/security_key_extension_session.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/security_key_extension_session.h"
 
 #include "base/bind.h"
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/logging.h"
@@ skipping 153 matching lines @@
   int connection_id;
   if (!message_data->GetInteger(kConnectionId, &connection_id)) {
     LOG(WARNING) << "Could not extract connection id from message.";
     return;
   }
 
   if (security_key_auth_handler_->IsValidConnectionId(connection_id)) {
     HOST_LOG << "Sending security key error";
     security_key_auth_handler_->SendErrorAndCloseConnection(connection_id);
   } else {
-    LOG(WARNING) << "Unknown gnubby-auth data connection: '" << connection_id
-                 << "'";
+    LOG(WARNING) << "Unknown gnubby-auth connection id: " << connection_id;
   }
 }
 
 void SecurityKeyExtensionSession::SendMessageToClient(
     int connection_id,
     const std::string& data) const {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(client_stub_);
 
   base::DictionaryValue request;
@@ skipping 20 matching lines @@
     std::unique_ptr<SecurityKeyAuthHandler> security_key_auth_handler) {
   DCHECK(security_key_auth_handler);
 
   security_key_auth_handler_ = std::move(security_key_auth_handler);
   security_key_auth_handler_->SetSendMessageCallback(
       base::Bind(&SecurityKeyExtensionSession::SendMessageToClient,
                  base::Unretained(this)));
 }
 
 }  // namespace remoting