Do not abort the process if thread owning the log mutex has crashed.

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);
-      DCHECK(r != WAIT_ABANDONED);
+      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.";

[cpu_(ooo_6.6-7.5), 2009/07/31 23:04:05]

hmmm ... you are calling LOG()which ends up back here?

I don't see anybody calling back into log here, I think for a a good reason.
Probably that DCHECK was also wrong.

+      }
 #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));
 }