Report CHECK/DCHECK to test launcher summary output.

base/logging.h

 // Copyright (c) 2012 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.
 
 #ifndef BASE_LOGGING_H_
 #define BASE_LOGGING_H_
 
 #include <stddef.h>
 
 #include <cassert>
 #include <cstring>
 #include <sstream>
 #include <string>
 #include <type_traits>
 #include <utility>
 
 #include "base/base_export.h"
+#include "base/callback_forward.h"
 #include "base/compiler_specific.h"
 #include "base/debug/debugger.h"
 #include "base/macros.h"
 #include "base/template_util.h"
 #include "build/build_config.h"
 
 //
 // Optional message capabilities
 // -----------------------------
 // Assertion failed messages and fatal errors are displayed in a dialog box
@@ skipping 242 matching lines @@
 
 // Sets whether or not you'd like to see fatal debug messages popped up in
 // a dialog box or not.
 // Dialogs are not shown by default.
 BASE_EXPORT void SetShowErrorDialogs(bool enable_dialogs);
 
 // Sets the Log Assert Handler that will be used to notify of check failures.
 // The default handler shows a dialog box and then terminate the process,
 // however clients can use this function to override with their own handling
 // (e.g. a silent one for Unit Tests)
-typedef void (*LogAssertHandlerFunction)(const std::string& str);
+typedef base::Callback<void(const char* file,
+                            int line,
+                            size_t message_start,

[Paweł Hajdan Jr., 2017/01/27 17:19:44]

I'm worried these indexes are a weird API.

Why not pass two strings instead, one for the message and second one for the
stack trace?

[alex-ac, 2017/02/11 20:12:19]

Done.

+                            size_t stack_start,
+                            const std::string& str)>
+    LogAssertHandlerFunction;
+
+// You probably must use ScopedLogAssertHandler instead of this functions.

[Paweł Hajdan Jr., 2017/01/27 17:19:44]

nit: "Probably" and "must" don't mix well for me. Could you change this to
something like "consider" or similar.

[alex-ac, 2017/02/11 20:12:19]

Done.

+// see base/test/logging_utils.h.
 BASE_EXPORT void SetLogAssertHandler(LogAssertHandlerFunction handler);

[Paweł Hajdan Jr., 2017/01/27 17:19:44]

How about changing the names to more like Push/Pop? That reflects the new
semantics better than Set/Reset.

[alex-ac, 2017/02/11 20:12:19]

Done.

+BASE_EXPORT void ResetLogAssertHandler();
 
 // Sets the Log Message Handler that gets passed every log message before
 // it's sent to other log destinations (if any).
 // Returns true to signal that it handled the message and the message
 // should not be sent to other log destinations.
 typedef bool (*LogMessageHandlerFunction)(int severity,
     const char* file, int line, size_t message_start, const std::string& str);
 BASE_EXPORT void SetLogMessageHandler(LogMessageHandlerFunction handler);
 BASE_EXPORT LogMessageHandlerFunction GetLogMessageHandler();
 
@@ skipping 726 matching lines @@
 #elif NOTIMPLEMENTED_POLICY == 5
 #define NOTIMPLEMENTED() do {\
   static bool logged_once = false;\
   LOG_IF(ERROR, !logged_once) << NOTIMPLEMENTED_MSG;\
   logged_once = true;\
 } while(0);\
 EAT_STREAM_PARAMETERS
 #endif
 
 #endif  // BASE_LOGGING_H_