Make assertions not display message boxes to the end user. This is causing...

base/logging.cc

 // Copyright (c) 2006-2008 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 472 matching lines @@
 
   if (severity_ == LOG_FATAL) {
     // display a message or break into the debugger on a fatal error
     if (DebugUtil::BeingDebugged()) {
       DebugUtil::BreakDebugger();
     } else {
       if (log_assert_handler) {
         // make a copy of the string for the handler out of paranoia
         log_assert_handler(std::string(stream_.str()));
       } else {
-        // don't use the string with the newline, get a fresh version to send to
-        // the debug message process
+        // Don't use the string with the newline, get a fresh version to send to
+        // the debug message process. We also don't display assertions to the
+        // user in release mode. The enduser can't do anything with this
+        // information, and displaying message boxes when the application is
+        // hosed can cause additional problems.
+#ifndef NDEBUG
         DisplayDebugMessage(stream_.str());
+#endif
         // Crash the process to generate a dump.
         DebugUtil::BreakDebugger();
         // TODO(mmentovai): when we have breakpad support, generate a breakpad
         // dump, but until then, do not invoke the Apple crash reporter.
       }
     }
   }
 }
 
 void CloseLogFile() {
   if (!log_file)
     return;
 
   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));
 }
-