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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/public/test/nested_message_pump_android.cc

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 years 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
« chrome/browser/autofill/form_structure_browsertest.cc ('K') | « content/public/test/browser_test_base.cc ('k') | content/public/test/test_fileapi_operation_waiter.cc » ('j') | remoting/host/curtain_mode_mac.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/test/nested_message_pump_android.cc
diff --git a/content/public/test/nested_message_pump_android.cc b/content/public/test/nested_message_pump_android.cc
index 1158c1938c174433dcb8d89f32088a0e3adc5316..17a7b92a08d9e0b714deda1a5e08ef860dc73fc9 100644
--- a/content/public/test/nested_message_pump_android.cc
+++ b/content/public/test/nested_message_pump_android.cc
@@ -94,7 +94,8 @@ void NestedMessagePumpAndroid::Run(Delegate* delegate) {
// is idle.
bool ret = Java_NestedSystemMessageHandler_runNestedLoopTillIdle(
env, g_message_handler_obj.Get());
- CHECK(ret) << "Error running java message loop, tests will likely fail.";
+ // Error running java message loop, tests will likely fail.
+ CHECK(ret);
base::ThreadRestrictions::ScopedAllowWait allow_wait;
if (state_->delayed_work_time.is_null()) {
« chrome/browser/autofill/form_structure_browsertest.cc ('K') | « content/public/test/browser_test_base.cc ('k') | content/public/test/test_fileapi_operation_waiter.cc » ('j') | remoting/host/curtain_mode_mac.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698