Changing the default path for RemoteSecurityKeyIpcServer test channels

remoting/host/security_key/security_key_ipc_constants.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_ipc_constants.h"
 
-#include "base/environment.h"
+#include "base/files/file_path.h"
+#include "base/files/file_util.h"
 #include "base/lazy_instance.h"
+#include "base/macros.h"

[Lei Zhang, 2016/07/19 22:55:04]

It's great that you are trying to IWYU, but what is this for? OS_POSIX?
LOG(ERROR) ? Either way, this is the wrong header.

OS_POSIX is in build/build_config.h
LOG(ERROR) is in base/logging.h

[joedow, 2016/07/19 23:04:12]

Done.

 
 namespace {
 base::LazyInstance<std::string> g_security_key_ipc_channel_name =
     LAZY_INSTANCE_INITIALIZER;
 
 const char kSecurityKeyIpcChannelName[] = "security_key_ipc_channel";
 
 }  // namespace
 
 namespace remoting {
 
-extern const char kSecurityKeyConnectionError[] = "ssh_connection_error";
+const char kSecurityKeyConnectionError[] = "ssh_connection_error";
 
 const std::string& GetSecurityKeyIpcChannelName() {
   if (g_security_key_ipc_channel_name.Get().empty()) {
     g_security_key_ipc_channel_name.Get() = kSecurityKeyIpcChannelName;
   }
 
   return g_security_key_ipc_channel_name.Get();
 }
 
 void SetSecurityKeyIpcChannelNameForTest(const std::string& channel_name) {
   g_security_key_ipc_channel_name.Get() = channel_name;
 }
 
 std::string GetChannelNamePathPrefixForTest() {
-  std::string path_prefix;
-#if defined(OS_LINUX)
-  path_prefix = "/dev/socket/";
-  std::unique_ptr<base::Environment> env(base::Environment::Create());
-  if (env->GetVar(base::env_vars::kHome, &path_prefix))
-    path_prefix += "/";
+  std::string base_path;
+#if defined(OS_POSIX)
+  base::FilePath base_file_path;
+  if (!base::GetTempDir(&base_file_path)) {
+    LOG(ERROR) << "Failed to retrieve temporary directory.";
+  } else {
+    base_path = base_file_path.value() + "/";
+  }
 #endif
-  return path_prefix;
+  return base_path;
 }
 
 }  // namespace remoting