Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(388)

Unified Diff: chrome/common/logging_chrome.cc

Issue 2828373002: Revert of Report CHECK/DCHECK to test launcher summary output. (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/test/test_suite.cc ('k') | ios/chrome/browser/ui/collection_view/collection_view_model_unittest.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/logging_chrome.cc
diff --git a/chrome/common/logging_chrome.cc b/chrome/common/logging_chrome.cc
index 6a5d7f88ee0674418f30f65cbb50e248045e7bbb..e9e51b130ea2b85aa63275000f81fbe022f939cd 100644
--- a/chrome/common/logging_chrome.cc
+++ b/chrome/common/logging_chrome.cc
@@ -69,7 +69,6 @@
// When true, this means that error dialogs should not be shown.
bool dialogs_are_suppressed_ = false;
-logging::ScopedLogAssertHandler* assert_handler_ = nullptr;
// This should be true for exactly the period between the end of
// InitChromeLogging() and the beginning of CleanupChromeLogging().
@@ -93,10 +92,7 @@
// silenced. To record a new error, pass the log string associated
// with that error in the str parameter.
MSVC_DISABLE_OPTIMIZE();
-void SilentRuntimeAssertHandler(const char* file,
- int line,
- const base::StringPiece message,
- const base::StringPiece stack_trace) {
+void SilentRuntimeAssertHandler(const std::string& str) {
base::debug::BreakDebugger();
}
MSVC_ENABLE_OPTIMIZE();
@@ -107,8 +103,7 @@
if (dialogs_are_suppressed_)
return;
- assert_handler_ = new logging::ScopedLogAssertHandler(
- base::Bind(SilentRuntimeAssertHandler));
+ logging::SetLogAssertHandler(SilentRuntimeAssertHandler);
#if defined(OS_WIN)
UINT new_flags = SEM_FAILCRITICALERRORS |
« no previous file with comments | « base/test/test_suite.cc ('k') | ios/chrome/browser/ui/collection_view/collection_view_model_unittest.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698