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

Unified Diff: content/browser/webrtc/webrtc_getusermedia_browsertest.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
Index: content/browser/webrtc/webrtc_getusermedia_browsertest.cc
diff --git a/content/browser/webrtc/webrtc_getusermedia_browsertest.cc b/content/browser/webrtc/webrtc_getusermedia_browsertest.cc
index 36bb9a64721b67b898decb7dd4ac7374e2be8838..a54ad7d74c64663504841dab4646db3e86b400c1 100644
--- a/content/browser/webrtc/webrtc_getusermedia_browsertest.cc
+++ b/content/browser/webrtc/webrtc_getusermedia_browsertest.cc
@@ -108,7 +108,8 @@ class WebRtcGetUserMediaBrowserTest : public WebRtcContentBrowserTestBase {
~WebRtcGetUserMediaBrowserTest() override {}
void StartTracing() {
- CHECK(trace_log_ == NULL) << "Can only can start tracing once";
+ // Can only can start tracing once
+ CHECK(trace_log_ == NULL);
trace_log_ = base::trace_event::TraceLog::GetInstance();
base::trace_event::TraceConfig trace_config(
"video", base::trace_event::RECORD_UNTIL_FULL);
@@ -119,8 +120,8 @@ class WebRtcGetUserMediaBrowserTest : public WebRtcContentBrowserTestBase {
}
void StopTracing() {
- CHECK(message_loop_runner_.get() == NULL)
- << "Calling StopTracing more than once";
+ // Calling StopTracing more than once
+ CHECK(message_loop_runner_.get() == NULL);
{
base::ThreadRestrictions::ScopedAllowIO allow_thread_join_caused_by_test;

Powered by Google App Engine
This is Rietveld 408576698