Remove even the logging statement from ~LogMessage. We shouldn't be using LOG macros there.

base/logging.cc

 // Copyright (c) 2006-2009 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 "base/logging.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 typedef HANDLE FileHandle;
 typedef HANDLE MutexHandle;
@@ skipping 467 matching lines @@
       InitializeLogFileHandle()) {
     // We can have multiple threads and/or processes, so try to prevent them
     // from clobbering each other's writes.
     if (lock_log_file == LOCK_LOG_FILE) {
       // Ensure that the mutex is initialized in case the client app did not
       // call InitLogging. This is not thread safe. See below.
       InitLogMutex();
 
 #if defined(OS_WIN)
       DWORD r = ::WaitForSingleObject(log_mutex, INFINITE);
-      if (r == WAIT_ABANDONED) {
-        // Do not abort the process here. UI tests might be crashy sometimes,
-        // and aborting the test binary only makes the problem worse.
-        // For more info see http://crbug.com/18028.
-        LOG(ERROR) << "Thread owning the log mutex has crashed.";
-      }
+      // WaitForSingleObject could have returned WAIT_ABANDONED. We don't
+      // abort the process here. UI tests might be crashy sometimes,
+      // and aborting the test binary only makes the problem worse.
+      // We also don't use LOG macros because that might lead to an infinite
+      // loop. For more info see http://crbug.com/18028.
 #elif defined(OS_POSIX)
       pthread_mutex_lock(&log_mutex);
 #endif
     } else {
       // use the lock
       if (!log_lock) {
         // The client app did not call InitLogging, and so the lock has not
         // been created. We do this on demand, but if two threads try to do
         // this at the same time, there will be a race condition to create
         // the lock. This is why InitLogging should be called from the main
@@ skipping 70 matching lines @@
 
   CloseFile(log_file);
   log_file = NULL;
 }
 
 }  // namespace logging
 
 std::ostream& operator<<(std::ostream& out, const wchar_t* wstr) {
   return out << base::SysWideToUTF8(std::wstring(wstr));
 }