base: Remove the string logging from CHECK().

chrome/browser/process_singleton_posix.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.
 
 // On Linux, when the user tries to launch a second copy of chrome, we check
 // for a socket in the user's profile directory.  If the socket file is open we
 // send a message to the first chrome browser process with the current
 // directory and second process command line flags.  The second process then
 // exits.
 //
@@ skipping 202 matching lines @@
       bytes_read += rv;
     }
   } while (bytes_read < bufsize);
 
   return bytes_read;
 }
 
 // Set up a sockaddr appropriate for messaging.
 void SetupSockAddr(const std::string& path, struct sockaddr_un* addr) {
   addr->sun_family = AF_UNIX;
-  CHECK(path.length() < arraysize(addr->sun_path))
-      << "Socket path too long: " << path;
+  // Socket path too long.
+  CHECK(path.length() < arraysize(addr->sun_path));
   base::strlcpy(addr->sun_path, path.c_str(), arraysize(addr->sun_path));
 }
 
 // Set up a socket appropriate for messaging.
 int SetupSocketOnly() {
   int sock = socket(PF_UNIX, SOCK_STREAM, 0);
-  PCHECK(sock >= 0) << "socket() failed";
+  // socket() failed
+  CHECK(sock >= 0);
 
   DCHECK(base::SetNonBlocking(sock)) << "Failed to make non-blocking socket.";
   int rv = SetCloseOnExec(sock);
   DCHECK_EQ(0, rv) << "Failed to set CLOEXEC on socket.";
 
   return sock;
 }
 
 // Set up a socket and sockaddr appropriate for messaging.
 void SetupSocket(const std::string& path, int* sock, struct sockaddr_un* addr) {
@@ skipping 727 matching lines @@
   }
 
   // Create the socket file somewhere in /tmp which is usually mounted as a
   // normal filesystem. Some network filesystems (notably AFS) are screwy and
   // do not support Unix domain sockets.
   if (!socket_dir_.CreateUniqueTempDir()) {
     LOG(ERROR) << "Failed to create socket directory.";
     return false;
   }
 
-  // Check that the directory was created with the correct permissions.
+  // Check that the directory was created with the correct permissions (0700).
   int dir_mode = 0;
   CHECK(base::GetPosixFilePermissions(socket_dir_.GetPath(), &dir_mode) &&
-        dir_mode == base::FILE_PERMISSION_USER_MASK)
-      << "Temp directory mode is not 700: " << std::oct << dir_mode;
+        dir_mode == base::FILE_PERMISSION_USER_MASK);
 
   // Setup the socket symlink and the two cookies.
   base::FilePath socket_target_path =
       socket_dir_.GetPath().Append(chrome::kSingletonSocketFilename);
   base::FilePath cookie(GenerateCookie());
   base::FilePath remote_cookie_path =
       socket_dir_.GetPath().Append(chrome::kSingletonCookieFilename);
   UnlinkPath(socket_path_);
   UnlinkPath(cookie_path_);
   if (!SymlinkPath(socket_target_path, socket_path_) ||
@@ skipping 70 matching lines @@
 }
 
 void ProcessSingleton::KillProcess(int pid) {
   // TODO(james.su@gmail.com): Is SIGKILL ok?
   int rv = kill(static_cast<base::ProcessHandle>(pid), SIGKILL);
   // ESRCH = No Such Process (can happen if the other process is already in
   // progress of shutting down and finishes before we try to kill it).
   DCHECK(rv == 0 || errno == ESRCH) << "Error killing process: "
                                     << base::safe_strerror(errno);
 }